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

Unified Diff: ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js

Issue 522533003: [fsp] Remove the unused streaming field. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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/metadata/metadata_cache.js
diff --git a/ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js b/ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js
index 4da1126bf56e5529d8510ab59c1a925b767540b1..92c9aec757daff1557884dce693f7fda07a5eaec 100644
--- a/ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js
+++ b/ui/file_manager/file_manager/foreground/js/metadata/metadata_cache.js
@@ -13,7 +13,6 @@
* filesystem: size, modificationTime
* internal: presence
* drive: pinned, present, hosted, availableOffline
- * streaming: (no property)
*
* Following are not fetched for non-present drive files.
* media: artist, album, title, width, height, imageTransform, etc.
@@ -763,7 +762,6 @@ FilesystemProvider.prototype.fetch = function(
* This provider returns the following objects:
* drive: { pinned, hosted, present, customIconUrl, etc. }
* thumbnail: { url, transform }
- * streaming: { }
* @param {VolumeManagerWrapper} volumeManager Volume manager instance.
* @constructor
*/
@@ -803,7 +801,7 @@ DriveProvider.prototype.supportsEntry = function(entry) {
*/
DriveProvider.prototype.providesType = function(type) {
return type === 'drive' || type === 'thumbnail' ||
- type === 'streaming' || type === 'media' || type === 'filesystem';
+ type === 'media' || type === 'filesystem';
};
/**
@@ -889,15 +887,12 @@ DriveProvider.prototype.convert_ = function(data, entry) {
} else if (data.isPresent) {
result.thumbnail = null;
} else {
- // Block the local fetch for drive files, which require downloading.
+ // Not present in cache, so do not allow to generate it by next providers.
result.thumbnail = {url: '', transform: null};
}
+ // If present in cache, then allow to fetch media by next providers.
result.media = data.isPresent ? null : {};
- // Indicate that the data is not available in local cache.
- // It used to have a field 'url' for streaming play, but it is
- // derprecated. See crbug.com/174560.
- result.streaming = data.isPresent ? null : {};
return result;
};

Powered by Google App Engine
This is Rietveld 408576698