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

Unified Diff: chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc

Issue 574293002: Files.app: Show thumbnail of non-image file even when the file cache is present (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 3 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
« no previous file with comments | « no previous file | ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
diff --git a/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc b/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
index 53acd495ddb6af56cac923094a241d2eaa3a3305..c22e75b52f538f9ab15bccb695aa811c92d558c5 100644
--- a/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
+++ b/chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc
@@ -76,10 +76,7 @@ void FillEntryPropertiesValueForDrive(const drive::ResourceEntry& entry_proto,
const drive::FileSpecificInfo& file_specific_info =
entry_proto.file_specific_info();
- // The web thumbnail is used only when the file is not present. Otherwise
- // it could be out of sync.
- if (!entry_proto.resource_id().empty() &&
- !file_specific_info.cache_state().is_present()) {
+ if (!entry_proto.resource_id().empty()) {
properties->thumbnail_url.reset(
new std::string("https://www.googledrive.com/thumb/" +
entry_proto.resource_id() + "?width=500&height=500"));
« no previous file with comments | « no previous file | ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698