Fix build with system ICU

Fix build with system ICU

BUG=551343
Committed: https://crrev.com/860564e6051956a73b673d4adae2c4a0df0b175b
Cr-Commit-Position: refs/heads/master@{#415252}

[Paweł Hajdan Jr., 2016-08-29 09:45:34]

The CQ bit was checked by phajdan.jr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-29 09:45:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-29 10:51:49]

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

[commit-bot: I haz the power, 2016-08-29 10:51:50]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Paweł Hajdan Jr., 2016-08-29 11:26:13]

phajdan.jr@chromium.org changed reviewers:
+ timvolodine@chromium.org

[Paweł Hajdan Jr., 2016-08-29 15:46:08]

phajdan.jr@chromium.org changed reviewers:
+ groby@chromium.org, rouslan@chromium.org

[Paweł Hajdan Jr., 2016-08-29 15:46:09]

+groby,rouslan

[please use gerrit instead, 2016-08-29 17:18:32]

lgtm

[groby-ooo-7-16, 2016-08-29 17:37:13]

LGTM, nit optional

https://codereview.chromium.org/2282403003/diff/1/components/spellcheck/rende...
File components/spellcheck/renderer/spellcheck_worditerator.cc (right):

https://codereview.chromium.org/2282403003/diff/1/components/spellcheck/rende...
components/spellcheck/renderer/spellcheck_worditerator.cc:333:
std::unique_ptr<base::i18n::BreakIterator> iterator(
Tiny nit: While you're here, would you mind switching to MakeUnique<>?

[Paweł Hajdan Jr., 2016-08-30 08:29:17]

https://codereview.chromium.org/2282403003/diff/1/components/spellcheck/rende...
File components/spellcheck/renderer/spellcheck_worditerator.cc (right):

https://codereview.chromium.org/2282403003/diff/1/components/spellcheck/rende...
components/spellcheck/renderer/spellcheck_worditerator.cc:333:
std::unique_ptr<base::i18n::BreakIterator> iterator(
On 2016/08/29 at 17:37:12, groby wrote:
> Tiny nit: While you're here, would you mind switching to MakeUnique<>?

Done.

[Paweł Hajdan Jr., 2016-08-30 08:29:21]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-08-30 08:29:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 09:13:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 09:15:02]

Description was changed from

==========
Fix build with system ICU

BUG=551343
==========

to

==========
Fix build with system ICU

BUG=551343

Committed: https://crrev.com/860564e6051956a73b673d4adae2c4a0df0b175b
Cr-Commit-Position: refs/heads/master@{#415252}
==========

[commit-bot: I haz the power, 2016-08-30 09:15:04]

Patchset 1 (id:??) landed as
https://crrev.com/860564e6051956a73b673d4adae2c4a0df0b175b
Cr-Commit-Position: refs/heads/master@{#415252}