[SystemInfo API] Implement storage.getAvailableCapacity API

chrome/common/extensions/api/system_storage.idl

 // Copyright 2013 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.system.storage</code> API to query storage device
 // information and be notified when a removable storage device is attached and
 // detached.
 namespace system.storage {
 
   enum StorageUnitType {
@@ skipping 12 matching lines @@
     // application, or between different applications.
     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.
     double capacity;
   };
 
+  dictionary StorageAvailableCapacityInfo {
+    DOMString id;

[Greg Billock, 2013/10/14 20:32:05]

Can we arrange to pass this in JS and just return the capacity in the callback?

[Haojian Wu, 2013/10/15 00:50:26]

Yes. We can just return the capacity instead of this structure.

But we have discussed with Hongbo in
CL:https://codereview.chromium.org/19053003/(see comments)

I think returning the capacity with corresponding storage device |id| in
callback function is better. Without the |id| field, developers should do extra
work to remember which storage device |id| does the |availableCapcity| belong
to, right?

[Greg Billock, 2013/10/15 15:26:04]

Sounds good.

On 2013/10/15 00:50:26, Haojian Wu wrote:

[Greg Billock, 2013/10/15 15:26:04]

Add a comment about this id that it is the same ID from StorageUnitInfo.

[Haojian Wu, 2013/10/16 04:04:23]

Done.

+    // The available capacity of the storage device, in bytes.
+    double availableCapacity;
+  };
+
   [inline_doc] enum EjectDeviceResultCode {
     // The ejection command is successful -- the application can prompt the user
     // to remove the device.
     success,
     // The device is in use by another application. The ejection did not
     // succeed; the user should not remove the device until the other
     // application is done with the device.
     in_use,
     // There is no such device known.
     no_such_device,
     // The ejection command failed.
     failure
   };
 
   callback EjectDeviceCallback = void (EjectDeviceResultCode result);
 
   callback StorageInfoCallback = void (StorageUnitInfo[] info);
 
+  callback getAvailableCapacityCallback = void (
+      StorageAvailableCapacityInfo info);
+
   interface Functions {
     // Get the storage information from the system. The argument passed to the
     // callback is an array of StorageUnitInfo objects.
     static void getInfo(StorageInfoCallback callback);
 
     // Ejects a removable storage device.
     static void ejectDevice(DOMString id, EjectDeviceCallback callback);
+
+    // Get the available capacity of a specified |id| storage device.

[Greg Billock, 2013/10/15 15:26:04]

Clarify here about id: "... of a specified storage device. The |id| is the
transient device ID from StorageUnitInfo."

[Haojian Wu, 2013/10/16 04:04:23]

Done.

+    static void getAvailableCapacity(DOMString id,
+                                     getAvailableCapacityCallback callback);
   };
 
   interface Events {
     // 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);
   };
 
 };
-