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

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

Issue 1196163006: Fix Javascript tests after commit f55771af6429. (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master
Patch Set: Created 5 years, 6 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/javascript/Document.cpp ('k') | testing/resources/javascript/document_methods_expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: testing/resources/javascript/document_methods.in
diff --git a/testing/resources/javascript/document_methods.in b/testing/resources/javascript/document_methods.in
index 662c05c8e05bfb8f2f9a3d77e3405389d3b69128..b2467f37c4f02ded457c3dd060acc9339e4dc73b 100644
--- a/testing/resources/javascript/document_methods.in
+++ b/testing/resources/javascript/document_methods.in
@@ -134,14 +134,6 @@ function testCalculateNow() {
// TODO(tsepez): test success cases.
}
-function testDeletePages() {
- // Method is present.
- expect('typeof this.deletePages', 'function');
-
- // TODO(tsepez): test with no permissions.
- // TODO(tsepez): test success cases.
-}
-
function testGetField() {
// Method is present.
expect('typeof this.getField', 'function');
@@ -255,16 +247,6 @@ function testRemoveField() {
// TODO(tsepez): test success cases.
}
-function testRemoveIcon() {
- // Method is present.
- expect('typeof this.removeIcon', 'function');
-
- // Method requires at least one argument.
- expectError('this.removeIcon()');
-
- // TODO(tsepez): test success cases.
-}
-
function testResetForm() {
// Method is present.
expect('typeof this.resetForm', 'function');
@@ -290,6 +272,7 @@ try {
testUnsupported('this.addLink');
testUnsupported('this.closeDoc');
testUnsupported('this.createDataObject');
+ testUnsupported('this.deletePages');
testUnsupported('this.exportAsFDF');
testUnsupported('this.exportAsText');
testUnsupported('this.exportAsXFDF');
@@ -305,13 +288,13 @@ try {
testUnsupported('this.importAnXFDF');
testUnsupported('this.importTextData');
testUnsupported('this.insertPages');
+ testUnsupported('this.removeIcon');
testUnsupported('this.replacePages');
testUnsupported('this.saveAs');
app.alert('*** Testing Supported Methods ***');
testAddIcon();
testCalculateNow();
- testDeletePages();
testGetField();
testGetIcon();
testGetNthFieldName();
@@ -323,7 +306,6 @@ try {
testMailForm();
testPrint();
testRemoveField();
- testRemoveIcon();
testResetForm();
testSubmitForm();
} catch (e) {
« no previous file with comments | « fpdfsdk/src/javascript/Document.cpp ('k') | testing/resources/javascript/document_methods_expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698