Move MainWorldScriptContext accessor/method from WebFrame to WebLocalFrame.

extensions/renderer/guest_view/guest_view_internal_custom_bindings.cc

 // 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 "extensions/renderer/guest_view/guest_view_internal_custom_bindings.h"
 
 #include <string>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 324 matching lines @@
 
   int view_id = args[0]->Int32Value();
   if (view_id == MSG_ROUTING_NONE)
     return;
 
   content::RenderView* view = content::RenderView::FromRoutingID(view_id);
   if (!view)
     return;
 
   blink::WebFrame* frame = view->GetWebView()->MainFrame();
-  // TODO(lazyboy,nasko): The WebLocalFrame branch is not used when running
-  // on top of out-of-process iframes. Remove it once the code is converted.

[Devlin, 2017/06/09 18:56:40]

Is isolate extensions sufficient for this, or does this need to wait for full
isolation?

-  v8::Local<v8::Value> window;
-  if (frame->IsWebLocalFrame()) {
-    window = frame->MainWorldScriptContext()->Global();
-  } else {
-    window = frame->ToWebRemoteFrame()->GlobalProxy();
-  }
+  v8::Local<v8::Value> window = frame->GlobalProxy();
   args.GetReturnValue().Set(window);
 }
 
 void GuestViewInternalCustomBindings::GetViewFromID(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
   // Default to returning null.
   args.GetReturnValue().SetNull();
   // There is one argument.
   CHECK(args.Length() == 1);
   // The view ID.
@@ skipping 95 matching lines @@
   // EnterFullscreen() and do it directly rather than having a generic "run with
   // user gesture" function.
   blink::WebScopedUserGesture user_gesture(context()->web_frame());
   CHECK_EQ(args.Length(), 1);
   CHECK(args[0]->IsFunction());
   context()->SafeCallFunction(
       v8::Local<v8::Function>::Cast(args[0]), 0, nullptr);
 }
 
 }  // namespace extensions