Migrate from URLs to Entries in the Files App's gallery.

chrome/browser/resources/file_manager/foreground/js/photo/gallery_item.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 /**
  * Object representing an image item (a photo or a video).
  *
- * @param {string} url Image url.
+ * @param {FileEntry} entry Image entry.
  * @constructor
  */
-Gallery.Item = function(url) {
-  this.url_ = url;
+Gallery.Item = function(entry) {
+  this.entry_ = entry;
   this.original_ = true;
 };
 
 /**
- * @return {string} Image url.
+ * @return {FileEntry} Image entry.
  */
-Gallery.Item.prototype.getUrl = function() { return this.url_ };
-
-/**
- * @param {string} url New url.
- */
-Gallery.Item.prototype.setUrl = function(url) { this.url_ = url };
+Gallery.Item.prototype.getEntry = function() { return this.entry_ };
 
 /**
  * @return {string} File name.
  */
 Gallery.Item.prototype.getFileName = function() {
-  return ImageUtil.getFullNameFromUrl(this.url_);
+  return this.entry_.name;
 };
 
 /**
  * @return {boolean} True if this image has not been created in this session.
  */
 Gallery.Item.prototype.isOriginal = function() { return this.original_ };
 
 // TODO: Localize?
 /**
  * @type {string} Suffix for a edited copy file name.
@@ skipping 81 matching lines @@
  * @param {HTMLCanvasElement} canvas Source canvas.
  * @param {ImageEncoder.MetadataEncoder} metadataEncoder MetadataEncoder.
  * @param {function(boolean)=} opt_callback Callback accepting true for success.
  */
 Gallery.Item.prototype.saveToFile = function(
     overrideDir, overwrite, canvas, metadataEncoder, opt_callback) {
   ImageUtil.metrics.startInterval(ImageUtil.getMetricName('SaveTime'));
 
   var name = this.getFileName();
 
-  var onSuccess = function(url) {
+  var onSuccess = function(entry) {
     ImageUtil.metrics.recordEnum(ImageUtil.getMetricName('SaveResult'), 1, 2);
     ImageUtil.metrics.recordInterval(ImageUtil.getMetricName('SaveTime'));
-    this.setUrl(url);
+    this.entry_ = entry;
     if (opt_callback) opt_callback(true);
   }.bind(this);
 
   function onError(error) {
     console.error('Error saving from gallery', name, error);
     ImageUtil.metrics.recordEnum(ImageUtil.getMetricName('SaveResult'), 0, 2);
     if (opt_callback) opt_callback(false);
   }
 
   function doSave(newFile, fileEntry) {
     fileEntry.createWriter(function(fileWriter) {
       function writeContent() {
-        fileWriter.onwriteend = onSuccess.bind(null, fileEntry.toURL());
+        fileWriter.onwriteend = onSuccess.bind(null, fileEntry);
         fileWriter.write(ImageEncoder.getBlob(canvas, metadataEncoder));
       }
       fileWriter.onerror = function(error) {
         onError(error);
         // Disable all callbacks on the first error.
         fileWriter.onerror = null;
         fileWriter.onwriteend = null;
       };
       if (newFile) {
         writeContent();
@@ skipping 23 matching lines @@
         this.original_ = false;
         name = copyName;
         checkExistence(dir);
       }.bind(this));
     }
   }.bind(this);
 
   if (overrideDir) {
     saveToDir(overrideDir);
   } else {
-    webkitResolveLocalFileSystemURL(this.getUrl(),
-        function(entry) { entry.getParent(saveToDir, onError)},
-        onError);
+    this.entry_.getParent(saveToDir, onError);
   }
 };
 
 /**
  * Rename the file.
  *
- * @param {string} name New file name.
- * @param {function} onSuccess Success callback.
- * @param {function} onExists Called if the file with the new name exists.
+ * @param {string} displayName New display name (without the extension).
+ * @param {function()} onSuccess Success callback.
+ * @param {function()} onExists Called if the file with the new name exists.
  */
-Gallery.Item.prototype.rename = function(name, onSuccess, onExists) {
-  var oldName = this.getFileName();
-  if (ImageUtil.getExtensionFromFullName(name) ==
-      ImageUtil.getExtensionFromFullName(oldName)) {
-    name = ImageUtil.getFileNameFromFullName(name);
-  }
-  var newName = ImageUtil.replaceFileNameInFullName(oldName, name);
-  if (oldName == newName) return;
+Gallery.Item.prototype.rename = function(displayName, onSuccess, onExists) {
+  var fileName = this.entry_.name.replace(
+      ImageUtil.getDisplayNameFromName(this.entry_.name), displayName);
 
-  function onError() {
-    console.error('Rename error: "' + oldName + '" to "' + newName + '"');
-  }
+  if (name == this.entry_.name)

[hirono, 2013/12/10 03:36:53]

=== is preferred.

[mtomasz, 2013/12/10 05:07:01]

Done.

+    return;
 
   var onRenamed = function(entry) {
-    this.setUrl(entry.toURL());
+    this.entry_ = entry;
     onSuccess();
   }.bind(this);
 
-  function moveIfDoesNotExist(entry, parentDir) {
-    parentDir.getFile(newName, {create: false, exclusive: false}, onExists,
-        function() { entry.moveTo(parentDir, newName, onRenamed, onError) });
-  }
+  var onError = function() {
+    console.error('Rename error: "' + oldName + '" to "' + newName + '"');
+  };
 
-  webkitResolveLocalFileSystemURL(this.getUrl(),
-      function(entry) {
-        entry.getParent(moveIfDoesNotExist.bind(null, entry), onError);
-      },
-      onError);
+  var moveIfDoesNotExist = function(parentDir) {
+    parentDir.getFile(fileName, {create: false, exclusive: false}, onExists,

[hirono, 2013/12/10 03:36:53]

Please place each argument per line.

[mtomasz, 2013/12/10 05:07:01]

Done.

+        function() {
+          this.entry_.moveTo(parentDir, fileName, onRenamed, onError);
+        }.bind(this));
+  }.bind(this);
+
+  this.entry_.getParent(moveIfDoesNotExist, onError);
 };