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

Unified Diff: ui/file_manager/gallery/js/image_editor/image_util.js

Issue 310433004: Gallery.app: Update thumbnail when a file is edited on the photo editor. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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: ui/file_manager/gallery/js/image_editor/image_util.js
diff --git a/ui/file_manager/gallery/js/image_editor/image_util.js b/ui/file_manager/gallery/js/image_editor/image_util.js
index bcd5696a03c51b0cd46e3fac207333bfdbb092e4..04b74c713d08b2deed3ae743d1e190d88d588445 100644
--- a/ui/file_manager/gallery/js/image_editor/image_util.js
+++ b/ui/file_manager/gallery/js/image_editor/image_util.js
@@ -489,14 +489,9 @@ ImageUtil.ImageLoader.prototype.load = function(
// general error should not be specified
this.image_.onerror = onError.bind(this, 'GALLERY_IMAGE_ERROR');
- // Extract the last modification date to determine if the cached image
- // is outdated.
- var modificationTime = opt_metadata &&
- opt_metadata.modificationTime &&
- opt_metadata.modificationTime.getTime();
-
- // Load the image directly.
- this.image_.src = entry.toURL();
+ // Load the image directly. The query parameter is needed to obtain the
+ // latest image just after editing it. crbug.com/378648
+ this.image_.src = entry.toURL() + "?nocache=" + Date.now();
mtomasz 2014/06/02 06:40:28 Does it happen on Downloads too? If so, then it se
hirono 2014/06/02 08:59:15 Yes it happens to Downloads too. Filed: crbug.com/
mtomasz 2014/06/03 05:33:19 Please write a comment that it is a workaround, an
hirono 2014/06/03 07:51:45 Done.
}.bind(this);
// Loads the image. If already loaded, then forces a reload.

Powered by Google App Engine
This is Rietveld 408576698