Merge the Sky Engine changes from the SkyDart branch

sky/engine/core/html/HTMLIFrameElement.cpp

Index: sky/engine/core/html/HTMLIFrameElement.cpp
diff --git a/sky/engine/core/html/HTMLIFrameElement.cpp b/sky/engine/core/html/HTMLIFrameElement.cpp
index 42db220209674c27587ca76794cc8a1765f83d88..57eff5476d9c769c4a25edcf7a2c5273a33db62d 100644
--- a/sky/engine/core/html/HTMLIFrameElement.cpp
+++ b/sky/engine/core/html/HTMLIFrameElement.cpp
@@ -70,23 +70,24 @@ void HTMLIFrameElement::OnViewDestroyed(mojo::View* view)
     m_contentView = nullptr;
 }
 
-ScriptValue HTMLIFrameElement::takeServicesHandle(ScriptState* scriptState)
+PassRefPtr<DartValue> HTMLIFrameElement::takeServicesHandle(DartState*)
 {
-    return ScriptValue(scriptState, gin::ConvertToV8(scriptState->isolate(), m_services.PassMessagePipe().release()));
+    return DartValue::Create();
 }
 
-ScriptValue HTMLIFrameElement::takeExposedServicesHandle(ScriptState* scriptState)
+PassRefPtr<DartValue> HTMLIFrameElement::takeExposedServicesHandle(DartState*)
 {
-    return ScriptValue(scriptState, gin::ConvertToV8(scriptState->isolate(), m_exposedServices.release()));
+    return DartValue::Create();
 }
 
-void HTMLIFrameElement::embedViewManagerClient(const ScriptValue& client) {
+void HTMLIFrameElement::embedViewManagerClient(DartValue* 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)));
+  // TODO(dart)
+  // mojo::MessagePipeHandle handle;
+  // if (gin::ConvertFromV8(client.isolate(), client.v8Value(), &handle))
+  //   m_contentView->Embed(mojo::MakeProxy<mojo::ViewManagerClient>(mojo::MakeScopedHandle(handle)));
 }
 
 void HTMLIFrameElement::navigateView()