Make app.cpp match XFA
Master side change for https://codereview.chromium.org/1504413002
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1508133003 .
diff --git a/fpdfsdk/src/javascript/app.cpp b/fpdfsdk/src/javascript/app.cpp
index 205673e..70ce771 100644
--- a/fpdfsdk/src/javascript/app.cpp
+++ b/fpdfsdk/src/javascript/app.cpp
@@ -196,7 +196,6 @@
CFX_WideString& sError) {
if (!vp.IsGetting())
return FALSE;
-
vp << JS_NUM_VIEWERVERSION;
return TRUE;
}
@@ -204,23 +203,19 @@
FX_BOOL app::platform(IJS_Context* cc,
CJS_PropValue& vp,
CFX_WideString& sError) {
- if (vp.IsGetting()) {
- vp << JS_STR_PLATFORM;
- return TRUE;
- }
-
- return FALSE;
+ if (!vp.IsGetting())
+ return FALSE;
+ vp << JS_STR_PLATFORM;
+ return TRUE;
}
FX_BOOL app::language(IJS_Context* cc,
CJS_PropValue& vp,
CFX_WideString& sError) {
- if (vp.IsGetting()) {
- vp << JS_STR_LANGUANGE;
- return TRUE;
- }
-
- return FALSE;
+ if (!vp.IsGetting())
+ return FALSE;
+ vp << JS_STR_LANGUANGE;
+ return TRUE;
}
// creates a new fdf object that contains no data