Convert app banners to use Mojo.

chrome/browser/banners/app_banner_manager.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_BANNERS_APP_BANNER_MANAGER_H_
 #define CHROME_BROWSER_BANNERS_APP_BANNER_MANAGER_H_
 
 #include <memory>
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/engagement/site_engagement_observer.h"
 #include "chrome/browser/installable/installable_logging.h"
 #include "chrome/browser/installable/installable_manager.h"
 #include "content/public/browser/web_contents_observer.h"
-#include "third_party/WebKit/public/platform/modules/app_banner/WebAppBannerPromptReply.h"
+#include "mojo/public/cpp/bindings/binding.h"
+#include "third_party/WebKit/public/platform/modules/app_banner/app_banner.mojom.h"
 
 class SkBitmap;
 struct WebApplicationInfo;
 
 namespace content {
 class RenderFrameHost;
 class WebContents;
 }
 
 // This forward declaration exists solely for the DidFinishCreatingBookmarkApp
 // callback, implemented and called on desktop platforms only.
 namespace extensions {
 class Extension;
 }
 
 namespace banners {
 
 // Coordinates the creation of an app banner, from detecting eligibility to
 // fetching data and creating the infobar. Sites declare that they want an app
 // banner using the web app manifest. One web/native app may occupy the pipeline
 // at a time; navigation resets the manager and discards any work in progress.
 //
 // This class contains the generic functionality shared between all platforms,
 // as well as no-op callbacks that the platform-specific implementations pass to
 // base::Bind. This allows a WeakPtrFactory to be housed in this class.
 //
-// The InstallableManager fetches and validate a site's eligibility for banners.
-// The manager is first called to fetch the manifest, so we can verify whether
-// the site is already installed (and on Android, divert the flow to a
+// The InstallableManager fetches and validates whether a site is eligible for
+// banners. The manager is first called to fetch the manifest, so we can verify
+// whether the site is already installed (and on Android, divert the flow to a
 // native app banner if requested). The second call completes the checking for a
 // web app banner (checking manifest validity, service worker, and icon).
 class AppBannerManager : public content::WebContentsObserver,
+                         public blink::mojom::AppBannerService,
                          public SiteEngagementObserver {
  public:
+  static void Create(content::WebContents* web_contents);
+  static AppBannerManager* Get(content::WebContents* web_contents);
+
+  static void BindToMojoRequest(content::RenderFrameHost* render_frame_host,
+                                blink::mojom::AppBannerServiceRequest request);
+
   static void DisableSecureSchemeCheckForTesting();
 
   // Returns the current time.
   static base::Time GetCurrentTime();
 
   // Fast-forwards the current time for testing.
   static void SetTimeDeltaForTesting(int days);
 
   // Sets the weights applied to direct and indirect navigations for triggering
   // the banner. Deprecated and will be removed when app banners fully migrates
   // to using site engagement as a trigger.
   static void SetEngagementWeights(double direct_engagement,
                                    double indirect_engagement);
 
   // Returns whether or not the URLs match for everything except for the ref.
   static bool URLsAreForTheSamePage(const GURL& first, const GURL& second);
 
   // Requests an app banner. If |is_debug_mode| is true, any failure in the
   // pipeline will be reported to the devtools console.
   virtual void RequestAppBanner(const GURL& validated_url, bool is_debug_mode);
 
-  // Overridden and passed through base::Bind on desktop platforms. Called when
+  // Sends a message to the renderer that the user accepted the banner. Does
+  // nothing if |request_id| does not match the current request.
+  void SendBannerAccepted(int request_id);
+
+  // Sends a message to the renderer that the user dismissed the banner. Does
+  // nothing if |request_id| does not match the current request.
+  void SendBannerDismissed(int request_id);
+
+  // Overridden and passed through base::Bind on desktop platforms. Called
+  // when

[Sam McNally, 2016/10/06 07:17:43]

Reflow.

[dominickn, 2016/10/06 22:58:12]

Done.

   // the bookmark app install initiated by a banner has completed. Not used on
   // Android.
   virtual void DidFinishCreatingBookmarkApp(
       const extensions::Extension* extension,
-      const WebApplicationInfo& web_app_info) { }
+      const WebApplicationInfo& web_app_info) {}
 
   // Overridden and passed through base::Bind on Android. Called when the
   // download of a native app's icon is complete, as native banners use an icon
   // provided from the Play Store rather than the web manifest. Not used on
   // desktop platforms.
-  virtual void OnAppIconFetched(const SkBitmap& bitmap) { }
+  virtual void OnAppIconFetched(const SkBitmap& bitmap) {}
 
   // Overridden and passed through base::Bind on Android. Called after a web app
   // banner was successfully used to add a web app to homescreen to kick off an
   // asynchronous fetch of a splash screen icon. Not used on desktop platforms.
   virtual base::Closure FetchWebappSplashScreenImageCallback(
       const std::string& webapp_id);
 
  protected:
   explicit AppBannerManager(content::WebContents* web_contents);
   ~AppBannerManager() override;
@@ skipping 95 matching lines @@
   // The URL of the icon.
   GURL icon_url_;
 
   // The icon object.
   std::unique_ptr<SkBitmap> icon_;
 
   // The referrer string (if any) specified in the app URL. Used only for native
   // app banners.
   std::string referrer_;
 
+  // The app banner client which we are communicating with.
+  blink::mojom::AppBannerClientPtr banner_client_;
+
  private:
   friend class AppBannerManagerTest;
 
   // Record that the banner could be shown at this point, if the triggering
   // heuristic allowed.
   void RecordCouldShowBanner();
 
   // Creates a banner for the app. Overridden by subclasses as the infobar is
   // platform-specific.
   virtual void ShowBanner() = 0;
 
   // Returns true if the banner should be shown.
   bool CheckIfShouldShowBanner();
 
-  bool OnMessageReceived(const IPC::Message& message,
-                         content::RenderFrameHost* render_frame_host) override;
-
   // Called after the manager sends a message to the renderer regarding its
   // intention to show a prompt. The renderer will send a message back with the
   // opportunity to cancel.
-  void OnBannerPromptReply(content::RenderFrameHost* render_frame_host,
-                           int request_id,
-                           blink::WebAppBannerPromptReply reply,
-                           std::string referrer);
+  void OnBannerPromptReply(int request_id,
+                           blink::mojom::AppBannerPromptReply reply,
+                           mojo::String referrer);
 
-  // Called when the client has prevented a banner from being shown, and is
-  // now requesting that it be shown later.
-  void OnRequestShowAppBanner(content::RenderFrameHost* render_frame_host,
-                              int request_id);
+  // blink::mojom::AppBannerService overrides.
+  // Called when the client has prevented a banner from being shown, and is now
+  // requesting that it be shown later.
+  void DisplayAppBanner(int request_id) override;
+
+  // Sets the current beforeinstallprompt event.
+  void SetEvent(blink::mojom::AppBannerEventPtr event) override;
 
   // The type of navigation made to the page
   ui::PageTransition last_transition_type_;
 
   // Fetches the data required to display a banner for the current page.
   InstallableManager* manager_;
 
   // A monotonically increasing id to verify the response to the
   // beforeinstallprompt event from the renderer.
   int event_request_id_;
 
   // We do not want to trigger a banner when the manager is attached to
   // a WebContents that is playing video. Banners triggering on a site in the
   // background will appear when the tab is reactivated.
   std::vector<MediaPlayerId> active_media_players_;
 
+  std::unique_ptr<mojo::Binding<blink::mojom::AppBannerService>> binding_;
+
+  // A pointer to the current beforeinstallprompt event (if it exists).
+  std::unique_ptr<blink::mojom::AppBannerEventPtr> event_;
+
   // Whether we are currently working on whether to show a banner.
   bool is_active_;
 
   // If a banner is requested before the page has finished loading, defer
   // triggering the pipeline until the load is complete.
   bool banner_request_queued_;
   bool load_finished_;
 
   // Record whether the page decides to defer showing the banner, and if it
   // requests for it to be shown later on.
@@ skipping 10 matching lines @@
   // scoped to the WebContents which they are observing. This allows us to use
   // weak pointers for callbacks.
   base::WeakPtrFactory<AppBannerManager> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(AppBannerManager);
 };
 
 }  // namespace banners
 
 #endif  // CHROME_BROWSER_BANNERS_APP_BANNER_MANAGER_H_