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

Unified Diff: gin/isolate_holder.cc

Issue 2841443005: [Bindings] Create and use V8 context snapshots (Closed)
Patch Set: Rebase Created 3 years, 5 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
« no previous file with comments | « extensions/shell/BUILD.gn ('k') | gin/public/isolate_holder.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gin/isolate_holder.cc
diff --git a/gin/isolate_holder.cc b/gin/isolate_holder.cc
index d42607d86ddea5d4c0915509e1b2dc72ca664176..6df4689e0ec516f075268a418eb308056cafbdf2 100644
--- a/gin/isolate_holder.cc
+++ b/gin/isolate_holder.cc
@@ -15,6 +15,7 @@
#include "base/message_loop/message_loop.h"
#include "base/single_thread_task_runner.h"
#include "base/sys_info.h"
+#include "build/build_config.h"
#include "gin/debug_impl.h"
#include "gin/function_template.h"
#include "gin/per_isolate_data.h"
@@ -35,15 +36,22 @@ IsolateHolder::IsolateHolder(
IsolateHolder::IsolateHolder(
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
AccessMode access_mode)
- : IsolateHolder(std::move(task_runner), access_mode, kAllowAtomicsWait) {}
+ : IsolateHolder(std::move(task_runner),
+ access_mode,
+ kAllowAtomicsWait,
+ nullptr,
+ nullptr) {}
IsolateHolder::IsolateHolder(
scoped_refptr<base::SingleThreadTaskRunner> task_runner,
AccessMode access_mode,
- AllowAtomicsWaitMode atomics_wait_mode)
+ AllowAtomicsWaitMode atomics_wait_mode,
+ intptr_t* reference,
+ v8::StartupData* startup_data)
: access_mode_(access_mode) {
v8::ArrayBuffer::Allocator* allocator = g_array_buffer_allocator;
CHECK(allocator) << "You need to invoke gin::IsolateHolder::Initialize first";
+
v8::Isolate::CreateParams params;
params.entry_hook = DebugImpl::GetFunctionEntryHook();
params.code_event_handler = DebugImpl::GetJitCodeEventHandler();
@@ -51,45 +59,28 @@ IsolateHolder::IsolateHolder(
base::SysInfo::AmountOfVirtualMemory());
params.array_buffer_allocator = allocator;
params.allow_atomics_wait = atomics_wait_mode == kAllowAtomicsWait;
- isolate_ = v8::Isolate::New(params);
- isolate_data_.reset(
- new PerIsolateData(isolate_, allocator, access_mode, task_runner));
- isolate_memory_dump_provider_.reset(new V8IsolateMemoryDumpProvider(this));
-#if defined(OS_WIN)
- {
- void* code_range;
- size_t size;
- isolate_->GetCodeRange(&code_range, &size);
- Debug::CodeRangeCreatedCallback callback =
- DebugImpl::GetCodeRangeCreatedCallback();
- if (code_range && size && callback)
- callback(code_range, size);
+ params.external_references = reference;
+
+ if (startup_data) {
+ CHECK(reference);
+ V8Initializer::GetV8ContextSnapshotData(&startup_data->data,
+ &startup_data->raw_size);
+ if (startup_data->data) {
+ params.snapshot_blob = startup_data;
+ }
}
-#endif
+ isolate_ = v8::Isolate::New(params);
+
+ SetUp(std::move(task_runner));
}
IsolateHolder::IsolateHolder(intptr_t* reference_table,
v8::StartupData* existing_blob)
: snapshot_creator_(
new v8::SnapshotCreator(reference_table, existing_blob)),
- access_mode_(kSingleThread) {
- v8::ArrayBuffer::Allocator* allocator = g_array_buffer_allocator;
- CHECK(allocator) << "You need to invoke gin::IsolateHolder::Initialize first";
- isolate_ = snapshot_creator_->GetIsolate();
- isolate_data_.reset(
- new PerIsolateData(isolate_, allocator, access_mode_, nullptr));
- isolate_memory_dump_provider_.reset(new V8IsolateMemoryDumpProvider(this));
-#if defined(OS_WIN)
- {
- void* code_range;
- size_t size;
- isolate_->GetCodeRange(&code_range, &size);
- Debug::CodeRangeCreatedCallback callback =
- DebugImpl::GetCodeRangeCreatedCallback();
- if (code_range && size && callback)
- callback(code_range, size);
- }
-#endif
+ isolate_(snapshot_creator_->GetIsolate()),
+ access_mode_(AccessMode::kSingleThread) {
+ SetUp(nullptr);
}
IsolateHolder::~IsolateHolder() {
@@ -109,7 +100,7 @@ IsolateHolder::~IsolateHolder() {
isolate_memory_dump_provider_.reset();
isolate_data_.reset();
isolate_->Dispose();
- isolate_ = NULL;
+ isolate_ = nullptr;
}
// static
@@ -139,4 +130,24 @@ void IsolateHolder::EnableIdleTasks(
isolate_data_->EnableIdleTasks(std::move(idle_task_runner));
}
+void IsolateHolder::SetUp(
+ scoped_refptr<base::SingleThreadTaskRunner> task_runner) {
+ v8::ArrayBuffer::Allocator* allocator = g_array_buffer_allocator;
+ CHECK(allocator) << "You need to invoke gin::IsolateHolder::Initialize first";
+ isolate_data_.reset(
+ new PerIsolateData(isolate_, allocator, access_mode_, task_runner));
+ isolate_memory_dump_provider_.reset(new V8IsolateMemoryDumpProvider(this));
+#if defined(OS_WIN)
+ {
+ void* code_range;
+ size_t size;
+ isolate_->GetCodeRange(&code_range, &size);
+ Debug::CodeRangeCreatedCallback callback =
+ DebugImpl::GetCodeRangeCreatedCallback();
+ if (code_range && size && callback)
+ callback(code_range, size);
+ }
+#endif
+}
+
} // namespace gin
« no previous file with comments | « extensions/shell/BUILD.gn ('k') | gin/public/isolate_holder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698