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

Unified Diff: chrome/browser/banners/app_banner_infobar_delegate_desktop.h

Issue 2156113002: Replace AppBannerDataFetcher with InstallableManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@banner-refactor
Patch Set: Naming, includes Created 4 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/banners/app_banner_infobar_delegate_desktop.h
diff --git a/chrome/browser/banners/app_banner_infobar_delegate_desktop.h b/chrome/browser/banners/app_banner_infobar_delegate_desktop.h
index bf4fc21ea4bd4ea89621c683632b5cc87614023c..3891d3daf461a0993b7dcc0841e5dfd6180bf694 100644
--- a/chrome/browser/banners/app_banner_infobar_delegate_desktop.h
+++ b/chrome/browser/banners/app_banner_infobar_delegate_desktop.h
@@ -5,29 +5,28 @@
#ifndef CHROME_BROWSER_BANNERS_APP_BANNER_INFOBAR_DELEGATE_DESKTOP_H_
#define CHROME_BROWSER_BANNERS_APP_BANNER_INFOBAR_DELEGATE_DESKTOP_H_
-#include <memory>
-
#include "base/macros.h"
+#include "base/memory/weak_ptr.h"
#include "base/strings/string16.h"
#include "components/infobars/core/confirm_infobar_delegate.h"
#include "content/public/common/manifest.h"
namespace content {
class WebContents;
-} // namespace content
+}
namespace extensions {
class BookmarkAppHelper;
class Extension;
-} // namespace extensions
+}
namespace infobars {
class InfoBar;
-} // namespace infobars
+}
namespace banners {
-class AppBannerDataFetcherDesktop;
+class AppBannerManager;
class AppBannerInfoBarDelegateDesktop : public ConfirmInfoBarDelegate {
@@ -35,10 +34,10 @@ class AppBannerInfoBarDelegateDesktop : public ConfirmInfoBarDelegate {
~AppBannerInfoBarDelegateDesktop() override;
static infobars::InfoBar* Create(
- scoped_refptr<AppBannerDataFetcherDesktop> fetcher,
content::WebContents* web_contents,
- const content::Manifest& web_manifest,
+ base::WeakPtr<AppBannerManager> weak_manager,
extensions::BookmarkAppHelper* bookmark_app_helper,
+ const content::Manifest& manifest,
int event_request_id);
// ConfirmInfoBarDelegate overrides.
@@ -54,15 +53,15 @@ class AppBannerInfoBarDelegateDesktop : public ConfirmInfoBarDelegate {
protected:
AppBannerInfoBarDelegateDesktop(
- scoped_refptr<AppBannerDataFetcherDesktop> fetcher,
- const content::Manifest& web_manifest,
+ base::WeakPtr<AppBannerManager> weak_manager,
extensions::BookmarkAppHelper* bookmark_app_helper,
+ const content::Manifest& manifest,
int event_request_id);
private:
- scoped_refptr<AppBannerDataFetcherDesktop> fetcher_;
- content::Manifest web_manifest_;
+ base::WeakPtr<AppBannerManager> weak_manager_;
extensions::BookmarkAppHelper* bookmark_app_helper_;
+ content::Manifest manifest_;
int event_request_id_;
bool has_user_interaction_;
« no previous file with comments | « chrome/browser/banners/app_banner_debug_log.cc ('k') | chrome/browser/banners/app_banner_infobar_delegate_desktop.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698