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

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

Issue 2221823003: Add support to Document::gotoNamedDest method. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 4 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
Index: testing/resources/javascript/document_methods.in
diff --git a/testing/resources/javascript/document_methods.in b/testing/resources/javascript/document_methods.in
index 3cd3330b878d22ef0fdfd1e865cce092f5df5607..bfb98514dd2ee10b5d409c7b2bad11cc7163d96f 100644
--- a/testing/resources/javascript/document_methods.in
+++ b/testing/resources/javascript/document_methods.in
@@ -214,6 +214,17 @@ function testGetPrintParams() {
// TODO(tsepez): test success cases.
}
+function testGotoNamedDest() {
+ // Method is present.
+ expect('typeof this.gotoNamedDest', 'function');
+
+ // Method needs exactly one argument.
+ expectError('this.gotoNamedDest()');
+ expectError('this.gotoNamedDest(1, 2)');
+
+ // TODO: test success cases.
dsinclair 2016/08/08 20:05:29 Typically the format is TODO(username). It doesn't
+}
+
function testMailDoc() {
// Method is present.
expect('typeof this.mailDoc', 'function');
@@ -303,6 +314,7 @@ try {
testGetPageNthWordQuads();
testGetPageNumWords();
testGetPrintParams();
+ testGotoNamedDest();
testMailDoc();
testMailForm();
testPrint();

Powered by Google App Engine
This is Rietveld 408576698