Adds Mojo IDL.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index ab4fd84d9ac2460bfd6fbccd0834096b4eed778c..e6b88a077366817c428bb46ac97776b03506b7bb 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -184,6 +184,7 @@
 #include "third_party/WebKit/public/web/WebKit.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
 #include "third_party/WebKit/public/web/WebMediaStreamRegistry.h"
+#include "third_party/WebKit/public/web/WebMojoBindings.h"
 #include "third_party/WebKit/public/web/WebNavigationPolicy.h"
 #include "third_party/WebKit/public/web/WebPlugin.h"
 #include "third_party/WebKit/public/web/WebPluginContainer.h"
@@ -2551,20 +2552,22 @@ void RenderFrameImpl::EnsureMojoBuiltinsAreAvailable(
     return;
 
   v8::HandleScope handle_scope(isolate);
-  registry->AddBuiltinModule(isolate, mojo::edk::js::Core::kModuleName,
-                             mojo::edk::js::Core::GetModule(isolate));
+  registry->AddBuiltinModule(

[Ken Rockot(use gerrit already), 2016/10/10 17:11:25]

I don't think this code should change yet. We should continue to expose the old
mojo/public/js/core module as it exists today, until the entire core API is
ported. I also don't think you want to use |registry| to expose your new
bindings object.

I'd rather we just stash the new core in a global object like window.mojoCore or
something similar (preferably just 'mojo' but that's already taken in layout
tests at the moment - can fix once the rest of the API is ported.)

Then JS contexts which are allowed to see mojo, like layout tests and your cast
client code, don't need to do any requires/defines/whatever to get the core API.
They can call mojoCore.createMessagePipe etc immediately.

[alokp, 2016/10/18 00:11:47]

Agreed. I have reverted the changes in this file for now.

+      isolate, mojo::edk::js::Core::kModuleName,
+      blink::WebMojoBindings::createCore(isolate, context));
   registry->AddBuiltinModule(isolate, mojo::edk::js::Support::kModuleName,
                              mojo::edk::js::Support::GetModule(isolate));
   registry->AddBuiltinModule(
       isolate, InterfaceProviderJsWrapper::kPerFrameModuleName,
-      InterfaceProviderJsWrapper::Create(
-          isolate, context, remote_interfaces_.get())
-          .ToV8());
+      blink::WebMojoBindings::createInterfaceProvider(
+          isolate, context, base::MakeUnique<BlinkInterfaceProviderImpl>(
+                                remote_interfaces_->GetWeakPtr())));
   registry->AddBuiltinModule(
       isolate, InterfaceProviderJsWrapper::kPerProcessModuleName,
-      InterfaceProviderJsWrapper::Create(
-          isolate, context, RenderThread::Get()->GetRemoteInterfaces())
-          .ToV8());
+      blink::WebMojoBindings::createInterfaceProvider(
+          isolate, context,
+          base::MakeUnique<BlinkInterfaceProviderImpl>(
+              RenderThread::Get()->GetRemoteInterfaces()->GetWeakPtr())));
 }
 
 void RenderFrameImpl::AddMessageToConsole(ConsoleMessageLevel level,