[fsp] Remove the unused streaming field.

The field is not used, so it should be removed.

NOTRY=True
TEST=Tested manually that Files app works correctly.
BUG=408017

Review URL: https://codereview.chromium.org/522533003

Cr-Commit-Position: refs/heads/master@{#292846}

chrome/browser/chromeos/extensions/file_manager/private_api_drive.cc

@@ -100,6 +100,24 @@ void FillDriveEntryPropertiesValue(const drive::ResourceEntry& entry_proto,
       new bool(file_specific_info.cache_state().is_pinned()));
   properties->is_present.reset(
       new bool(file_specific_info.cache_state().is_present()));
+
+  if (file_specific_info.cache_state().is_present()) {
+    properties->is_available_offline.reset(new bool(true));
+  } else if (file_specific_info.is_hosted_document() &&
+             file_specific_info.has_document_extension()) {
+    const std::string file_extension = file_specific_info.document_extension();
+    // What's available offline? See the 'Web' column at:
+    // http://support.google.com/drive/answer/1628467
+    properties->is_available_offline.reset(
+        new bool(file_extension == ".gdoc" || file_extension == ".gdraw" ||
+                 file_extension == ".gsheet" || file_extension == ".gslides"));
+  } else {
+    properties->is_available_offline.reset(new bool(false));
+  }
+
+  properties->is_available_when_metered.reset(
+      new bool(file_specific_info.cache_state().is_present() ||
+               file_specific_info.is_hosted_document()));
 }
 
 // Creates entry definition list for (metadata) search result info list.

chrome/common/extensions/api/file_browser_private.idl

@@ -202,6 +202,12 @@ dictionary DriveEntryProperties {
   // True if the file is hosted on a Drive server instead of local.
   boolean? isHosted;
 
+  // True if the file is available offline.
+  boolean? isAvailableOffline;
+
+  // True if the file is available on metered connection.
+  boolean? isAvailableWhenMetered;
+
   // URL to the custom icon for this file.
   DOMString? customIconUrl;
 

ui/file_manager/file_manager/audio_player/js/audio_player.js

@@ -230,7 +230,7 @@ AudioPlayer.prototype.select_ = function(newTrack) {
  * @private
  */
 AudioPlayer.prototype.fetchMetadata_ = function(entry, callback) {
-  this.metadataCache_.getOne(entry, 'thumbnail|media|streaming',
+  this.metadataCache_.getOne(entry, 'thumbnail|media|drive',
       function(generation, metadata) {
         // Do nothing if another load happened since the metadata request.
         if (this.playlistGeneration_ == generation)
@@ -250,7 +250,7 @@ AudioPlayer.prototype.onError_ = function() {
   this.fetchMetadata_(
       this.entries_[track],
       function(metadata) {
-        var error = (!navigator.onLine && metadata.streaming) ?
+        var error = (!navigator.onLine && !metadata.drive.present) ?
             this.offlineString_ : this.errorString_;
         this.displayMetadata_(track, metadata, error);
         this.scheduleAutoAdvance_();

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';
 };
 
 /**
@@ -852,36 +850,6 @@ DriveProvider.prototype.callApi_ = function() {
       });
 };
 
-/**
- * @param {DriveEntryProperties} data Drive entry properties.
- * @param {Entry} entry File entry.
- * @return {boolean} True if the file is available offline.
- */
-DriveProvider.isAvailableOffline = function(data, entry) {
-  if (data.isPresent)
-    return true;
-
-  if (!data.isHosted)
-    return false;
-
-  // What's available offline? See the 'Web' column at:
-  // http://support.google.com/drive/answer/1628467
-  var subtype = FileType.getType(entry).subtype;
-  return (subtype === 'doc' ||
-          subtype === 'draw' ||
-          subtype === 'sheet' ||
-          subtype === 'slides');
-};
-
-/**
- * @param {DriveEntryProperties} data Drive entry properties.
- * @return {boolean} True if opening the file does not require downloading it
- *    via a metered connection.
- */
-DriveProvider.isAvailableWhenMetered = function(data) {
-  return data.isPresent || data.isHosted;
-};
-
 /**
  * Converts API metadata to internal format.
  * @param {Object} data Metadata from API call.
@@ -898,8 +866,8 @@ DriveProvider.prototype.convert_ = function(data, entry) {
     imageWidth: data.imageWidth,
     imageHeight: data.imageHeight,
     imageRotation: data.imageRotation,
-    availableOffline: DriveProvider.isAvailableOffline(data, entry),
-    availableWhenMetered: DriveProvider.isAvailableWhenMetered(data),
+    availableOffline: data.isAvailableOffline,
+    availableWhenMetered: data.isAvailableWhenMetered,
     customIconUrl: data.customIconUrl || '',
     contentMimeType: data.contentMimeType || '',
     sharedWithMe: data.sharedWithMe,
@@ -919,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;
 };

ui/file_manager/gallery/js/gallery.js

@@ -254,7 +254,7 @@ Gallery.MOSAIC_BACKGROUND_INIT_DELAY = 1000;
  * @const
  * @type {string}
  */
-Gallery.METADATA_TYPE = 'thumbnail|filesystem|media|streaming|drive';
+Gallery.METADATA_TYPE = 'thumbnail|filesystem|media|drive';
 
 /**
  * Initializes listeners.

ui/file_manager/gallery/js/image_editor/image_view.js

@@ -381,8 +381,7 @@ ImageView.prototype.load =
         loadType, ImageView.LOAD_TYPE_TOTAL);
 
     if (loadType === ImageView.LOAD_TYPE_ERROR &&
-        !navigator.onLine && metadata.streaming) {
-      // |streaming| is set only when the file is not locally cached.
+        !navigator.onLine && !metadata.drive.present) {
       loadType = ImageView.LOAD_TYPE_OFFLINE;
     }
     if (loadCallback) loadCallback(loadType, animationDuration, opt_error);