Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(503)

Unified Diff: testing/resources/javascript/consts.in

Issue 1342433002: Fix strings, remove stringify macros, void return types for Consts.h (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Added tests Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « fpdfsdk/src/jsapi/fxjs_v8.cpp ('k') | testing/resources/javascript/consts_expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/resources/javascript/consts.in
diff --git a/testing/resources/javascript/consts.in b/testing/resources/javascript/consts.in
index 2ed239a63dd15091aa6d4474499d44819cfa4762..a717ec873ac1f7d66b22ebe890c42770e379cc02 100644
--- a/testing/resources/javascript/consts.in
+++ b/testing/resources/javascript/consts.in
@@ -36,12 +36,37 @@ endobj
stream
function doTest(name, props) {
- for (var i = 0; i < props.length; ++i) {
- var expr = name + "." + props[i];
- app.alert(expr + " is " + eval(expr));
+ try {
+ for (var i = 0; i < props.length; ++i) {
+ var expr = name + "." + props[i];
+ app.alert(expr + " is " + eval(expr));
+ }
+ } catch (e) {
+ app.alert("ERROR: " + e.toString());
}
}
+function doGlobalTest(props) {
+ try {
+ for (var i = 0; i < props.length; ++i) {
+ app.alert(props[i] + " is " + eval(props[i]));
+ }
+ } catch (e) {
+ app.alert("ERROR: " + e.toString());
+ }
+}
+
+function doGlobalArrayTest(props) {
+ try {
+ for (var i = 0; i < props.length; ++i) {
+ app.alert(props[i] + " is\n " + eval(props[i]).join(",\n "));
+ }
+ } catch (e) {
+ app.alert("ERROR: " + e.toString());
+ }
+}
+
+
try {
doTest("border", ["s", "b", "d", "i", "u", "nonesuch"]);
doTest("display", ["visible", "hidden", "noPrint", "noView", "nonesuch"]);
@@ -58,6 +83,35 @@ try {
doTest("style", ["ch", "cr", "di", "ci", "st", "sq", "nonesuch"]);
doTest("zoomtype", ["none", "fitP", "fitW", "fitH", "fitV", "pref", "refW",
"nonesuch"]);
+
+ doGlobalTest([
+ "IDS_GREATER_THAN",
+ "IDS_GT_AND_LT",
+ "IDS_LESS_THAN",
+ "IDS_INVALID_MONTH",
+ "IDS_INVALID_DATE",
+ "IDS_INVALID_VALUE",
+ "IDS_AM",
+ "IDS_PM",
+ "IDS_MONTH_INFO",
+ "IDS_STARTUP_CONSOLE_MSG"
+ ]);
+
+ doGlobalArrayTest([
+ "RE_NUMBER_ENTRY_DOT_SEP",
+ "RE_NUMBER_COMMIT_DOT_SEP",
+ "RE_NUMBER_ENTRY_COMMA_SEP",
+ "RE_NUMBER_COMMIT_COMMA_SEP",
+ "RE_ZIP_ENTRY",
+ "RE_ZIP_COMMIT",
+ "RE_ZIP4_ENTRY",
+ "RE_ZIP4_COMMIT",
+ "RE_PHONE_ENTRY",
+ "RE_PHONE_COMMIT",
+ "RE_SSN_ENTRY",
+ "RE_SSN_COMMIT"
+ ]);
+
} catch (e) {
app.alert("ERROR: " + e.toString());
}
@@ -69,3 +123,4 @@ trailer <<
>>
{{startxref}}
%%EOF
+
« no previous file with comments | « fpdfsdk/src/jsapi/fxjs_v8.cpp ('k') | testing/resources/javascript/consts_expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698