[SystemInfo API] Rewrite storage info provider using storage monitor impl

chrome/browser/storage_monitor/storage_monitor.h

Index: chrome/browser/storage_monitor/storage_monitor.h
diff --git a/chrome/browser/storage_monitor/storage_monitor.h b/chrome/browser/storage_monitor/storage_monitor.h
index 7eab0b7743b925d9b3fffa009623e946c32b496d..1965aef0ce625bc0d306dac00dd1407e68e992b0 100644
--- a/chrome/browser/storage_monitor/storage_monitor.h
+++ b/chrome/browser/storage_monitor/storage_monitor.h
@@ -73,6 +73,11 @@ class StorageMonitor {
   // Should be invoked on the UI thread; callbacks will be run on the UI thread.
   void Initialize(base::Closure callback);
 
+  // Register a callback to be called when the initialization is completed. If
+  // the initialization is already completed, the provided callback will be
+  // invoked with the calling stack.
+  void RegisterInitCompletedCallback(base::Closure callback);
+
   // Return true if the storage monitor has already been initialized.
   bool IsInitialized();
 
@@ -102,6 +107,11 @@ class StorageMonitor {
       media_transfer_protocol_manager() = 0;
 #endif
 
+  // Returns information for all available storages on the system. It may
+  // return a empty vector if the storage metadata retrieval is not completed
+  // yet on FILE thread.
+  virtual std::vector<StorageInfo> GetAllAvailableStorages() const;

[Lei Zhang, 2013/05/28 08:08:07]

How is this different from GetAttachedStorage() below? Did you want this to
return all storages, removable or fixed? If so, you should check all the
implementations. The CrOS StorageMonitor, for instance, currently only handles
removable devices.

[Hongbo Min, 2013/05/28 14:24:01]

Yes, GetAllAvailableStorages is designed to return the fixed and removabled
storages on the system, while GetAttachedStorage only returns the removable
storages.

[Lei Zhang, 2013/05/28 21:16:44]

I would check and make sure all the per-platform implementations are actually
returning all the storages and not just the removable ones.

+
   // Returns information for attached removable storage.
   std::vector<StorageInfo> GetAttachedStorage() const;