Add blink::ServiceRegistry and expose it from LocalFrame and Platform.

third_party/WebKit/public/web/WebLocalFrame.h

 // Copyright 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.
 
 #ifndef WebLocalFrame_h
 #define WebLocalFrame_h
 
 #include "WebFrame.h"
 #include "WebFrameLoadType.h"
 
 namespace blink {
 
 enum class WebAppBannerPromptReply;
 enum class WebSandboxFlags;
 enum class WebTreeScopeType;
+class ServiceRegistry;
 class WebAutofillClient;
 class WebContentSettingsClient;
 class WebDevToolsAgent;
 class WebDevToolsAgentClient;
 class WebFrameClient;
 class WebNode;
 class WebScriptExecutionCallback;
 class WebSuspendableTask;
 class WebTestInterfaceFactory;
 struct WebPrintPresetOptions;
 
 // Interface for interacting with in process frames. This contains methods that
 // require interacting with a frame's document.
 // FIXME: Move lots of methods from WebFrame in here.
 class WebLocalFrame : public WebFrame {
 public:
     // Creates a WebFrame. Delete this WebFrame by calling WebFrame::close().
     // It is valid to pass a null client pointer.
-    BLINK_EXPORT static WebLocalFrame* create(WebTreeScopeType, WebFrameClient*, WebFrame* opener = nullptr);
+    BLINK_EXPORT static WebLocalFrame* create(WebTreeScopeType, WebFrameClient*, WebFrame* opener = nullptr, ServiceRegistry* = nullptr);

[dcheng, 2016/03/29 04:51:56]

Hmm. Would it make more sense to have a WebFrameClient::serviceRegistry() method
instead? Then you don't need to plumb this pointer everywhere in
WebLocalFrameImpl and WebRemoteFrameImpl.

[esprehn, 2016/03/29 05:26:20]

In general I want WebFrameClient to die, it's a god class that knows about
everything everywhere. Instead we should plumb the right dependencies to the
right layers. That said, this might make more sense there for now.

[Sam McNally, 2016/03/30 00:20:53]

Done.

 
     // Used to create a provisional local frame in prepration for replacing a
     // remote frame if the load commits. The returned frame is only partially
     // attached to the frame tree: it has the same parent as its potential
     // replacee but is invisible to the rest of the frames in the frame tree.
     // If the load commits, call swap() to fully attach this frame.
-    BLINK_EXPORT static WebLocalFrame* createProvisional(WebFrameClient*, WebRemoteFrame*, WebSandboxFlags, const WebFrameOwnerProperties&);
+    BLINK_EXPORT static WebLocalFrame* createProvisional(WebFrameClient*, WebRemoteFrame*, WebSandboxFlags, const WebFrameOwnerProperties&, ServiceRegistry* = nullptr);
 
     // Returns the WebFrame associated with the current V8 context. This
     // function can return 0 if the context is associated with a Document that
     // is not currently being displayed in a Frame.
     BLINK_EXPORT static WebLocalFrame* frameForCurrentContext();
 
     // Returns the frame corresponding to the given context. This can return 0
     // if the context is detached from the frame, or if the context doesn't
     // correspond to a frame (e.g., workers).
     BLINK_EXPORT static WebLocalFrame* frameForContext(v8::Local<v8::Context>);
@@ skipping 279 matching lines @@
     // to call these on a WebLocalFrame.
     bool isWebLocalFrame() const override = 0;
     WebLocalFrame* toWebLocalFrame() override = 0;
     bool isWebRemoteFrame() const override = 0;
     WebRemoteFrame* toWebRemoteFrame() override = 0;
 };
 
 } // namespace blink
 
 #endif // WebLocalFrame_h