Remove FXJSE_HOBJECT and FXJSE_HVALUE for CFXJSE_Value*

Remove FXJSE_HOBJECT and FXJSE_HVALUE for CFXJSE_Value*

This CL replaces FXJSE_HOBJECT and FXJSE_HVALUE with the concrete CFXJSE_Value*
type. All variables are renamed to match.

Committed: https://pdfium.googlesource.com/pdfium/+/12a6b0c1bb3ab86a03a84464bed168995ae0d82a

[dsinclair, 2016-05-26 15:21:29]

Patchset #1 (id:1) has been deleted

[dsinclair, 2016-05-26 15:40:17]

dsinclair@chromium.org changed reviewers:
+ thestig@chromium.org, tsepez@chromium.org

[dsinclair, 2016-05-26 15:40:18]

PTAL.

[Tom Sepez, 2016-05-26 15:51:57]

lgtm

https://codereview.chromium.org/2012253002/diff/20001/xfa/fxfa/parser/xfa_scr...
File xfa/fxfa/parser/xfa_script_imp.cpp (left):

https://codereview.chromium.org/2012253002/diff/20001/xfa/fxfa/parser/xfa_scr...
xfa/fxfa/parser/xfa_script_imp.cpp:222:
(CXFA_ScriptContext*)pOrginalObject->GetDocument()->GetScriptContext();
nice.

[dsinclair, 2016-05-26 17:18:46]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-05-26 17:18:47]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/2012253002/#ps80001
(title: "Rebase to master")

[commit-bot: I haz the power, 2016-05-26 17:18:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2012253002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2012253002/80001

[commit-bot: I haz the power, 2016-05-26 17:30:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-26 17:30:55]

Try jobs failed on following builders:
  mac_xfa on master.tryserver.client.pdfium (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.pdfium/builders/mac_xfa/builds/806)

[dsinclair, 2016-05-26 17:59:20]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-05-26 17:59:21]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/2012253002/#ps100001
(title: " ")

[commit-bot: I haz the power, 2016-05-26 17:59:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2012253002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2012253002/100001

[commit-bot: I haz the power, 2016-05-26 18:14:11]

Description was changed from

==========
Remove FXJSE_HOBJECT and FXJSE_HVALUE for CFXJSE_Value*

This CL replaces FXJSE_HOBJECT and FXJSE_HVALUE with the concrete CFXJSE_Value*
type. All variables are renamed to match.
==========

to

==========
Remove FXJSE_HOBJECT and FXJSE_HVALUE for CFXJSE_Value*

This CL replaces FXJSE_HOBJECT and FXJSE_HVALUE with the concrete CFXJSE_Value*
type. All variables are renamed to match.

Committed:
https://pdfium.googlesource.com/pdfium/+/12a6b0c1bb3ab86a03a84464bed168995ae0...
==========

[commit-bot: I haz the power, 2016-05-26 18:14:13]

Committed patchset #5 (id:100001) as
https://pdfium.googlesource.com/pdfium/+/12a6b0c1bb3ab86a03a84464bed168995ae0...