don't mix system and bundled ICU

don't mix system and bundled ICU

The ICU 54.1 upgrade added a call to uloc_getDefault() to
base/test/test_suite.cc, but did not add a libicuuc dependency
to the corresponding gyp/gn targets.  This makes it possible to
mix system and bundled ICU headers (and will fail to build if
they differ in version).

BUG=428145
Committed: https://crrev.com/18f4375774ff56b72896a5b0b792b753ae4ef584
Cr-Commit-Position: refs/heads/master@{#317616}

[Mostyn Bramley-Moore, 2015-02-22 10:39:04]

mostynb@opera.com changed reviewers:
+ jshin@chromium.org, mark@chromium.org

[Mostyn Bramley-Moore, 2015-02-22 10:44:05]

@Mark, Jungshik: please take a look at this small fixup after
https://codereview.chromium.org/878723002.

[Mostyn Bramley-Moore, 2015-02-22 15:04:16]

I spotted another source file, whose target was missing the icuuc dep:
extensions/common/extension_l10n_util.cc

[Mark Mentovai, 2015-02-23 16:31:24]

LGTM

[Mostyn Bramley-Moore, 2015-02-23 17:08:10]

mostynb@opera.com changed reviewers:
+ rockot@chromium.org

[Mostyn Bramley-Moore, 2015-02-23 17:08:44]

@Ken: can you take a look at the extensions/extensions.gyp change?

[Ken Rockot(use gerrit already), 2015-02-23 17:26:11]

lgtm

[Mostyn Bramley-Moore, 2015-02-23 17:26:38]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2015-02-23 17:27:15]

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

[jungshik at Google, 2015-02-23 17:49:22]

LGTM. Thanks

[commit-bot: I haz the power, 2015-02-23 19:02:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-23 19:03:45]

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