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

Unified Diff: third_party/WebKit/Source/bindings/core/v8/RemoteWindowProxy.cpp

Issue 2721693002: Revert of Switch RemoteWindowProxy to use v8::Context::NewRemoteContext. (Closed)
Patch Set: Created 3 years, 10 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/bindings/core/v8/RemoteWindowProxy.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/RemoteWindowProxy.cpp b/third_party/WebKit/Source/bindings/core/v8/RemoteWindowProxy.cpp
index fe75242a7614b221ae0fc82504773b0669efc3b8..fa804758ff6b7bf4bc9f06e97419829fff8cd994 100644
--- a/third_party/WebKit/Source/bindings/core/v8/RemoteWindowProxy.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/RemoteWindowProxy.cpp
@@ -28,15 +28,49 @@
* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
-#include "bindings/core/v8/RemoteWindowProxy.h"
+#include "bindings/core/v8/WindowProxy.h"
+#include <v8-debug.h>
+#include <v8.h>
+#include <algorithm>
+#include <utility>
+#include "bindings/core/v8/ConditionalFeatures.h"
#include "bindings/core/v8/DOMWrapperWorld.h"
-#include "bindings/core/v8/V8DOMWrapper.h"
+#include "bindings/core/v8/ScriptController.h"
+#include "bindings/core/v8/ToV8.h"
+#include "bindings/core/v8/V8Binding.h"
+#include "bindings/core/v8/V8DOMActivityLogger.h"
+#include "bindings/core/v8/V8Document.h"
+#include "bindings/core/v8/V8GCForContextDispose.h"
+#include "bindings/core/v8/V8HTMLCollection.h"
+#include "bindings/core/v8/V8HTMLDocument.h"
+#include "bindings/core/v8/V8HiddenValue.h"
+#include "bindings/core/v8/V8Initializer.h"
+#include "bindings/core/v8/V8ObjectConstructor.h"
+#include "bindings/core/v8/V8PagePopupControllerBinding.h"
+#include "bindings/core/v8/V8PrivateProperty.h"
#include "bindings/core/v8/V8Window.h"
+#include "core/frame/LocalFrame.h"
+#include "core/frame/LocalFrameClient.h"
+#include "core/frame/csp/ContentSecurityPolicy.h"
+#include "core/html/DocumentNameCollection.h"
+#include "core/html/HTMLCollection.h"
+#include "core/html/HTMLIFrameElement.h"
+#include "core/inspector/InspectorInstrumentation.h"
+#include "core/inspector/MainThreadDebugger.h"
+#include "core/loader/DocumentLoader.h"
+#include "core/loader/FrameLoader.h"
+#include "core/origin_trials/OriginTrialContext.h"
#include "platform/Histogram.h"
+#include "platform/RuntimeEnabledFeatures.h"
+#include "platform/ScriptForbiddenScope.h"
+#include "platform/heap/Handle.h"
#include "platform/instrumentation/tracing/TraceEvent.h"
-#include "v8/include/v8.h"
+#include "platform/weborigin/SecurityOrigin.h"
+#include "public/platform/Platform.h"
#include "wtf/Assertions.h"
+#include "wtf/StringExtras.h"
+#include "wtf/text/CString.h"
namespace blink {
@@ -49,16 +83,7 @@
if (m_lifecycle != Lifecycle::ContextInitialized)
return;
- if (behavior == DetachGlobal && !m_globalProxy.isEmpty()) {
- m_globalProxy.get().SetWrapperClassId(0);
- V8DOMWrapper::clearNativeInfo(isolate(), m_globalProxy.newLocal(isolate()));
-#if DCHECK_IS_ON()
- didDetachGlobalProxy();
-#endif
- }
-
- DCHECK_EQ(Lifecycle::ContextInitialized, m_lifecycle);
- m_lifecycle = Lifecycle::ContextDetached;
+ WindowProxy::disposeContext(behavior);
}
void RemoteWindowProxy::initialize() {
@@ -68,11 +93,23 @@
frame()->isMainFrame() ? "Blink.Binding.InitializeMainWindowProxy"
: "Blink.Binding.InitializeNonMainWindowProxy");
+ ScriptForbiddenScope::AllowUserAgentScript allowScript;
+
v8::HandleScope handleScope(isolate());
createContext();
+ ScriptState::Scope scope(m_scriptState.get());
+ v8::Local<v8::Context> context = m_scriptState->context();
+ if (m_globalProxy.isEmpty()) {
+ m_globalProxy.set(isolate(), context->Global());
+ CHECK(!m_globalProxy.isEmpty());
+ }
+
setupWindowPrototypeChain();
+
+ // Remote frames always require a full canAccess() check.
+ context->UseDefaultSecurityToken();
}
void RemoteWindowProxy::createContext() {
@@ -85,41 +122,22 @@
V8Window::domTemplate(isolate(), *m_world)->InstanceTemplate();
CHECK(!globalTemplate.IsEmpty());
- v8::Local<v8::Object> globalProxy =
- v8::Context::NewRemoteContext(isolate(), globalTemplate,
- m_globalProxy.newLocal(isolate()))
- .ToLocalChecked();
- if (m_globalProxy.isEmpty())
- m_globalProxy.set(isolate(), globalProxy);
- else
- DCHECK(m_globalProxy.get() == globalProxy);
- CHECK(!m_globalProxy.isEmpty());
+ v8::Local<v8::Context> context;
+ {
+ V8PerIsolateData::UseCounterDisabledScope useCounterDisabled(
+ V8PerIsolateData::from(isolate()));
+ context = v8::Context::New(isolate(), nullptr, globalTemplate,
+ m_globalProxy.newLocal(isolate()));
+ }
+ CHECK(!context.IsEmpty());
+
+ m_scriptState = ScriptState::create(context, m_world);
// TODO(haraken): Currently we cannot enable the following DCHECK because
// an already detached window proxy can be re-initialized. This is wrong.
// DCHECK(m_lifecycle == Lifecycle::ContextUninitialized);
m_lifecycle = Lifecycle::ContextInitialized;
-}
-
-void RemoteWindowProxy::setupWindowPrototypeChain() {
- DOMWindow* window = frame()->domWindow();
- const WrapperTypeInfo* wrapperTypeInfo = window->wrapperTypeInfo();
- // The global proxy object. Note this is not the global object.
- v8::Local<v8::Object> globalProxy = m_globalProxy.newLocal(isolate());
- V8DOMWrapper::setNativeInfo(isolate(), globalProxy, wrapperTypeInfo, window);
- // Mark the handle to be traced by Oilpan, since the global proxy has a
- // reference to the DOMWindow.
- m_globalProxy.get().SetWrapperClassId(wrapperTypeInfo->wrapperClassId);
-
-#if DCHECK_IS_ON()
- didAttachGlobalProxy();
-#endif
-
- // The global object, aka window wrapper object.
- v8::Local<v8::Object> windowWrapper =
- globalProxy->GetPrototype().As<v8::Object>();
- V8DOMWrapper::setNativeInfo(isolate(), windowWrapper, wrapperTypeInfo,
- window);
+ DCHECK(m_scriptState->contextIsValid());
}
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698