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

Unified Diff: gin/isolate_holder.cc

Issue 765053003: Windows-specific bits for loading V8's initial snapshot from external files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move FILES.cfg to a separate CL and assume blobs are in versions folder Created 6 years 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 | « chrome/installer/mini_installer.gypi ('k') | no next file » | 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 2b6d64b35c042b2e183e6cc0c963c2f2ab96d392..26450b652c409ecb190976ac35587a70bb91465a 100644
--- a/gin/isolate_holder.cc
+++ b/gin/isolate_holder.cc
@@ -47,9 +47,13 @@ bool MapV8Files(base::FilePath* natives_path, base::FilePath* snapshot_path,
g_mapped_natives = new base::MemoryMappedFile;
if (!g_mapped_natives->IsValid()) {
+#ifndef OS_WIN
if (natives_fd == -1
? !g_mapped_natives->Initialize(base::File(*natives_path, flags))
: !g_mapped_natives->Initialize(base::File(natives_fd))) {
+#else
+ if (!g_mapped_natives->Initialize(base::File(*natives_path, flags))) {
+#endif // OS_WIN
delete g_mapped_natives;
g_mapped_natives = NULL;
LOG(FATAL) << "Couldn't mmap v8 natives data file";
@@ -59,9 +63,13 @@ bool MapV8Files(base::FilePath* natives_path, base::FilePath* snapshot_path,
g_mapped_snapshot = new base::MemoryMappedFile;
if (!g_mapped_snapshot->IsValid()) {
+#ifndef OS_WIN
if (snapshot_fd == -1
? !g_mapped_snapshot->Initialize(base::File(*snapshot_path, flags))
: !g_mapped_snapshot->Initialize(base::File(snapshot_fd))) {
+#else
+ if (!g_mapped_snapshot->Initialize(base::File(*snapshot_path, flags))) {
+#endif // OS_WIN
delete g_mapped_snapshot;
g_mapped_snapshot = NULL;
LOG(ERROR) << "Couldn't mmap v8 snapshot data file";
@@ -78,6 +86,8 @@ const int v8_snapshot_dir =
base::DIR_ANDROID_APP_DATA;
#elif defined(OS_POSIX)
base::DIR_EXE;
+#elif defined(OS_WIN)
+ base::DIR_MODULE;
#endif // defined(OS_ANDROID)
#endif // !defined(OS_MACOSX)
« no previous file with comments | « chrome/installer/mini_installer.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698