Remove unused methods from JS_Object.cpp

Remove unused methods from JS_Object.cpp

Committed: https://pdfium.googlesource.com/pdfium/+/3999d6a4ec124b2e5f68aadb7b51f387733a96b5

[Tom Sepez, 2016-07-21 18:57:14]

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

[Tom Sepez, 2016-07-21 18:57:15]

dsinclair, nothing but red.

[Tom Sepez, 2016-07-21 18:57:36]

The CQ bit was checked by tsepez@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-21 18:57:38]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dsinclair, 2016-07-21 19:00:20]

lgtm w/ question

https://codereview.chromium.org/2170913003/diff/1/fpdfsdk/javascript/JS_Objec...
File fpdfsdk/javascript/JS_Object.cpp (right):

https://codereview.chromium.org/2170913003/diff/1/fpdfsdk/javascript/JS_Objec...
fpdfsdk/javascript/JS_Object.cpp:15: int FXJS_MsgBox(CPDFDoc_Environment* pApp,
Can we move this straight into CJS_EmbedObj::MsgBox since it's the only caller
now?

[Tom Sepez, 2016-07-21 19:02:55]

On 2016/07/21 19:00:20, dsinclair wrote:
> lgtm w/ question
> 
>
https://codereview.chromium.org/2170913003/diff/1/fpdfsdk/javascript/JS_Objec...
> File fpdfsdk/javascript/JS_Object.cpp (right):
> 
>
https://codereview.chromium.org/2170913003/diff/1/fpdfsdk/javascript/JS_Objec...
> fpdfsdk/javascript/JS_Object.cpp:15: int FXJS_MsgBox(CPDFDoc_Environment*
pApp,
> Can we move this straight into CJS_EmbedObj::MsgBox since it's the only caller
> now?

Yeah, that's the next step.

[Tom Sepez, 2016-07-21 19:03:01]

The CQ bit was checked by tsepez@chromium.org

[Tom Sepez, 2016-07-21 19:03:02]

The CQ bit was unchecked by tsepez@chromium.org

[Tom Sepez, 2016-07-21 19:03:07]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-07-21 19:03:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-21 19:12:40]

Description was changed from

==========
Remove unused methods from JS_Object.cpp
==========

to

==========
Remove unused methods from JS_Object.cpp

Committed:
https://pdfium.googlesource.com/pdfium/+/3999d6a4ec124b2e5f68aadb7b51f387733a...
==========

[commit-bot: I haz the power, 2016-07-21 19:12:41]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/3999d6a4ec124b2e5f68aadb7b51f387733a...