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

Unified Diff: chrome/test/data/extensions/api_test/filebrowser_component/main.js

Issue 7811006: Add full support for filesystem URLs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Style fixes, removed duplicate set of var Created 8 years, 9 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/extensions/api_test/filebrowser_component/main.js
diff --git a/chrome/test/data/extensions/api_test/filebrowser_component/main.js b/chrome/test/data/extensions/api_test/filebrowser_component/main.js
index 38ffa8a6ad9aa5ee7361e92edc119099440bd504..102b9278b5283579d9a12c1b8470a49734c824a8 100644
--- a/chrome/test/data/extensions/api_test/filebrowser_component/main.js
+++ b/chrome/test/data/extensions/api_test/filebrowser_component/main.js
@@ -14,7 +14,7 @@ This component extension test does the following:
var cleanupError = 'Got unexpected error while cleaning up test directory.';
-// Class specified by the client runnig the TestRunner.
+// Class specified by the client running the TestRunner.
// |expectedTasks| should contain list of actions defined for abc files defined
// by filesystem_handler part of the test.
// |fileVerifierFunction| method that will verify test results received from the
@@ -79,7 +79,9 @@ TestExpectations.prototype.verifyTasks = function(tasks,
patterns = patterns.sort();
expectedPatterns = expectedPatterns.sort();
for (var j = 0; j < patterns.length; ++j) {
- if (patterns[j] != expectedPatterns[j]) {
+ var translatedPattern = expectedPatterns[j].replace(
+ /^filesystem:/, "chrome-extension://*/");
+ if (patterns[j] != translatedPattern) {
errorCallback({message: 'Wrong patterns set for task ' +
taskName + '. ' +
'Got: ' + patterns +

Powered by Google App Engine
This is Rietveld 408576698