Chromium Code Reviews| Index: chrome/browser/android/tab_android.cc |
| diff --git a/chrome/browser/android/tab_android.cc b/chrome/browser/android/tab_android.cc |
| index 4b1a8762b9090705d026b1fa8c153ba9d59ed6d3..e4e1d32b851d4e471b3f258a85a8a02c5611e58c 100644 |
| --- a/chrome/browser/android/tab_android.cc |
| +++ b/chrome/browser/android/tab_android.cc |
| @@ -12,6 +12,7 @@ |
| #include "chrome/browser/browser_about_handler.h" |
| #include "chrome/browser/chrome_notification_types.h" |
| #include "chrome/browser/content_settings/tab_specific_content_settings.h" |
| +#include "chrome/browser/favicon/favicon_tab_helper.h" |
| #include "chrome/browser/google/google_url_tracker_factory.h" |
| #include "chrome/browser/infobars/infobar_service.h" |
| #include "chrome/browser/prerender/prerender_contents.h" |
| @@ -46,7 +47,12 @@ |
| #include "content/public/browser/user_metrics.h" |
| #include "content/public/browser/web_contents.h" |
| #include "jni/Tab_jni.h" |
| +#include "skia/ext/image_operations.h" |
| #include "third_party/WebKit/public/platform/WebReferrerPolicy.h" |
| +#include "ui/base/resource/resource_bundle.h" |
| +#include "ui/gfx/android/java_bitmap.h" |
| +#include "ui/gfx/favicon_size.h" |
| +#include "ui/gfx/image/image_skia.h" |
| TabAndroid* TabAndroid::FromWebContents(content::WebContents* web_contents) { |
| CoreTabHelper* core_tab_helper = CoreTabHelper::FromWebContents(web_contents); |
| @@ -511,6 +517,36 @@ bool TabAndroid::Print(JNIEnv* env, jobject obj) { |
| return true; |
| } |
| +ScopedJavaLocalRef<jobject> TabAndroid::GetFavicon(JNIEnv* env, jobject obj) { |
| + ScopedJavaLocalRef<jobject> bitmap; |
| + FaviconTabHelper* favicon_tab_helper = |
| + FaviconTabHelper::FromWebContents(web_contents_.get()); |
| + if (!favicon_tab_helper) |
| + return bitmap; |
| + if (!favicon_tab_helper->FaviconIsValid()) |
| + return bitmap; |
| + |
| + SkBitmap favicon = |
| + favicon_tab_helper->GetFavicon() |
| + .AsImageSkia() |
| + .GetRepresentation( |
| + ResourceBundle::GetSharedInstance().GetMaxScaleFactor()) |
| + .sk_bitmap(); |
| + |
| + if (!favicon.isNull()) { |
|
David Trainor- moved to gerrit
2014/07/21 18:18:17
If this is null use the default gfx::Image::AsBitm
Jitu( very slow this week)
2014/07/24 14:08:21
Done.
|
| + if (favicon.width() != gfx::kFaviconSize || |
| + favicon.height() != gfx::kFaviconSize) |
|
David Trainor- moved to gerrit
2014/07/21 18:18:17
This scaling is still wrong it'll still scale to 1
Jitu( very slow this week)
2014/07/24 14:08:21
Done.
|
| + favicon = |
| + skia::ImageOperations::Resize(favicon, |
| + skia::ImageOperations::RESIZE_BEST, |
| + gfx::kFaviconSize, |
| + gfx::kFaviconSize); |
| + |
| + bitmap = gfx::ConvertToJavaBitmap(&favicon); |
| + } |
| + return bitmap; |
| +} |
| + |
| prerender::PrerenderManager* TabAndroid::GetPrerenderManager() const { |
| Profile* profile = GetProfile(); |
| if (!profile) |