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

Unified Diff: ui/file_manager/file_manager/foreground/js/ui/file_grid.js

Issue 831833004: Files.app: Stop to use drive thumbnail for cached files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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/file_manager/foreground/js/ui/file_grid.js
diff --git a/ui/file_manager/file_manager/foreground/js/ui/file_grid.js b/ui/file_manager/file_manager/foreground/js/ui/file_grid.js
index 8c0983325b62fc585cac77be424be0af3f0611b3..e2f499da057a396ad3628b0e64a9a503dc79b1ca 100644
--- a/ui/file_manager/file_manager/foreground/js/ui/file_grid.js
+++ b/ui/file_manager/file_manager/foreground/js/ui/file_grid.js
@@ -243,16 +243,20 @@ FileGrid.decorateThumbnailBox = function(
// Drive provides high quality thumbnails via USE_EMBEDDED, however local
mtomasz 2015/01/09 05:58:50 nit: Please update the comment.
hirono 2015/01/09 07:11:47 Done.
// images usually provide very tiny thumbnails, therefore USE_EMBEDDE can't
// be used to obtain high quality output.
- var useEmbedded;
+ var loadTargets;
switch (quality) {
case FileGrid.ThumbnailQuality.LOW:
- useEmbedded = ThumbnailLoader.UseEmbedded.USE_EMBEDDED;
+ loadTargets = [
+ ThumbnailLoader.LoadTarget.CONTENT_METADATA,
+ ThumbnailLoader.LoadTarget.EXTERNAL_METADATA,
+ ThumbnailLoader.LoadTarget.FILE_ENTRY
+ ];
break;
case FileGrid.ThumbnailQuality.HIGH:
- // TODO(mtomasz): Use Entry instead of paths.
- useEmbedded = (locationInfo && locationInfo.isDriveBased) ?
- ThumbnailLoader.UseEmbedded.USE_EMBEDDED :
- ThumbnailLoader.UseEmbedded.NO_EMBEDDED;
+ loadTargets = [
+ ThumbnailLoader.LoadTarget.EXTERNAL_METADATA,
+ ThumbnailLoader.LoadTarget.FILE_ENTRY
+ ];
break;
}
@@ -262,7 +266,7 @@ FileGrid.decorateThumbnailBox = function(
ThumbnailLoader.LoaderType.IMAGE,
metadata,
undefined, // opt_mediaType
- useEmbedded).
+ loadTargets).
load(box,
fillMode,
ThumbnailLoader.OptimizationMode.DISCARD_DETACHED,

Powered by Google App Engine
This is Rietveld 408576698