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

Unified Diff: chrome/test/data/pdf/basic_test.js

Issue 266413002: Refactor pdf_scripting_api.js to remove the PDFMessagingHost in OOP PDF (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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: chrome/test/data/pdf/basic_test.js
diff --git a/chrome/test/data/pdf/basic_test.js b/chrome/test/data/pdf/basic_test.js
index 1ad0012aeed2898c7139898d510dd0f20b90c708..e17ad05eebbbc1591016b22938c1a1d880dadf7a 100644
--- a/chrome/test/data/pdf/basic_test.js
+++ b/chrome/test/data/pdf/basic_test.js
@@ -22,6 +22,7 @@ var tests = [
chrome.test.assertTrue(
String(element.constructor).indexOf(elementNames[i]) != -1);
}
+ chrome.test.succeed();
},
/**
@@ -33,15 +34,8 @@ var tests = [
chrome.test.assertTrue(
plugin.getAttribute('src').indexOf('/pdf/test.pdf') != -1);
+ chrome.test.succeed();
},
];
-function runTests() {
- for (var i = 0; i < tests.length; ++i) {
- console.log('Running: ' + tests[i].name);
- tests[i]();
- }
- chrome.test.notifyPass();
-}
-
-runTests();
+chrome.test.runTests(tests);

Powered by Google App Engine
This is Rietveld 408576698