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 328 matching lines @@
 
   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.
   v8::Local<v8::Value> window;
   if (frame->IsWebLocalFrame()) {
-    window = frame->MainWorldScriptContext()->Global();
+    window = frame->ToWebLocalFrame()->MainWorldScriptContext()->Global();

[dcheng, 2017/06/06 19:57:17]

Perhaps we should provide a WebFrame getter to get the global proxy; wdyt?

[Łukasz Anforowicz, 2017/06/07 20:35:07]

Yes - thank you for noticing this cleanup opportunity.

As part of this work, I've removed the TODO(lazyboy,nasko) comments, but
honestly I am not sure if I understand those comments fully.  I hope the removal
of the comments was the right thing to do.

   } else {
     window = frame->ToWebRemoteFrame()->GlobalProxy();
   }
   args.GetReturnValue().Set(window);
 }
 
 void GuestViewInternalCustomBindings::GetViewFromID(
     const v8::FunctionCallbackInfo<v8::Value>& args) {
   // Default to returning null.
   args.GetReturnValue().SetNull();
@@ skipping 98 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