Move registration of LocalFrame mojo interfaces to ModulesInitializer.

third_party/WebKit/Source/modules/ModulesInitializer.cpp

 // Copyright (c) 2014 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.
 
 #include "ModulesInitializer.h"
 
 #include "bindings/modules/v8/ModuleBindingsInitializer.h"
 #include "core/EventTypeNames.h"
 #include "core/css/CSSPaintImageGenerator.h"
 #include "core/dom/Document.h"
+#include "core/frame/LocalFrame.h"
 #include "core/html/HTMLCanvasElement.h"
 #include "core/offscreencanvas/OffscreenCanvas.h"
 #include "modules/EventModulesFactory.h"
 #include "modules/EventModulesNames.h"
 #include "modules/EventTargetModulesNames.h"
 #include "modules/IndexedDBNames.h"
 #include "modules/accessibility/AXObjectCacheImpl.h"
+#include "modules/app_banner/AppBannerController.h"
 #include "modules/canvas2d/CanvasRenderingContext2D.h"
 #include "modules/compositorworker/CompositorWorkerThread.h"
 #include "modules/csspaint/CSSPaintImageGeneratorImpl.h"
 #include "modules/filesystem/DraggedIsolatedFileSystemImpl.h"
 #include "modules/imagebitmap/ImageBitmapRenderingContext.h"
+#include "modules/installation/InstallationServiceImpl.h"
 #include "modules/offscreencanvas2d/OffscreenCanvasRenderingContext2D.h"
 #include "modules/time_zone_monitor/TimeZoneMonitorClient.h"
 #include "modules/webdatabase/DatabaseManager.h"
 #include "modules/webgl/WebGL2RenderingContext.h"
 #include "modules/webgl/WebGLRenderingContext.h"
 #include "platform/mojo/MojoHelper.h"
+#include "public/platform/InterfaceRegistry.h"
 #include "wtf/PtrUtil.h"
 
 namespace blink {
 
 void ModulesInitializer::initialize() {
   ASSERT(!isInitialized());
 
   // Strings must be initialized before calling CoreInitializer::init().
   const unsigned modulesStaticStringsCount =
       EventNames::EventModulesNamesCount +
@@ skipping 29 matching lines @@
       WTF::makeUnique<ImageBitmapRenderingContext::Factory>());
 
   // OffscreenCanvas context types must be registered with the OffscreenCanvas.
   OffscreenCanvas::registerRenderingContextFactory(
       WTF::makeUnique<OffscreenCanvasRenderingContext2D::Factory>());
   OffscreenCanvas::registerRenderingContextFactory(
       WTF::makeUnique<WebGLRenderingContext::Factory>());
   OffscreenCanvas::registerRenderingContextFactory(
       WTF::makeUnique<WebGL2RenderingContext::Factory>());
 
+  // Mojo Interfaces registered with LocalFrame
+  LocalFrame::registerInitializationCallback([](LocalFrame* frame) {
+    frame->interfaceRegistry()->addInterface(
+        WTF::bind(&InstallationServiceImpl::create, wrapWeakPersistent(frame)));
+  });
+
+  // TODO(dominickn): This interface should be document-scoped rather than
+  // frame-scoped, as the resulting banner event is dispatched to
+  // frame()->document().
+  LocalFrame::registerInitializationCallback([](LocalFrame* frame) {

[sashab, 2017/03/29 00:33:23]

I can't read this code. Is this lambdas?

I believe you if you say it works

[slangley, 2017/03/29 02:27:08]

lambdas - so hot right now

+    frame->interfaceRegistry()->addInterface(WTF::bind(
+        &AppBannerController::bindMojoRequest, wrapWeakPersistent(frame)));
+  });
   ASSERT(isInitialized());
 }
 
 }  // namespace blink