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

Unified Diff: chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js

Issue 340543002: Enable gallery watches on Mac (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make tests pass if recursive watch not supported Created 6 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
Index: chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js
diff --git a/chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js b/chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js
index 47281bdf3437a55eb3702d73143a5ce61139d354..c28e436e1c0546864e031af89e338b5d6dad2fe7 100644
--- a/chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js
+++ b/chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js
@@ -33,14 +33,14 @@ var getAllGalleryWatchCallback = function (results) {
if (results.length == 0) {
chrome.test.sendMessage('gallery_watchers_does_not_exists');
} else {
- var gallery_ids_str = "";
- for (var i = 0; i < results.length; ++i) {
- if (gallery_ids_str != "")
- gallery_ids_str += ", ";
- gallery_ids_str += results[i];
+ for (var i = 0; i < results.lendth; ++i) {
+ var info = chrome.mediaGalleries.getMediaFileSystemMetadata(galleries[i]);
+ if (results[i] !== info.galleryId) {
+ chrome.test.sendMessage('gallery_watcher_mismatch');
+ return;
+ }
}
- chrome.test.sendMessage(
- 'watchers_for_galleries_{' + gallery_ids_str + '}_found');
+ chrome.test.sendMessage('gallery_watcher_checks');
}
};

Powered by Google App Engine
This is Rietveld 408576698