Fix MSVC warnings about possible value truncation.

Fix MSVC warnings about possible value truncation.

BUG=81439
TEST=none

[Peter Kasting, 2014-08-28 01:02:38]

pkasting@chromium.org changed reviewers:
+ yusukes@chromium.org

[Peter Kasting, 2014-08-28 01:02:38]

These warnings are not on by default yet, but I'm trying to enable them.

I erred on the side of manually checking for overflow when it wasn't obvious the
code would prevent it.  I'm happy to remove any checks that aren't actually
necessary.  Please do look closely at the changes, this is not a rubber-stamp
review.

[Peter Kasting, 2014-09-10 22:44:18]

Ping yusukes

[Peter Kasting, 2014-09-25 06:03:03]

On 2014/09/10 22:44:18, Peter Kasting wrote:
> Ping yusukes

Ping?

[Yusuke Sato, 2014-09-25 06:18:51]

LGTM

Sorry for the delay. I somehow missed this. Since I'm no longer working on
Chrome itself (my current focus is App Runtime for Chrome), please ask behdad@
for review next time. I think he is the current contact for OTS.

[Peter Kasting, 2014-09-25 08:23:18]

The CQ bit was checked by pkasting@chromium.org

[commit-bot: I haz the power, 2014-09-25 08:25:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-25 08:25:13]

Commit queue rejected this change because it did not recognize the base URL.
Please commit your change manually.

[Peter Kasting, 2014-09-25 21:15:48]

pkasting@chromium.org changed reviewers:
+ behdad@chromium.org

[Peter Kasting, 2014-09-25 21:15:48]

Behdad, can you land this for me, or tell me how to land it?  As you can see
above the CQ didn't work, and "git cl land" locally gives me "Git access
forbidden".

[behdad_google, 2014-09-25 21:37:57]

On 2014/09/25 21:15:48, Peter Kasting wrote:
> Behdad, can you land this for me, or tell me how to land it?  As you can see
> above the CQ didn't work, and "git cl land" locally gives me "Git access
> forbidden".

Oh, I didn't notice this at all.

Umm, we are in the (slow) process of switching to github.com/khaledhosny/ots. 
I'll ping Khaled to take this there.

I don't know how to land this; I never committed to OTS.  If I'm listed in
OWNERS that's probably because currently I care more about this piece than other
people.

behdad

[Kunihiko Sakamoto, 2014-09-26 03:32:33]

ksakamoto@chromium.org changed reviewers:
+ ksakamoto@chromium.org

[Kunihiko Sakamoto, 2014-09-26 03:32:34]

lgtm
Thanks. I will land this. (Currently I and bashi@ maintain googlecode OTS.)

[Kunihiko Sakamoto, 2014-09-26 03:56:02]

Committed manually as r116

https://code.google.com/p/ots/source/detail?r=116

[behdad_google, 2014-09-29 15:00:55]

On 2014/09/26 03:56:02, Kunihiko Sakamoto wrote:
> Committed manually as r116
> 
> https://code.google.com/p/ots/source/detail?r=116

Thanks.  Got integrated in the github repo as well.