Extend FaviconService to support fetching favicons from a Google server

chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java

Index: chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
index 3fbc9a5544fd3eb47ce1ed39e7a75821aa23c1d9..8a4cdef1b0da08934e90d4e81308d71d86da8faa 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/favicon/FaviconHelper.java
@@ -124,6 +124,8 @@ public class FaviconHelper {
      * @param isTemporary   Specifies whether the icon at iconUrl is temporary and should be updated
      *                      as soon as the page at pageUrl is revisited.
      * @param callback      Called when completed (download not needed, finished or failed).
+     * TODO(jkrcal): change this code to use
+     * FaviconService::Get1xFaviconForPageURLDownloadFromGoogleServerIfMissing.

[pkotwicz, 2016/12/20 02:33:01]

Super nit: I think that it makes more sense to have the comment in
chrome/browser/android/favicon_helper.cc It is slightly weird to refer to a C++
file from Java code

[jkrcal, 2017/02/06 19:01:22]

Done.

      */
     public void ensureIconIsAvailable(Profile profile, WebContents webContents, String pageUrl,
             String iconUrl, boolean isLargeIcon, boolean isTemporary,