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

Issue 1394103002: Wean CJS_Value off of v8::Isolate. (Closed)

Created:
5 years, 2 months ago by Tom Sepez
Modified:
5 years, 2 months ago
Reviewers:
Lei Zhang
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Wean CJS_Value off of v8::Isolate. CJS_Values should belong to CJS_Runtimes so that we may eventually cram much of the v8 dependencies down into fxjs. This is a first step; the remaining split in this code between isolate and CJS_Runtime goes away when fxjs provides a CFXJS_Runtime object, and the CJS_Runtime is-a/has-a CFXJS_Runtime. But that can't happen until this is resolved. R=thestig@chromium.org Committed: https://pdfium.googlesource.com/pdfium/+/287b63d9ab410783d71cf37291f09fd57d3059b4

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+377 lines, -487 lines) Patch
M fpdfsdk/src/javascript/Consts.h View 1 chunk +2 lines, -2 lines 0 comments Download
M fpdfsdk/src/javascript/Consts.cpp View 8 chunks +32 lines, -32 lines 1 comment Download
M fpdfsdk/src/javascript/Document.cpp View 11 chunks +28 lines, -39 lines 1 comment Download
M fpdfsdk/src/javascript/Field.cpp View 27 chunks +37 lines, -67 lines 0 comments Download
M fpdfsdk/src/javascript/JS_Define.h View 8 chunks +22 lines, -16 lines 0 comments Download
M fpdfsdk/src/javascript/JS_Runtime.h View 1 chunk +2 lines, -0 lines 0 comments Download
M fpdfsdk/src/javascript/JS_Runtime.cpp View 2 chunks +9 lines, -2 lines 0 comments Download
M fpdfsdk/src/javascript/JS_Value.h View 7 chunks +26 lines, -25 lines 0 comments Download
M fpdfsdk/src/javascript/JS_Value.cpp View 21 chunks +89 lines, -84 lines 0 comments Download
M fpdfsdk/src/javascript/PublicMethods.h View 1 chunk +1 line, -1 line 0 comments Download
M fpdfsdk/src/javascript/PublicMethods.cpp View 24 chunks +42 lines, -94 lines 0 comments Download
M fpdfsdk/src/javascript/app.cpp View 12 chunks +36 lines, -47 lines 0 comments Download
M fpdfsdk/src/javascript/color.cpp View 8 chunks +34 lines, -47 lines 0 comments Download
M fpdfsdk/src/javascript/global.h View 2 chunks +3 lines, -4 lines 0 comments Download
M fpdfsdk/src/javascript/global.cpp View 6 chunks +10 lines, -10 lines 0 comments Download
M fpdfsdk/src/javascript/util.cpp View 4 chunks +4 lines, -17 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Tom Sepez
Lei, first step. https://codereview.chromium.org/1394103002/diff/1/fpdfsdk/src/javascript/Consts.cpp File fpdfsdk/src/javascript/Consts.cpp (right): https://codereview.chromium.org/1394103002/diff/1/fpdfsdk/src/javascript/Consts.cpp#newcode135 fpdfsdk/src/javascript/Consts.cpp:135: FXJS_DefineGlobalConst(pRuntime->GetIsolate(), pConstName, i.e. down the road, ...
5 years, 2 months ago (2015-10-07 23:06:23 UTC) #2
Lei Zhang
lgtm https://codereview.chromium.org/1394103002/diff/1/fpdfsdk/src/javascript/Document.cpp File fpdfsdk/src/javascript/Document.cpp (right): https://codereview.chromium.org/1394103002/diff/1/fpdfsdk/src/javascript/Document.cpp#newcode1387 fpdfsdk/src/javascript/Document.cpp:1387: CJS_Context* pContext = static_cast<CJS_Context*>(cc); Just move this 1 ...
5 years, 2 months ago (2015-10-08 05:40:51 UTC) #3
Tom Sepez
5 years, 2 months ago (2015-10-08 19:04:44 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
287b63d9ab410783d71cf37291f09fd57d3059b4 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698