Implementation of GetFavicon for current tab.

chrome/android/java/src/org/chromium/chrome/browser/Tab.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser;
 
 import android.app.Activity;
 import android.content.Context;
 import android.graphics.Bitmap;
 import android.graphics.Color;
@@ skipping 862 matching lines @@
     @CalledByNative
     public String getTitle() {
         if (mNativePage != null) return mNativePage.getTitle();
         if (mContentViewCore != null) return mContentViewCore.getTitle();
         return "";
     }
 
     /**
      * @return The bitmap of the favicon scaled to 16x16dp. null if no favicon
      *         is specified or it requires the default favicon.
      *         TODO(bauerb): Upstream implementation.

[Yaron, 2014/07/02 17:43:08]

At a minimum this needs to be updated. We should probably change to something
like "reconcile with the downstream" version.

[Jitu( very slow this week), 2014/07/03 12:21:17]

Done.

      */
     public Bitmap getFavicon() {
-        return null;
+        if (mNativeTabAndroid == 0)
+            return null;
+        return nativeGetFavicon(mNativeTabAndroid);
     }
 
     /**
      * Loads the tab if it's not loaded (e.g. because it was killed in background).
      * @return true iff tab load was triggered
      */
     @CalledByNative
     public boolean loadIfNeeded() {
         return false;
     }
@@ skipping 104 matching lines @@
      * @return The current {@link TabChromeWebContentsDelegateAndroid} instance.
      */
     protected TabChromeWebContentsDelegateAndroid getChromeWebContentsDelegateAndroid() {
         return mWebContentsDelegate;
     }
 
     /**
      * Called when the favicon of the content this tab represents changes.
      */
     @CalledByNative
     protected void onFaviconUpdated() {

[Yaron, 2014/07/02 17:43:08]

Does this get called with your change?

[Jitu( very slow this week), 2014/07/03 12:21:17]

Yes.

         for (TabObserver observer : mObservers) observer.onFaviconUpdated(this);
     }
 
     /**
      * Called when the navigation entry containing the historyitem changed,
      * for example because of a scroll offset or form field change.
      */
     @CalledByNative
     protected void onNavEntryChanged() {
     }
@@ skipping 122 matching lines @@
     private native void nativeDestroyWebContents(long nativeTabAndroid, boolean deleteNative);
     private native WebContents nativeGetWebContents(long nativeTabAndroid);
     private native Profile nativeGetProfileAndroid(long nativeTabAndroid);
     private native int nativeLoadUrl(long nativeTabAndroid, String url, String extraHeaders,
             byte[] postData, int transition, String referrerUrl, int referrerPolicy,
             boolean isRendererInitiated);
     private native int nativeGetSecurityLevel(long nativeTabAndroid);
     private native void nativeSetActiveNavigationEntryTitleForUrl(long nativeTabAndroid, String url,
             String title);
     private native boolean nativePrint(long nativeTabAndroid);
+    private native Bitmap nativeGetFavicon(long nativeTabAndroid);
 }