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

Unified Diff: chrome/browser/resources/file_manager/foreground/js/file_tasks.js

Issue 109973002: Migrate from URLs to Entries in the Files App's gallery. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Cleaned up. Created 7 years 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/browser/resources/file_manager/foreground/js/file_tasks.js
diff --git a/chrome/browser/resources/file_manager/foreground/js/file_tasks.js b/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
index adf28ae29d0e584835b6de4028d2f67e40c54282..ee7afc3250ca04916054631cccde3f3506309bd3 100644
--- a/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
+++ b/chrome/browser/resources/file_manager/foreground/js/file_tasks.js
@@ -617,9 +617,8 @@ FileTasks.prototype.openGalleryInternal_ = function(entries) {
// changes in the Gallery and popped when the Gallery is closed.
util.updateAppState();
- // TODO(mtomasz): Pass entries instead of urls.
- var onBack = function(selectedUrls) {
- fm.directoryModel_.selectUrls(selectedUrls);
+ var onBack = function(selectedEntries) {
+ fm.directoryModel_.selectEntries(selectedEntries);
fm.closeFilePopup(); // Will call Gallery.unload.
window.appState = savedAppState;
util.saveAppState();
@@ -673,10 +672,8 @@ FileTasks.prototype.openGalleryInternal_ = function(entries) {
displayStringFunction: strf
};
// TODO(mtomasz): Pass entries instead.
hirono 2013/12/09 06:53:50 nit: The TODO has already done!
mtomasz 2013/12/10 02:37:34 Yay! Done.
- var allUrls = util.entriesToURLs(allEntries);
- var urls = util.entriesToURLs(entries);
galleryFrame.contentWindow.Gallery.open(
- context, fm.volumeManager_, allUrls, urls);
+ context, fm.volumeManager_, allEntries, entries);
}.bind(this);
galleryFrame.src = 'gallery.html';

Powered by Google App Engine
This is Rietveld 408576698