commit 3466e27e5c7f44773045fbf0469fd830165ed650
author Tom Sepez <tsepez@chromium.org> Wed Mar 18 23:33:35 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Wed Mar 18 23:33:35 2020 +0000
tree cf2ef302aae02dc287474f7cd233cd875c2fa38e
parent 6fb1c39246b2fc67d53408d41d416757cbd60218

Use proper namespace instead of CFX_V8 static functions.

Closer to what our style guide would suggest. In the end, CFX_V8
should go away as it doesn't buy us anything, esp. given that
callbacks will have an isolate but no means of reaching the
CFX_V8 wrapping it.

Change-Id: If2188767ad829cbeccba007d246ce0f65b0ccf22
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/67491
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>

fxjs/cjs_runtime.cpp

diff --git a/fxjs/cjs_runtime.cpp b/fxjs/cjs_runtime.cpp
index efa2ba6..dba7e32 100644
--- a/fxjs/cjs_runtime.cpp
+++ b/fxjs/cjs_runtime.cpp
@@ -36,6 +36,7 @@
 #include "fxjs/cjs_timerobj.h"
 #include "fxjs/cjs_util.h"
 #include "fxjs/cjs_zoomtype.h"
+#include "fxjs/fxv8.h"
 #include "fxjs/js_define.h"
 #include "third_party/base/ptr_util.h"
 
@@ -181,7 +182,7 @@
   v8::Isolate::Scope isolate_scope(GetIsolate());
   v8::Local<v8::Context> context = GetV8Context();
   v8::Context::Scope context_scope(context);
-  v8::Local<v8::String> str = CFX_V8::NewStringHelper(GetIsolate(), utf8Name);
+  v8::Local<v8::String> str = fxv8::NewStringHelper(GetIsolate(), utf8Name);
   v8::MaybeLocal<v8::Value> maybe_propvalue =
       context->Global()->Get(context, str);
   if (maybe_propvalue.IsEmpty())
@@ -200,7 +201,7 @@
   v8::Isolate::Scope isolate_scope(pIsolate);
   v8::Local<v8::Context> context = GetV8Context();
   v8::Context::Scope context_scope(context);
-  v8::Local<v8::String> str = CFX_V8::NewStringHelper(pIsolate, utf8Name);
+  v8::Local<v8::String> str = fxv8::NewStringHelper(pIsolate, utf8Name);
   v8::Maybe<bool> result = context->Global()->Set(context, str, pValue);
   return result.IsJust() && result.FromJust();
 }