Convert app banners to use Mojo.

third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Index: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
index 2c3891b0241afdff6d33ae526e37e046c29b42e1..b0407a79948383336fb34968d49d446523e7536c 100644
--- a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
@@ -177,6 +177,7 @@
 #include "platform/weborigin/SchemeRegistry.h"
 #include "platform/weborigin/SecurityPolicy.h"
 #include "public/platform/InterfaceProvider.h"
+#include "public/platform/InterfaceRegistry.h"
 #include "public/platform/WebDoubleSize.h"
 #include "public/platform/WebFloatPoint.h"
 #include "public/platform/WebFloatRect.h"
@@ -228,6 +229,7 @@
 #include "wtf/PtrUtil.h"
 #include <algorithm>
 #include <memory>
+#include <utility>
 
 namespace blink {
 
@@ -1614,11 +1616,16 @@ void WebLocalFrameImpl::initializeCoreFrame(FrameHost* host,
   // during init(). Note that this may dispatch JS events; the frame may be
   // detached after init() returns.
   frame()->init();
-  if (frame() &&
-      frame()->loader().stateMachine()->isDisplayingInitialEmptyDocument() &&
-      !parent() && !opener() &&
-      frame()->settings()->shouldReuseGlobalForUnownedMainFrame())
-    frame()->document()->getSecurityOrigin()->grantUniversalAccess();
+  if (frame()) {
+    if (frame()->loader().stateMachine()->isDisplayingInitialEmptyDocument() &&
+        !parent() && !opener() &&
+        frame()->settings()->shouldReuseGlobalForUnownedMainFrame()) {
+      frame()->document()->getSecurityOrigin()->grantUniversalAccess();
+    }
+
+    frame()->interfaceRegistry()->addInterface(WTF::bind(

[dcheng, 2016/10/27 00:44:09]

Please add a TODO here that this interface should really be Document-scoped.

[dominickn, 2016/10/27 02:08:43]

Done.

+        &AppBannerController::bindMojoRequest, wrapWeakPersistent(frame())));
+  }
 }
 
 LocalFrame* WebLocalFrameImpl::createChildFrame(
@@ -2089,17 +2096,6 @@ void WebLocalFrameImpl::sendOrientationChangeEvent() {
     frame()->localDOMWindow()->sendOrientationChangeEvent();
 }
 
-void WebLocalFrameImpl::willShowInstallBannerPrompt(
-    int requestId,
-    const WebVector<WebString>& platforms,
-    WebAppBannerPromptReply* reply) {
-  if (!frame())
-    return;
-
-  AppBannerController::willShowInstallBannerPrompt(
-      requestId, client()->appBannerClient(), frame(), platforms, reply);
-}
-
 void WebLocalFrameImpl::requestRunTask(WebSuspendableTask* task) const {
   DCHECK(frame());
   DCHECK(frame()->document());