Fix some sign-compare warnings in webrtc/api.

Fix some sign-compare warnings in webrtc/api.

The disabling of the warnings doesn't seem to work when Chromium
is using our targets (https://codereview.chromium.org/2022833002)
so better fix them.

BUG=webrtc:4256, webrtc:3307
NOTRY=True

Committed: https://crrev.com/3e33bfeb6d4a7b9e14ce856e849f5e0367689e09
Cr-Commit-Position: refs/heads/master@{#13217}

[kjellander_webrtc, 2016-06-20 10:00:55]

kjellander@webrtc.org changed reviewers:
+ perkj@webrtc.org

[kjellander_webrtc, 2016-06-20 13:52:05]

kjellander@webrtc.org changed reviewers:
+ henrika@webrtc.org

[kjellander_webrtc, 2016-06-20 13:52:06]

+henrika since perkj is OOO today.

[henrika_webrtc, 2016-06-20 13:55:55]

LGTM with nit comment

https://codereview.webrtc.org/2074423002/diff/20001/webrtc/api/webrtcsdp.cc
File webrtc/api/webrtcsdp.cc (right):

https://codereview.webrtc.org/2074423002/diff/20001/webrtc/api/webrtcsdp.cc#n...
webrtc/api/webrtcsdp.cc:2190: int payload_type = *it;
can't you do 

size_t payload_type = *it 

here instead

[kjellander_webrtc, 2016-06-20 14:02:26]

https://codereview.webrtc.org/2074423002/diff/20001/webrtc/api/webrtcsdp.cc
File webrtc/api/webrtcsdp.cc (right):

https://codereview.webrtc.org/2074423002/diff/20001/webrtc/api/webrtcsdp.cc#n...
webrtc/api/webrtcsdp.cc:2190: int payload_type = *it;
On 2016/06/20 13:55:55, henrika_webrtc wrote:
> can't you do 
> 
> size_t payload_type = *it 
> 
> here instead

I tried, but line 2192 checks if it's >=0 so it seems it can be <0, which is why
I don't want to change the behavior of the code.

[kjellander_webrtc, 2016-06-20 14:02:30]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-06-20 14:02:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2074423002/20001

[commit-bot: I haz the power, 2016-06-20 14:04:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-20 14:04:28]

Description was changed from

==========
Fix some sign-compare warnings in webrtc/api.

The disabling of the warnings doesn't seem to work when Chromium
is using our targets (https://codereview.chromium.org/2022833002)
so better fix them.

BUG=webrtc:4256,webrtc:3307
NOTRY=True
==========

to

==========
Fix some sign-compare warnings in webrtc/api.

The disabling of the warnings doesn't seem to work when Chromium
is using our targets (https://codereview.chromium.org/2022833002)
so better fix them.

BUG=webrtc:4256,webrtc:3307
NOTRY=True

Committed: https://crrev.com/3e33bfeb6d4a7b9e14ce856e849f5e0367689e09
Cr-Commit-Position: refs/heads/master@{#13217}
==========

[commit-bot: I haz the power, 2016-06-20 14:04:29]

Patchset 2 (id:??) landed as
https://crrev.com/3e33bfeb6d4a7b9e14ce856e849f5e0367689e09
Cr-Commit-Position: refs/heads/master@{#13217}