Load V8 external data prior to initializng PDFium SDK.

pdf/out_of_process_instance.cc

 // Copyright (c) 2012 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 "pdf/out_of_process_instance.h"
 
 #include <algorithm>  // for min/max()
 #define _USE_MATH_DEFINES  // for M_PI
 #include <cmath>      // for log() and pow()
 #include <math.h>
@@ skipping 22 matching lines @@
 #include "ppapi/cpp/module.h"
 #include "ppapi/cpp/point.h"
 #include "ppapi/cpp/private/pdf.h"
 #include "ppapi/cpp/private/var_private.h"
 #include "ppapi/cpp/rect.h"
 #include "ppapi/cpp/resource.h"
 #include "ppapi/cpp/url_request_info.h"
 #include "ppapi/cpp/var_array.h"
 #include "ppapi/cpp/var_dictionary.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 #include "v8/include/v8.h"

[raymes, 2014/12/21 22:52:02]

nit: I think we can remove this include now

[Tom Sepez, 2014/12/29 18:18:37]

Done.

 
 #if defined(OS_MACOSX)
 #include "base/mac/mac_util.h"
 #endif
 
 namespace chrome_pdf {
 
 const char kChromePrint[] = "chrome://print/";
 const char kChromeExtension[] =
     "chrome-extension://mhjfbmdgcfjbbpaeojofohoefgiehjai";
@@ skipping 221 matching lines @@
   RequestFilteringInputEvents(PP_INPUTEVENT_CLASS_TOUCH);
 }
 
 OutOfProcessInstance::~OutOfProcessInstance() {
   RemovePerInstanceObject(kPPPPdfInterface, this);
 }
 
 bool OutOfProcessInstance::Init(uint32_t argc,
                                 const char* argn[],
                                 const char* argv[]) {
-  v8::StartupData natives;
-  v8::StartupData snapshot;
-  pp::PDF::GetV8ExternalSnapshotData(this, &natives.data, &natives.raw_size,
-                                     &snapshot.data, &snapshot.raw_size);
-  if (natives.data) {
-    v8::V8::SetNativesDataBlob(&natives);
-    v8::V8::SetSnapshotDataBlob(&snapshot);
-  }
-
   // Check if the PDF is being loaded in the PDF chrome extension. We only allow
   // the plugin to be put into "full frame" mode when it is being loaded in the
   // extension because this enables some features that we don't want pages
   // abusing outside of the extension.
   pp::Var document_url_var = pp::URLUtil_Dev::Get()->GetDocumentURL(this);
   std::string document_url = document_url_var.is_string() ?
       document_url_var.AsString() : std::string();
   std::string extension_url = std::string(kChromeExtension);
   bool in_extension =
       !document_url.compare(0, extension_url.size(), extension_url);
@@ skipping 1092 matching lines @@
 pp::FloatPoint OutOfProcessInstance::BoundScrollOffsetToDocument(
     const pp::FloatPoint& scroll_offset) {
   float max_x = document_size_.width() * zoom_ - plugin_dip_size_.width();
   float x = std::max(std::min(scroll_offset.x(), max_x), 0.0f);
   float max_y = document_size_.height() * zoom_ - plugin_dip_size_.height();
   float y = std::max(std::min(scroll_offset.y(), max_y), 0.0f);
   return pp::FloatPoint(x, y);
 }
 
 }  // namespace chrome_pdf