Fix an issue that no response from the call back function of FFI_GetLanguage()

Fix an issue that no response from the call back function of FFI_GetLanguage()

BUG=pdfium:295
R=tsepez@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/487d1a90a0902de675261f27f8186c84440ca573

[jun_fang, 2015-11-30 11:14:49]

jun_fang@foxitsoftware.com changed reviewers:
+ kai_jing@foxitsoftware.com, thestig@chromium.org, tsepez@chromium.org

[jun_fang, 2015-11-30 11:14:50]

Hi Tom and Lei,

Please review this CL. Thanks!

[Tom Sepez, 2015-11-30 17:08:39]

https://codereview.chromium.org/1482253002/diff/1/fpdfsdk/src/javascript/app.cpp
File fpdfsdk/src/javascript/app.cpp (right):

https://codereview.chromium.org/1482253002/diff/1/fpdfsdk/src/javascript/app....
fpdfsdk/src/javascript/app.cpp:233: CJS_Context* pContext =
static_cast<CJS_Context*>(cc);
Can these actually be null? we don't check it in other places, e.g. see beep().
I'd expect we can remove most of these.

[jun_fang, 2015-12-01 11:05:29]

On 2015/11/30 17:08:39, Tom Sepez wrote:
>
https://codereview.chromium.org/1482253002/diff/1/fpdfsdk/src/javascript/app.cpp
> File fpdfsdk/src/javascript/app.cpp (right):
> 
>
https://codereview.chromium.org/1482253002/diff/1/fpdfsdk/src/javascript/app....
> fpdfsdk/src/javascript/app.cpp:233: CJS_Context* pContext =
> static_cast<CJS_Context*>(cc);
> Can these actually be null? we don't check it in other places, e.g. see
beep().
> I'd expect we can remove most of these.

Hi Tom,

Please check patch 2.

[Tom Sepez, 2015-12-01 17:21:24]

lgtm

[jun_fang, 2015-12-02 05:20:17]

Description was changed from

==========
Fix an issue that no response from the call back function of FFI_GetLanguage()

BUG=pdfium:295
==========

to

==========
Fix an issue that no response from the call back function of FFI_GetLanguage()

BUG=pdfium:295
R=tsepez@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/487d1a90a0902de675261f27f8186c84440c...
==========

[jun_fang, 2015-12-02 05:20:21]

Committed patchset #2 (id:20001) manually as
487d1a90a0902de675261f27f8186c84440ca573 (presubmit successful).