Adds support for embedViewManagerClient() to IFrame.

sky/engine/core/html/HTMLIFrameElement.cpp

 // 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.
 
 #include "sky/engine/config.h"
 #include "sky/engine/core/html/HTMLIFrameElement.h"
 
 #include "gen/sky/core/HTMLNames.h"
 #include "mojo/edk/js/handle.h"
 #include "sky/engine/core/frame/LocalFrame.h"
@@ skipping 62 matching lines @@
 ScriptValue HTMLIFrameElement::takeServicesHandle(ScriptState* scriptState)
 {
     return ScriptValue(scriptState, gin::ConvertToV8(scriptState->isolate(), m_services.PassMessagePipe().release()));
 }
 
 ScriptValue HTMLIFrameElement::takeExposedServicesHandle(ScriptState* scriptState)
 {
     return ScriptValue(scriptState, gin::ConvertToV8(scriptState->isolate(), m_exposedServices.release()));
 }
 
+void HTMLIFrameElement::embedViewManagerClient(blink::ScriptValue client) {
+  if (!m_contentView)
+    return;
+
+  mojo::MessagePipeHandle handle;
+  if (gin::ConvertFromV8(client.isolate(), client.v8Value(), &handle))
+    m_contentView->Embed(mojo::MakeProxy<mojo::ViewManagerClient>(mojo::MakeScopedHandle(handle).Pass()));

[abarth-chromium, 2015/02/11 01:02:13]

Is the Pass() needed here?  It looks redundant.

[hansmuller1, 2015/02/11 01:09:00]

Done.

+}
+
 void HTMLIFrameElement::navigateView()
 {
     if (!m_contentView)
         return;
 
     String urlString = stripLeadingAndTrailingHTMLSpaces(getAttribute(HTMLNames::srcAttr));
     if (urlString.isEmpty())
         urlString = blankURL().string();
 
     KURL url = document().completeURL(urlString);
 
     mojo::MessagePipe exposedServicesPipe;
     m_exposedServices = exposedServicesPipe.handle0.Pass();
 
     m_contentView->Embed(mojo::String::From(url.string().utf8().data()),
         mojo::GetProxy(&m_services),
         mojo::MakeProxy<mojo::ServiceProvider>(exposedServicesPipe.handle1.Pass()));
 }
 
 }