Chromium Code Reviews

Unified Diff: Source/bindings/v8/V8Initializer.cpp

Issue 26868004: Set default resource constraints on V8 before the the default isolate is initialized (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix workers too. Created 7 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
« no previous file with comments | « no previous file | Source/bindings/v8/WorkerScriptController.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/v8/V8Initializer.cpp
diff --git a/Source/bindings/v8/V8Initializer.cpp b/Source/bindings/v8/V8Initializer.cpp
index 7e6e1cfecab97f8ec89fc3963d7b5de8d8ea41d0..1fc8ec310c8c8632c230b655810e89293b2b975a 100644
--- a/Source/bindings/v8/V8Initializer.cpp
+++ b/Source/bindings/v8/V8Initializer.cpp
@@ -51,7 +51,6 @@
#include "wtf/RefPtr.h"
#include "wtf/text/WTFString.h"
#include <v8-debug.h>
-#include <v8-defaults.h>
namespace WebCore {
@@ -174,7 +173,6 @@ void V8Initializer::initializeMainThreadIfNeeded(v8::Isolate* isolate)
v8::V8::AddMessageListener(messageHandlerInMainThread);
v8::V8::SetFailedAccessCheckCallbackFunction(failedAccessCheckCallbackInMainThread);
v8::V8::SetAllowCodeGenerationFromStringsCallback(codeGenerationCheckCallbackInMainThread);
- v8::SetDefaultResourceConstraintsForCurrentPlatform();
ScriptProfiler::initialize();
V8PerIsolateData::ensureInitialized(isolate);
}
@@ -218,7 +216,6 @@ void V8Initializer::initializeWorker(v8::Isolate* isolate)
v8::V8::SetFatalErrorHandler(reportFatalErrorInWorker);
v8::ResourceConstraints resourceConstraints;
- v8::ConfigureResourceConstraintsForCurrentPlatform(&resourceConstraints);
uint32_t here;
resourceConstraints.set_stack_limit(&here - kWorkerMaxStackSize / sizeof(uint32_t*));
v8::SetResourceConstraints(&resourceConstraints);
« no previous file with comments | « no previous file | Source/bindings/v8/WorkerScriptController.cpp » ('j') | no next file with comments »

Powered by Google App Engine