Chromium Code Reviews| 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 c78aa8def8af41f30db46695348dfc8db65437ab..801e24be44dacc908e78e404f740d53b5acbddfc 100644 |
| --- a/chrome/browser/android/banners/app_banner_infobar_delegate_android.h |
| +++ b/chrome/browser/android/banners/app_banner_infobar_delegate_android.h |
| @@ -11,7 +11,6 @@ |
| #include "base/strings/string16.h" |
| #include "chrome/browser/banners/app_banner_manager.h" |
| #include "components/infobars/core/confirm_infobar_delegate.h" |
| -#include "content/public/common/manifest.h" |
| #include "ui/gfx/image/image.h" |
| #include "url/gurl.h" |
| @@ -24,6 +23,7 @@ class InfoBarManager; |
| } |
| class AppBannerInfoBar; |
| +struct ShortcutInfo; |
| namespace banners { |
| @@ -35,7 +35,7 @@ class AppBannerInfoBarDelegateAndroid : public ConfirmInfoBarDelegate { |
| base::WeakPtr<AppBannerManager> weak_manager, |
| const base::string16& app_title, |
| const GURL& manifest_url, |
| - const content::Manifest& manifest, |
| + std::unique_ptr<ShortcutInfo> info, |
| const GURL& icon_url, |
| std::unique_ptr<SkBitmap> icon, |
| int event_request_id, |
| @@ -67,6 +67,12 @@ class AppBannerInfoBarDelegateAndroid : public ConfirmInfoBarDelegate { |
| const base::android::JavaParamRef<jobject>& obj, |
| jboolean success); |
| + // Called when user clicks the "Add To Homescreen" from the app menu to |
| + // install a WebAPK. This function sets the text on infobar to "Adding" and |
| + // disables the button to avoid user interaction. It also starts the WebAPK's |
| + // installation process automatically. |
| + void InstallWebApk(content::WebContents* web_contents); |
| + |
| private: |
| void CreateJavaDelegate(); |
| bool AcceptNativeApp(content::WebContents* web_contents); |
| @@ -84,6 +90,7 @@ class AppBannerInfoBarDelegateAndroid : public ConfirmInfoBarDelegate { |
| int GetButtons() const override; |
| bool Accept() override; |
| bool LinkClicked(WindowOpenDisposition disposition) override; |
| + AppBannerInfoBarDelegateAndroid* AsAppBannerInfoBarDelegateAndroid() override; |
| base::android::ScopedJavaGlobalRef<jobject> java_delegate_; |
| @@ -92,7 +99,7 @@ class AppBannerInfoBarDelegateAndroid : public ConfirmInfoBarDelegate { |
| base::string16 app_title_; |
| GURL manifest_url_; |
|
dominickn
2016/09/06 01:49:23
Nit: manifest_url and icon_url aren't needed in th
Xi Han
2016/09/06 17:20:07
Done.
|
| - content::Manifest manifest_; |
| + std::unique_ptr<ShortcutInfo> shortcut_info_; |
| base::android::ScopedJavaGlobalRef<jobject> native_app_data_; |