Convert app banners to use Mojo.

chrome/renderer/chrome_render_frame_observer.h

 // Copyright 2013 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_RENDERER_CHROME_RENDER_FRAME_OBSERVER_H_
 #define CHROME_RENDERER_CHROME_RENDER_FRAME_OBSERVER_H_
 
+#include <memory>
+
 #include "base/macros.h"
 #include "base/timer/timer.h"
 #include "chrome/common/prerender_types.h"
 #include "content/public/renderer/render_frame_observer.h"
+#include "mojo/public/cpp/bindings/binding.h"
+#include "third_party/WebKit/public/platform/modules/app_banner/app_banner.mojom.h"
 
 class GURL;
 
 namespace gfx {
 class Size;
 }
 
 namespace safe_browsing {
 class PhishingClassifierDelegate;
 }
 
 namespace translate {
 class TranslateHelper;
 }
 
 // This class holds the Chrome specific parts of RenderFrame, and has the same
 // lifetime.
-class ChromeRenderFrameObserver : public content::RenderFrameObserver {
+class ChromeRenderFrameObserver : public content::RenderFrameObserver,
+                                  public blink::mojom::AppBannerClient {
  public:
+  static void BindBannerClient(

[dcheng, 2016/10/07 07:03:16]

Nit: any reason this needs to be static?

[dominickn, 2016/10/13 00:18:15]

Nope! Done.

+      ChromeRenderFrameObserver* observer,
+      mojo::InterfaceRequest<blink::mojom::AppBannerClient> request);
   explicit ChromeRenderFrameObserver(content::RenderFrame* render_frame);
   ~ChromeRenderFrameObserver() override;
 
  private:
   enum TextCaptureType { PRELIMINARY_CAPTURE, FINAL_CAPTURE };
 
   // RenderFrameObserver implementation.
   bool OnMessageReceived(const IPC::Message& message) override;
   void DidStartProvisionalLoad() override;
   void DidFinishLoad() override;
   void DidCommitProvisionalLoad(bool is_new_navigation,
                                 bool is_same_page_navigation) override;
   void DidMeaningfulLayout(blink::WebMeaningfulLayout layout_type) override;
   void OnDestruct() override;
 
   // IPC handlers
   void OnSetIsPrerendering(prerender::PrerenderMode mode);
   void OnRequestReloadImageForContextNode();
   void OnRequestThumbnailForContextNode(
       int thumbnail_min_area_pixels,
       const gfx::Size& thumbnail_max_size_pixels,
       int callback_id);
   void OnPrintNodeUnderContextMenu();
   void OnSetClientSidePhishingDetection(bool enable_phishing_detection);
-  void OnAppBannerPromptRequest(int request_id,
-                                const std::string& platform);
+
+  // blink::mojom::AppBannerClient implementation.
+  void BannerPromptRequest(
+      blink::mojom::AppBannerServicePtr banner_service,
+      blink::mojom::AppBannerEventRequest event_request,
+      const std::string& platform,
+      const BannerPromptRequestCallback& callback) override;
 
   // Captures page information using the top (main) frame of a frame tree.
   // Currently, this page information is just the text content of the all
   // frames, collected and concatenated until a certain limit (kMaxIndexChars)
   // is reached.
   // TODO(dglazkov): This is incompatible with OOPIF and needs to be updated.
   void CapturePageText(TextCaptureType capture_type);
 
   void CapturePageTextLater(TextCaptureType capture_type,
                             base::TimeDelta delay);
 
+  std::unique_ptr<mojo::Binding<blink::mojom::AppBannerClient>> banner_binding_;
+
   // Have the same lifetime as us.
   translate::TranslateHelper* translate_helper_;
   safe_browsing::PhishingClassifierDelegate* phishing_classifier_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeRenderFrameObserver);
 };
 
 #endif  // CHROME_RENDERER_CHROME_RENDER_FRAME_OBSERVER_H_