android: move ResourcePrefetchPredictor usage to LoadingPredictor.

android: move ResourcePrefetchPredictor usage to LoadingPredictor.

Following the native side refactoring (crrev.com/2847183002), update the
android code entry points to LoadingPredictor.

BUG=715525
Review-Url: https://codereview.chromium.org/2879103003
Cr-Commit-Position: refs/heads/master@{#471776}
Committed: https://chromium.googlesource.com/chromium/src/+/cfe486d5448e3b37f38c7f854c0f284cff8850fa

[Benoit L, 2017-05-15 12:09:03]

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

[commit-bot: I haz the power, 2017-05-15 12:09:17]

Dry run: CQ is trying da patch.

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

[Benoit L, 2017-05-15 12:28:54]

lizeb@chromium.org changed reviewers:
+ alexilin@chromium.org

[alexilin, 2017-05-15 13:05:51]

lgtm, thanks!

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/customtabs/LoadingPredictor.java
(right):

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/customtabs/LoadingPredictor.java:41:
* @param url The URL to start the prefetch for.
nit: 
Update url parameter description?

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java_sources...
File chrome/android/java_sources.gni (right):

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java_sources...
chrome/android/java_sources.gni:298:
"java/src/org/chromium/chrome/browser/customtabs/LoadingPredictor.java",
nit:
Shouldn't it be sorted alphabetically?

[Benoit L, 2017-05-15 13:33:23]

Thanks!

+bauerb: for chrome_jni_registrar.cc, only a rename and a bit of sorting. PTAL.

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/customtabs/LoadingPredictor.java
(right):

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/customtabs/LoadingPredictor.java:41:
* @param url The URL to start the prefetch for.
On 2017/05/15 13:05:50, alexilin wrote:
> nit: 
> Update url parameter description?

Done.

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java_sources...
File chrome/android/java_sources.gni (right):

https://codereview.chromium.org/2879103003/diff/1/chrome/android/java_sources...
chrome/android/java_sources.gni:298:
"java/src/org/chromium/chrome/browser/customtabs/LoadingPredictor.java",
On 2017/05/15 13:05:50, alexilin wrote:
> nit:
> Shouldn't it be sorted alphabetically?

Done.

[Benoit L, 2017-05-15 13:33:29]

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

[commit-bot: I haz the power, 2017-05-15 13:34:13]

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, 2017-05-15 14:55:30]

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

[commit-bot: I haz the power, 2017-05-15 14:55:32]

Dry run: This issue passed the CQ dry run.

[Benoit L, 2017-05-15 15:17:52]

lizeb@chromium.org changed reviewers:
+ bauerb@chromium.org

[Benoit L, 2017-05-15 15:17:52]

Actually +bauerb!

[Bernhard Bauer, 2017-05-15 15:29:34]

I like sorting! :) LGTM.

[Benoit L, 2017-05-15 15:40:08]

The CQ bit was checked by lizeb@chromium.org

[Benoit L, 2017-05-15 15:40:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
alexilin@chromium.org
Link to the patchset: https://codereview.chromium.org/2879103003/#ps40001
(title: ".")

[commit-bot: I haz the power, 2017-05-15 15:40:41]

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, 2017-05-15 15:53:37]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494862808214050,
"parent_rev": "fbaa170f914ca9eb0c75f9bd9d589d0a1d6f548c", "commit_rev":
"cfe486d5448e3b37f38c7f854c0f284cff8850fa"}

[commit-bot: I haz the power, 2017-05-15 15:53:56]

Description was changed from

==========
android: move ResourcePrefetchPredictor usage to LoadingPredictor.

Following the native side refactoring (crrev.com/2847183002), update the
android code entry points to LoadingPredictor.

BUG=715525
==========

to

==========
android: move ResourcePrefetchPredictor usage to LoadingPredictor.

Following the native side refactoring (crrev.com/2847183002), update the
android code entry points to LoadingPredictor.

BUG=715525

Review-Url: https://codereview.chromium.org/2879103003
Cr-Commit-Position: refs/heads/master@{#471776}
Committed:
https://chromium.googlesource.com/chromium/src/+/cfe486d5448e3b37f38c7f854c0f...
==========

[commit-bot: I haz the power, 2017-05-15 15:53:59]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/cfe486d5448e3b37f38c7f854c0f...