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 f175c0f92587ebb466db9a01ec39c4323ecce6c8..b7f2e89f2f0307e1cdd02d249fd4a2c154f4765d 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,9 +47,16 @@ |
| #include "content/public/browser/notification_service.h" |
| #include "content/public/browser/user_metrics.h" |
| #include "content/public/browser/web_contents.h" |
| +#include "grit/ui_resources.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/base/window_open_disposition.h" |
| +#include "ui/gfx/android/device_display_info.h" |
| +#include "ui/gfx/android/java_bitmap.h" |
| +#include "ui/gfx/favicon_size.h" |
| +#include "ui/gfx/image/image_skia.h" |
| using content::GlobalRequestID; |
| using content::NavigationController; |
| @@ -572,6 +580,48 @@ 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.empty()) { |
|
Jitu( very slow this week)
2014/08/01 11:35:50
If favicon is empty get the default favicon.
David Trainor- moved to gerrit
2014/08/08 20:48:57
Sorry for the communication issues. I meant, "Gra
|
| + gfx::Image image = |
| + gfx::Image(*ResourceBundle::GetSharedInstance().GetImageSkiaNamed( |
| + IDR_DEFAULT_FAVICON)); |
| + favicon = image.AsBitmap(); |
| + } |
| + |
| + if (!favicon.empty()) { |
|
Jitu( very slow this week)
2014/08/01 11:35:50
If both default and favicon is not empty then scal
|
| + gfx::DeviceDisplayInfo device_info; |
| + const float device_scale_factor = device_info.GetDIPScale(); |
| + int target_size_dip = device_scale_factor * gfx::kFaviconSize; |
| + if (favicon.width() != target_size_dip || |
| + favicon.height() != target_size_dip) { |
| + favicon = |
| + skia::ImageOperations::Resize(favicon, |
| + skia::ImageOperations::RESIZE_BEST, |
| + target_size_dip, |
| + target_size_dip); |
| + } |
| + |
| + bitmap = gfx::ConvertToJavaBitmap(&favicon); |
| + } |
| + return bitmap; |
| +} |
| + |
| prerender::PrerenderManager* TabAndroid::GetPrerenderManager() const { |
| Profile* profile = GetProfile(); |
| if (!profile) |