Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(178)

Unified Diff: third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp

Issue 2626183003: Switch RemoteWindowProxy to use v8::Context::NewRemoteContext. (Closed)
Patch Set: Finish partial comment Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp b/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
index ba6c90bd6896a599af749749d7e1e4d4fdf87c5f..4f0a869089e3f01f0d8dff46f945ac3467e63d35 100644
--- a/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp
@@ -4,6 +4,8 @@
#include "web/WebRemoteFrameImpl.h"
+#include "bindings/core/v8/DOMWrapperWorld.h"
+#include "bindings/core/v8/WindowProxy.h"
#include "core/dom/Fullscreen.h"
#include "core/dom/RemoteSecurityContext.h"
#include "core/dom/SecurityContext.h"
@@ -212,11 +214,6 @@ v8::Local<v8::Context> WebRemoteFrameImpl::mainWorldScriptContext() const {
return v8::Local<v8::Context>();
}
-v8::Local<v8::Context> WebRemoteFrameImpl::deprecatedMainWorldScriptContext()
- const {
- return toV8Context(frame(), DOMWrapperWorld::mainWorld());
-}
-
void WebRemoteFrameImpl::reload(WebFrameLoadType) {
NOTREACHED();
}
@@ -520,6 +517,12 @@ void WebRemoteFrameImpl::willEnterFullscreen() {
Fullscreen::RequestType::PrefixedForCrossProcessDescendant);
}
+v8::Local<v8::Object> WebRemoteFrameImpl::globalProxy() const {
+ return frame()
+ ->windowProxy(DOMWrapperWorld::mainWorld())
+ ->globalIfNotDetached();
+}
+
WebRemoteFrameImpl::WebRemoteFrameImpl(WebTreeScopeType scope,
WebRemoteFrameClient* client)
: WebRemoteFrame(scope),

Powered by Google App Engine
This is Rietveld 408576698