Make it compile with -Werror on gcc 4.6

Make it compile with -Werror on gcc 4.6

BUG=87490
TEST=None

[Nico, 2011-10-03 21:34:31]

lgtm

Thanks for the patch! Looks fine to me, but jshin owns all things ICU, so he
should have a look too.

[Nico, 2011-10-03 21:34:33]

lgtm

lgtm

Thanks for the patch! Looks fine to me, but jshin owns all things ICU, so he
should have a look too.

[Nico, 2011-10-03 21:34:34]

lgtm

lgtm

Thanks for the patch! Looks fine to me, but jshin owns all things ICU, so he
should have a look too.

[Nico, 2011-10-03 21:35:14]

(Sorry, I didn't mean to say "lgtm"; I hit the wrong button accidentally.
s/lgtm// in the messages above please :-) )

[ph34r, 2011-10-13 11:46:55]

Jungshik Shin, ping :)

[jungshik at Google, 2011-10-21 16:52:51]

On 2011/10/13 11:46:55, ph34r wrote:
> Jungshik Shin, ping :)

Sorry I missed this. Thanks for the patch
LGTM. 

I'm about to file a bug against the upstream and add a reference to that bug in
README.chromium before checking this in.

[jungshik at Google, 2011-10-21 17:16:12]

It turned out that the upstream does not have this code any more. So, it's
pointless to report bug against the upstream. I'll check 'commit' box.

[jungshik at Google, 2011-10-21 20:50:48]

I landed it (http://crrev.com/106776) . Thank you for the patch.

[tony, 2011-10-21 20:58:31]

On 2011/10/21 20:50:48, Jungshik Shin wrote:
> I landed it (http://crrev.com/106776) . Thank you for the patch.

Jungshik, I think you missed the following:
http://code.google.com/p/chromium/issues/detail?id=97482#c5
http://codereview.chromium.org/8333007/