Expose gallery watching functionality to the public mediaGalleries API.

chrome/test/data/extensions/api_test/media_galleries/gallerywatch/test.js

Index: chrome/test/data/extensions/api_test/media_galleries/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/gallerywatch/test.js
similarity index 65%
copy from chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js
copy to chrome/test/data/extensions/api_test/media_galleries/gallerywatch/test.js
index c28e436e1c0546864e031af89e338b5d6dad2fe7..94fd8afc4679459349e4ced9297b428ce6e0cd17 100644
--- a/chrome/test/data/extensions/api_test/media_galleries_private/gallerywatch/test.js
+++ b/chrome/test/data/extensions/api_test/media_galleries/gallerywatch/test.js
@@ -18,10 +18,11 @@ var onGalleryChangedCallback = function (details) {
 
 // Add watch request callback.
 var onAddWatchRequestCallback = function (details) {
-  if (!details || !details.success)
+  if (!details || !details.success) {
     chrome.test.sendMessage('add_watch_request_failed');
-  else
+  } else {
     chrome.test.sendMessage('add_watch_request_succeeded');
+  }
 };
 
 // chrome.mediaGalleries.getAllGalleryWatch callback.
@@ -33,7 +34,7 @@ var getAllGalleryWatchCallback = function (results) {
   if (results.length == 0) {
     chrome.test.sendMessage('gallery_watchers_does_not_exists');
   } else {
-    for (var i = 0; i < results.lendth; ++i) {
+    for (var i = 0; i < results.length; ++i) {
       var info = chrome.mediaGalleries.getMediaFileSystemMetadata(galleries[i]);
       if (results[i] !== info.galleryId) {
         chrome.test.sendMessage('gallery_watcher_mismatch');
@@ -46,7 +47,7 @@ var getAllGalleryWatchCallback = function (results) {
 
 // Helpers to add and remove event listeners.
 function addGalleryChangedListener() {
-  chrome.mediaGalleriesPrivate.onGalleryChanged.addListener(
+  chrome.mediaGalleries.onGalleryChanged.addListener(
       onGalleryChangedCallback);
   chrome.test.sendMessage('add_gallery_changed_listener_ok');
 };
@@ -54,15 +55,40 @@ function addGalleryChangedListener() {
 function setupWatchOnValidGalleries() {
   for (var i = 0; i < galleries.length; ++i) {
     var info = chrome.mediaGalleries.getMediaFileSystemMetadata(galleries[i]);
-    chrome.mediaGalleriesPrivate.addGalleryWatch(info.galleryId,
-                                                 onAddWatchRequestCallback);
+    chrome.mediaGalleries.addGalleryWatch(info.galleryId,
+                                          onAddWatchRequestCallback);
+  }
+  chrome.test.sendMessage('add_gallery_watch_ok');
+};
+
+function getUnlistenedAddWatchCallback(expectedNumCalls) {

[tommycli, 2014/08/20 23:21:13]

Since this is a bit non-obvious, maybe add some documentation for the next guy.

[Oren Blasberg, 2014/08/21 02:45:21]

Done.

+  var numCalls = 0;
+  var numErrors = 0;
+  return function() {
+    numCalls++;
+    if (chrome.runtime.lastError) {
+      numErrors++;
+    }
+
+    if (numCalls == expectedNumCalls && numErrors == expectedNumCalls) {
+      chrome.test.sendMessage('add_watch_request_runtime_error');
+    }
+  };
+}
+
+function setupWatchOnUnlistenedValidGalleries() {
+  var callback = getUnlistenedAddWatchCallback(galleries.length);
+  for (var i = 0; i < galleries.length; ++i) {
+    var info = chrome.mediaGalleries.getMediaFileSystemMetadata(galleries[i]);
+    chrome.mediaGalleries.addGalleryWatch(info.galleryId, callback);
   }
+
   chrome.test.sendMessage('add_gallery_watch_ok');
 };
 
 function setupWatchOnInvalidGallery() {
-  chrome.mediaGalleriesPrivate.addGalleryWatch(invalidGalleryId,
-                                               onAddWatchRequestCallback);
+  chrome.mediaGalleries.addGalleryWatch(invalidGalleryId,
+                                        onAddWatchRequestCallback);
   chrome.test.sendMessage('add_gallery_watch_ok');
 }
 
@@ -74,23 +100,23 @@ function getMediaFileSystems() {
 function removeGalleryWatch() {
   for (var i = 0; i < galleries.length; ++i) {
     var info = chrome.mediaGalleries.getMediaFileSystemMetadata(galleries[i]);
-    chrome.mediaGalleriesPrivate.removeGalleryWatch(info.galleryId);
+    chrome.mediaGalleries.removeGalleryWatch(info.galleryId);
   }
   chrome.test.sendMessage('remove_gallery_watch_ok');
 };
 
 function removeGalleryChangedListener() {
-  chrome.mediaGalleriesPrivate.onGalleryChanged.removeListener(
+  chrome.mediaGalleries.onGalleryChanged.removeListener(
       onGalleryChangedCallback);
   chrome.test.sendMessage('remove_gallery_changed_listener_ok');
 };
 
 function getAllWatchedGalleryIds() {
-  chrome.mediaGalleriesPrivate.getAllGalleryWatch(getAllGalleryWatchCallback);
+  chrome.mediaGalleries.getAllGalleryWatch(getAllGalleryWatchCallback);
   chrome.test.sendMessage('get_all_gallery_watch_ok');
 };
 
 function removeAllGalleryWatch() {
-  chrome.mediaGalleriesPrivate.removeAllGalleryWatch();
+  chrome.mediaGalleries.removeAllGalleryWatch();
   chrome.test.sendMessage('remove_all_gallery_watch_ok');
 };