Merge to XFA: Pin V8 version until initialization issue in b584bab resolved.
authorTom Sepez <tsepez@chromium.org>
Tue, 28 Apr 2015 21:31:15 +0000 (14:31 -0700)
committerTom Sepez <tsepez@chromium.org>
Tue, 28 Apr 2015 21:31:15 +0000 (14:31 -0700)
Also make a Handle => Local change for XFA required at
https://chromium.googlesource.com/v8/v8.git/+/202a97c88eb6807915135a1e6be9aa921cd8efac
to fix compile.

Original Review URL https://codereview.chromium.org/1108913004/
TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/1108973005

DEPS
xfa/src/fxjse/src/value.cpp

diff --git a/DEPS b/DEPS
index 88e3a3c..6f8417f 100644 (file)
--- a/DEPS
+++ b/DEPS
@@ -12,7 +12,7 @@ deps = {
      "https://chromium.googlesource.com/external/googletest.git@8245545b6dc9c4703e6496d1efd19e975ad2b038",
 
   "v8":
-    "https://chromium.googlesource.com/v8/v8.git",
+    "https://chromium.googlesource.com/v8/v8.git@6988aec61f071a1caec96abae1cf39353a234455",
 
   "v8/third_party/icu":
     "https://chromium.googlesource.com/chromium/deps/icu46",
index 9bbfdf7..de7e054 100644 (file)
@@ -506,7 +506,7 @@ FX_BOOL CFXJSE_Value::Call(CFXJSE_Value* lpReceiver, CFXJSE_Value* lpRetValue, F
     }\r
     if(lpLocalArgs) {\r
         for (FX_UINT32 i = 0; i < nArgCount; i++) {\r
-            lpLocalArgs[i].~Handle();\r
+            lpLocalArgs[i].~Local();\r
         }\r
         FX_Free(lpLocalArgs);\r
     }\r