Remove unused context param from execute.

Remove unused context param from execute.

The IJS_Context parameter is never used, removed.

Committed: https://pdfium.googlesource.com/pdfium/+/884b4f942d510974e713361b84a717340eb928b9

[dsinclair, 2016-06-02 14:30:06]

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

[dsinclair, 2016-06-02 14:30:07]

PTAL.

[Tom Sepez, 2016-06-02 18:25:36]

https://codereview.chromium.org/2035743002/diff/20001/fpdfsdk/jsapi/fxjs_v8.cpp
File fpdfsdk/jsapi/fxjs_v8.cpp (right):

https://codereview.chromium.org/2035743002/diff/20001/fpdfsdk/jsapi/fxjs_v8.c...
fpdfsdk/jsapi/fxjs_v8.cpp:427: CFX_ByteString bsScript =
CFX_WideString(script).UTF8Encode();
this started out as a widestring in the caller, then we made it a wchar*, now we
make it a widestring again.  Just pass the widestring and save a re-allocation.

[dsinclair, 2016-06-02 18:57:45]

https://codereview.chromium.org/2035743002/diff/20001/fpdfsdk/jsapi/fxjs_v8.cpp
File fpdfsdk/jsapi/fxjs_v8.cpp (right):

https://codereview.chromium.org/2035743002/diff/20001/fpdfsdk/jsapi/fxjs_v8.c...
fpdfsdk/jsapi/fxjs_v8.cpp:427: CFX_ByteString bsScript =
CFX_WideString(script).UTF8Encode();
On 2016/06/02 18:25:35, Tom Sepez wrote:
> this started out as a widestring in the caller, then we made it a wchar*, now
we
> make it a widestring again.  Just pass the widestring and save a
re-allocation.

Done.

[Tom Sepez, 2016-06-02 20:39:31]

Several other places.

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/JS_R...
File fpdfsdk/javascript/JS_Runtime_Stub.cpp (right):

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/JS_R...
fpdfsdk/javascript/JS_Runtime_Stub.cpp:151: int Execute(const CFX_WideString
script, CFX_WideString* info) override {
const CFX_WideString& script

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/cjs_...
File fpdfsdk/javascript/cjs_runtime.h (right):

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/cjs_...
fpdfsdk/javascript/cjs_runtime.h:46: int Execute(const CFX_WideString script,
CFX_WideString* info) override;
ditto

[dsinclair, 2016-06-06 12:56:17]

PTAL.

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/JS_R...
File fpdfsdk/javascript/JS_Runtime_Stub.cpp (right):

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/JS_R...
fpdfsdk/javascript/JS_Runtime_Stub.cpp:151: int Execute(const CFX_WideString
script, CFX_WideString* info) override {
On 2016/06/02 20:39:30, Tom Sepez wrote:
> const CFX_WideString& script

Done.

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/cjs_...
File fpdfsdk/javascript/cjs_runtime.h (right):

https://codereview.chromium.org/2035743002/diff/40001/fpdfsdk/javascript/cjs_...
fpdfsdk/javascript/cjs_runtime.h:46: int Execute(const CFX_WideString script,
CFX_WideString* info) override;
On 2016/06/02 20:39:30, Tom Sepez wrote:
> ditto

Done.

[Tom Sepez, 2016-06-06 16:45:04]

lgtm

[dsinclair, 2016-06-06 16:49:06]

The CQ bit was checked by dsinclair@chromium.org

[commit-bot: I haz the power, 2016-06-06 16:49:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2035743002/60001

[commit-bot: I haz the power, 2016-06-06 16:49:37]

Description was changed from

==========
Remove unused context param from execute.

The IJS_Context parameter is never used, removed.
==========

to

==========
Remove unused context param from execute.

The IJS_Context parameter is never used, removed.

Committed:
https://pdfium.googlesource.com/pdfium/+/884b4f942d510974e713361b84a717340eb9...
==========

[commit-bot: I haz the power, 2016-06-06 16:49:38]

Committed patchset #4 (id:60001) as
https://pdfium.googlesource.com/pdfium/+/884b4f942d510974e713361b84a717340eb9...