Beef up app_props.in and tidy app.cpp

Beef up app_props.in and tidy app.cpp.

Some of the values returned are dubious; capture the
current state of affairs.

R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/b830fbc03a6d12d8b5be50e85499c281a4c5da7f

[Tom Sepez, 2015-09-08 22:35:06]

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

[Tom Sepez, 2015-09-08 22:35:07]

Lei, for review.

[Tom Sepez, 2015-09-08 23:00:10]

On 2015/09/08 22:35:07, Tom Sepez wrote:
> Lei, for review.

And since no good CL is no good without code changes :( ... just because.

[Lei Zhang, 2015-09-08 23:21:36]

lgtm

https://codereview.chromium.org/1317393003/diff/20001/fpdfsdk/src/javascript/...
File fpdfsdk/src/javascript/app.cpp (left):

https://codereview.chromium.org/1317393003/diff/20001/fpdfsdk/src/javascript/...
fpdfsdk/src/javascript/app.cpp:56: #define JS_STR_VIEWERTYPE_READER L"Reader"
Ah, these are the standard return values for Acrobat.

https://codereview.chromium.org/1317393003/diff/20001/testing/resources/javas...
File testing/resources/javascript/app_props_expected.txt (right):

https://codereview.chromium.org/1317393003/diff/20001/testing/resources/javas...
testing/resources/javascript/app_props_expected.txt:35: Alert: app.platform is
string WIN
I didn't check the values against the Acrobat PDF JS doc, so some of these may
not correct. This one seems wrong for sure when we're not on Windows. We can fix
them as we go.

[Tom Sepez, 2015-09-08 23:23:43]

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