Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(514)

Unified Diff: chrome/browser/android/banners/app_banner_infobar_delegate_android.h

Issue 1310223002: webapps: initial addition of splash screen icon downloading (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@webapps-database-exp
Patch Set: Fix review comments and update code as discussed Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/banners/app_banner_infobar_delegate_android.h
diff --git a/chrome/browser/android/banners/app_banner_infobar_delegate_android.h b/chrome/browser/android/banners/app_banner_infobar_delegate_android.h
index 52c77c1166e29f50679ad5ce4dc132a2dfb40c56..030bdafe13031ca330cc599a636e75ab97a93dba 100644
--- a/chrome/browser/android/banners/app_banner_infobar_delegate_android.h
+++ b/chrome/browser/android/banners/app_banner_infobar_delegate_android.h
@@ -7,6 +7,7 @@
#include "base/android/scoped_java_ref.h"
#include "base/strings/string16.h"
+#include "chrome/browser/android/banners/app_banner_data_fetcher_android.h"
#include "components/infobars/core/confirm_infobar_delegate.h"
#include "content/public/common/manifest.h"
#include "ui/gfx/image/image.h"
@@ -30,6 +31,7 @@ class AppBannerInfoBarDelegateAndroid : public ConfirmInfoBarDelegate {
// Delegate for promoting a web app.
AppBannerInfoBarDelegateAndroid(
int event_request_id,
+ scoped_refptr<AppBannerDataFetcherAndroid> data_fetcher,
const base::string16& app_title,
SkBitmap* app_icon,
const content::Manifest& web_app_data);
@@ -74,10 +76,14 @@ class AppBannerInfoBarDelegateAndroid : public ConfirmInfoBarDelegate {
base::android::ScopedJavaGlobalRef<jobject> java_delegate_;
+ int event_request_id_;
+
+ // Used to fetch the splashscreen icon for webapps.
+ scoped_refptr<AppBannerDataFetcherAndroid> data_fetcher_;
+
base::string16 app_title_;
scoped_ptr<SkBitmap> app_icon_;
- int event_request_id_;
content::Manifest web_app_data_;
base::android::ScopedJavaGlobalRef<jobject> native_app_data_;

Powered by Google App Engine
This is Rietveld 408576698