Show the eject button only for removabled and file handlers.

chrome/browser/chromeos/file_system_provider/service.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/file_system_provider/service.h"
 
 #include "base/files/file_path.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/scoped_user_pref_update.h"
 #include "base/stl_util.h"
@@ skipping 142 matching lines @@
           storage::FileSystemMountOption(
               storage::FlushPolicy::FLUSH_ON_COMPLETION),
           mount_path)) {
     FOR_EACH_OBSERVER(
         Observer, observers_,
         OnProvidedFileSystemMount(ProvidedFileSystemInfo(), context,
                                   base::File::FILE_ERROR_INVALID_OPERATION));
     return base::File::FILE_ERROR_INVALID_OPERATION;
   }
 
+  ProvidingExtensionInfo provider_info;
+  const bool result = GetProvidingExtensionInfo(extension_id, &provider_info);
+  DCHECK(result);
   // Store the file system descriptor. Use the mount point name as the file
   // system provider file system id.
   // Examples:
   //   file_system_id = hello_world
   //   mount_point_name =  b33f1337-hello_world-5aa5
   //   writable = false
   //   supports_notify_tag = false
   //   mount_path = /provided/b33f1337-hello_world-5aa5
-  ProvidedFileSystemInfo file_system_info(extension_id, options, mount_path);
+  //   configurable = true
+  //   source = SOURCE_FILE
+  ProvidedFileSystemInfo file_system_info(
+      extension_id, options, mount_path,
+      provider_info.capabilities.configurable(),
+      provider_info.capabilities.source());
 
   ProvidedFileSystemInterface* file_system =
       file_system_factory_.Run(profile_, file_system_info);
   DCHECK(file_system);
   file_system_map_[FileSystemKey(extension_id, options.file_system_id)] =
       file_system;
   mount_point_name_to_key_map_[mount_point_name] =
       FileSystemKey(extension_id, options.file_system_id);
   registry_->RememberFileSystem(file_system_info, *file_system->GetWatchers());
 
@@ skipping 121 matching lines @@
 }
 
 std::vector<ProvidingExtensionInfo> Service::GetProvidingExtensionInfoList()
     const {
   extensions::ExtensionRegistry* const registry =
       extensions::ExtensionRegistry::Get(profile_);
   DCHECK(registry);
 
   std::vector<ProvidingExtensionInfo> result;
   for (const auto& extension : registry->enabled_extensions()) {
-    if (!extension->permissions_data()->HasAPIPermission(
-            extensions::APIPermission::kFileSystemProvider)) {
-      continue;
-    }
-
     ProvidingExtensionInfo info;
-    info.extension_id = extension->id();
-    info.name = extension->name();
-    const extensions::FileSystemProviderCapabilities* const capabilities =
-        extensions::FileSystemProviderCapabilities::Get(extension.get());
-    DCHECK(capabilities);
-    info.capabilities = *capabilities;
-    result.push_back(info);
+    if (GetProvidingExtensionInfo(extension->id(), &info))
+      result.push_back(info);
   }
 
   return result;
 }
 
+bool Service::GetProvidingExtensionInfo(const std::string& extension_id,
+                                        ProvidingExtensionInfo* result) const {
+  DCHECK(result);
+  extensions::ExtensionRegistry* const registry =
+      extensions::ExtensionRegistry::Get(profile_);
+  DCHECK(registry);
+
+  const extensions::Extension* const extension = registry->GetExtensionById(
+      extension_id, extensions::ExtensionRegistry::ENABLED);
+  if (!extension ||
+      !extension->permissions_data()->HasAPIPermission(
+          extensions::APIPermission::kFileSystemProvider)) {
+    return false;
+    }

[hirono, 2015/05/14 10:43:09]

Please fix the indent.

[mtomasz, 2015/05/15 02:03:37]

Done.

+
+    result->extension_id = extension->id();
+    result->name = extension->name();
+    const extensions::FileSystemProviderCapabilities* const capabilities =
+        extensions::FileSystemProviderCapabilities::Get(extension);
+    DCHECK(capabilities);
+    result->capabilities = *capabilities;
+
+    return true;
+}
+
 void Service::OnExtensionUnloaded(
     content::BrowserContext* browser_context,
     const extensions::Extension* extension,
     extensions::UnloadedExtensionInfo::Reason reason) {
   // Unmount all of the provided file systems associated with this extension.
   ProvidedFileSystemMap::iterator it = file_system_map_.begin();
   while (it != file_system_map_.end()) {
     const ProvidedFileSystemInfo& file_system_info =
         it->second->GetFileSystemInfo();
     // Advance the iterator beforehand, otherwise it will become invalidated
@@ skipping 88 matching lines @@
 }
 
 void Service::OnWatcherListChanged(
     const ProvidedFileSystemInfo& file_system_info,
     const Watchers& watchers) {
   registry_->RememberFileSystem(file_system_info, watchers);
 }
 
 }  // namespace file_system_provider
 }  // namespace chromeos