Fix compile warnings for 64-bit constants.

Fix compile warnings for 64-bit constants.


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=170601

[danakj, 2012-11-30 20:34:20]

lgtm

[Ryan Sleevi, 2012-11-30 20:35:15]

LGTM, mod macro switch below.

https://chromiumcodereview.appspot.com/11428117/diff/1/cc/hash_pair_unittest.cc
File cc/hash_pair_unittest.cc (right):

https://chromiumcodereview.appspot.com/11428117/diff/1/cc/hash_pair_unittest....
cc/hash_pair_unittest.cc:34: (GG_LONGLONG(1) << 60) + GG_LONGLONG(78931732321));
Even though they're equivalent, the last few reviews that I've seen this come
up, mark@ has chimed in that the macro used should be GG_INT64_C, rather than
assuming int64 == long long

So s/GG_LONGLONG/GG_INT64_C throughout.

[Ilya Sherman, 2012-11-30 21:06:14]

https://chromiumcodereview.appspot.com/11428117/diff/1/cc/hash_pair_unittest.cc
File cc/hash_pair_unittest.cc (right):

https://chromiumcodereview.appspot.com/11428117/diff/1/cc/hash_pair_unittest....
cc/hash_pair_unittest.cc:34: (GG_LONGLONG(1) << 60) + GG_LONGLONG(78931732321));
On 2012/11/30 20:35:15, Ryan Sleevi wrote:
> Even though they're equivalent, the last few reviews that I've seen this come
> up, mark@ has chimed in that the macro used should be GG_INT64_C, rather than
> assuming int64 == long long
> 
> So s/GG_LONGLONG/GG_INT64_C throughout.

Done.

[commit-bot: I haz the power, 2012-11-30 21:07:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11428117/5001

[commit-bot: I haz the power, 2012-11-30 21:28:47]

Retried try job too often on win_aura for step(s) aura_unittests,
content_browsertests, content_unittests, views_unittests

[commit-bot: I haz the power, 2012-11-30 21:30:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/11428117/5001

[commit-bot: I haz the power, 2012-11-30 23:49:43]

Change committed as 170601