[Bindings] Create and use V8 context snapshots

gin/isolate_holder.cc

 // Copyright 2013 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 "gin/public/isolate_holder.h"
 
 #include <stddef.h>
 #include <stdlib.h>
 #include <string.h>
 
@@ skipping 17 matching lines @@
 v8::ArrayBuffer::Allocator* g_array_buffer_allocator = nullptr;
 }  // namespace
 
 IsolateHolder::IsolateHolder(
     scoped_refptr<base::SingleThreadTaskRunner> task_runner)
     : IsolateHolder(std::move(task_runner), AccessMode::kSingleThread) {}
 
 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();
   params.constraints.ConfigureDefaults(base::SysInfo::AmountOfPhysicalMemory(),
                                        base::SysInfo::AmountOfVirtualMemory());
   params.array_buffer_allocator = allocator;
   params.allow_atomics_wait = atomics_wait_mode == kAllowAtomicsWait;
+  params.external_references = reference;
+
+  if (startup_data) {
+    CHECK(reference);
+    V8Initializer::GetBlinkV8SnapshotData(&startup_data->data,
+                                          &startup_data->raw_size);
+    if (startup_data->data) {
+      params.snapshot_blob = startup_data;
+    }
+  }
   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);
-  }
-#endif
+
+  Setup(std::move(task_runner), access_mode);
 }
 
 IsolateHolder::IsolateHolder(intptr_t* reference_table,
                              v8::StartupData* existing_blob)
     : snapshot_creator_(
           new v8::SnapshotCreator(reference_table, existing_blob)),
-      access_mode_(kSingleThread) {
+      isolate_(snapshot_creator_->GetIsolate()) {
+  Setup(nullptr, kSingleThread);

[Yuki, 2017/06/20 14:20:10]

AccessMode::kSingleThread

[peria, 2017/06/21 07:19:15]

Done.

+}
+
+void IsolateHolder::Setup(

[Yuki, 2017/06/20 14:20:10]

nit: gtest uses SetUp().

[Yuki, 2017/06/20 14:20:10]

Keep the same order of declarations in the header and definitions in the
implementation file.

[peria, 2017/06/21 07:19:15]

Done.

[peria, 2017/06/21 07:19:15]

Done.

+    scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+    AccessMode access_mode) {
+  access_mode_ = access_mode;

[Yuki, 2017/06/20 14:20:10]

access_mode_ was already initialized on line 50.  Shouldn't initialize it twice.

[peria, 2017/06/21 07:19:15]

Done.

   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));
+      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);
@@ skipping 11 matching lines @@
     isolate_->GetCodeRange(&code_range, &size);
     Debug::CodeRangeDeletedCallback callback =
         DebugImpl::GetCodeRangeDeletedCallback();
     if (code_range && callback)
       callback(code_range);
   }
 #endif
   isolate_memory_dump_provider_.reset();
   isolate_data_.reset();
   isolate_->Dispose();
-  isolate_ = NULL;
+  isolate_ = nullptr;
 }
 
 // static
 void IsolateHolder::Initialize(ScriptMode mode,
                                V8ExtrasMode v8_extras_mode,
                                v8::ArrayBuffer::Allocator* allocator) {
   CHECK(allocator);
   V8Initializer::Initialize(mode, v8_extras_mode);
   g_array_buffer_allocator = allocator;
 }
@@ skipping 10 matching lines @@
   task_observer_.reset();
 }
 
 void IsolateHolder::EnableIdleTasks(
     std::unique_ptr<V8IdleTaskRunner> idle_task_runner) {
   DCHECK(isolate_data_.get());
   isolate_data_->EnableIdleTasks(std::move(idle_task_runner));
 }
 
 }  // namespace gin