Fix cronet build after JNI registration change.

Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936
Committed: https://crrev.com/d8b7d9253622fca97b6fe5cc36c56382b30156fa
Cr-Commit-Position: refs/heads/master@{#409867}

[Torne, 2016-08-04 16:04:10]

torne@chromium.org changed reviewers:
+ mikecase@chromium.org, mkwst@chromium.org, mleung380@gmail.com,
sievers@chromium.org

[Torne, 2016-08-04 16:04:10]

+mkwst for url OWNERS
+mef for net and cronet OWNERS

Sorry we broke this; is it possible to at least get a compile-only cronet bot on
the cq? Nothing covers use_platform_icu_alternatives right now :/

[Torne, 2016-08-04 16:04:28]

Description was changed from

==========
Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936
==========

to

==========
Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936
==========

[Torne, 2016-08-04 16:04:28]

torne@chromium.org changed reviewers:
+ mef@chromium.org
- mleung380@gmail.com

[Torne, 2016-08-04 16:04:41]

actually +mef, whoops

[mikecase (-- gone --), 2016-08-04 16:05:38]

On 2016/08/04 at 16:04:10, torne wrote:
> +mkwst for url OWNERS
> +mef for net and cronet OWNERS
> 
> Sorry we broke this; is it possible to at least get a compile-only cronet bot
on the cq? Nothing covers use_platform_icu_alternatives right now :/

I filed a bug for this,
https://bugs.chromium.org/p/chromium/issues/detail?id=634350

[mef, 2016-08-04 16:08:30]

lgtm

[no sievers, 2016-08-04 17:21:09]

lgtm, thanks!

[Mike West, 2016-08-04 17:28:00]

//url LGTM.

[mikecase (-- gone --), 2016-08-04 18:13:36]

The CQ bit was checked by mikecase@chromium.org

[mikecase (-- gone --), 2016-08-04 18:13:37]

lgtm

[commit-bot: I haz the power, 2016-08-04 18:14:22]

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-04 19:42:01]

Description was changed from

==========
Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936
==========

to

==========
Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936
==========

[commit-bot: I haz the power, 2016-08-04 19:42:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-04 19:44:11]

Description was changed from

==========
Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936
==========

to

==========
Fix cronet build after JNI registration change.

Cronet isn't covered by the CQ and broke when we removed
RegisterNativesImpl for classes that don't need it. Fix up by removing
the corresponding parts for use_platform_icu_alternatives=true which is
only used by cronet.

BUG=603936

Committed: https://crrev.com/d8b7d9253622fca97b6fe5cc36c56382b30156fa
Cr-Commit-Position: refs/heads/master@{#409867}
==========

[commit-bot: I haz the power, 2016-08-04 19:44:12]

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