[SystemInfo API] Move Storage API out of experimental namespace and rename to the "system" namespace

chrome/common/extensions/api/experimental_system_info_storage.idl

 // Copyright (c) 2012 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.
 
 // Use the <code>chrome.experimental.systemInfo.storage</code> API to query
 // storage device information and be notified when it changes.
 namespace experimental.systemInfo.storage {
 
   enum StorageUnitType {

[Hongbo Min, 2013/07/08 06:52:43]

nit: Now we can remove StorageUnitType definition.

[Haojian Wu, 2013/07/08 07:46:30]

Done.

     // The storage has fixed media, e.g. hard disk or SSD.
     fixed,
     // The storage is removable, e.g. USB flash drive.
     removable,
     // The storage type is unknown.
     unknown
   };
 
-  dictionary StorageUnitInfo {
-    // The unique storage id. It will use the transient ID.
-    DOMString id;
-    // The name of the storage unit.
-    DOMString name;
-    // The media type of the storage unit.
-    StorageUnitType type;
-    // The total amount of the storage space, in bytes.
-    // Default value is 0 if query operation fails.
-    double capacity;
-    // The available amount of the storage space, in bytes.
-    // Default value is 0 if query operation fails.
-    double availableCapacity;
-  };
-
   dictionary StorageFreeSpaceChangeInfo {
     // The transient id of the storage unit already changed.
     DOMString id;
     // The new value of the available capacity.
     double availableCapacity;
   };
 
   // A dictionary that describes the add particular storage device watch
   // request results.
   dictionary AddWatchResult {
     DOMString id;
     boolean success;
   };
 
-  callback StorageInfoCallback = void (StorageUnitInfo[] info);
-
   callback AddWatchCallback = void (AddWatchResult info);
 
   callback GetAllWatchCallback = void (DOMString[] storageIds);
 
   interface Functions {
-    // Get the storage information from the system. The argument passed to the
-    // callback is an array of StorageUnitInfo objects.
-    static void get(StorageInfoCallback callback);
-
     // Monitor a particular storage device available change capacity.
     static void addWatch(DOMString id, AddWatchCallback callback);
 
     // Remove the monitor of a particular device.
     static void removeWatch(DOMString id);
 
     // Get all the watched storage devices.
     static void getAllWatch(GetAllWatchCallback callback);
 
     // Remove all the storage devices monitors.
     static void removeAllWatch();
   };
 
   interface Events {
     // Fired when the storage device available capacity is changed.
     // |info|: The changed information for the specified storage unit.
     static void onAvailableCapacityChanged(StorageFreeSpaceChangeInfo info);

[Hongbo Min, 2013/07/08 06:52:43]

Maybe here we need to add 'getAvailableCapacity' function and leave it in
experimental namespace. It should be fine to do this in another CL.

[Haojian Wu, 2013/07/08 07:46:30]

Yes. My plan is to implement the 'getAvailableCapacity' function in a new CL.

-
-    // Fired when a new removable storage is attached to the system.
-    static void onAttached(StorageUnitInfo info);
-
-    // Fired when a removable storage is detached from the system.
-    static void onDetached(DOMString id);
   };
-
 };