[SystemInfo API] Update systemInfo.storage API StorageUnitInfo Definition.

[SystemInfo API] Update systemInfo.storage API StorageUnitInfo Definition.

Remove |available_capacity| field.

BUG=252994
TEST=browser_tests --gtest_fitler=SystemInfoStorageApiTest.*
TEST=unit_tests --gtest_filter=StorageInfoProviderTest.*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213368

[Haojian Wu, 2013-07-23 03:17:26]

A new separated CL from the storage API renamed CL
https://chromiumcodereview.appspot.com/18578008/.

Please have a review. Thanks.

[not at google - send to devlin, 2013-07-23 15:39:43]

lgtm but what is the reason for this change? The BUG is just some generic bug
about the API.

[Hongbo Min, 2013-07-24 00:10:40]

On 2013/07/23 15:39:43, kalman wrote:
> lgtm but what is the reason for this change? The BUG is just some generic bug
> about the API.

LGTM. Thanks.

kalman@, maybe you miss some background about systemInfo.storage API changes.
Let me explain it.

As Greg Billock's suggestion, the systemInfo.storage API is planned to be moved
out experimental namespace and placed into 'system' namespace in M30, and
mediaGalleried API will depend on the storage API which provides storage
attachment event notification.
Since this API will become a stable version, we need to carefully define and
refactor the API interface and implementation, for |available_capability| field,
we agreed that at this point we don't have a solid use case for storage
available capability, and from the implementation perspective, the available
capability is queried by a way of polling based solution, which sounds not so
good, since it might always wake up CPU and prevent CPU to enter sleep state.
That is the reason why this change happen.

Did I answer your question?

[not at google - send to devlin, 2013-07-24 00:12:44]

yes thanks. Perhaps file a bug and link to it in the future.

[commit-bot: I haz the power, 2013-07-24 01:03:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/Hokein.Wu@gmail.com/19672022/1

[commit-bot: I haz the power, 2013-07-24 07:30:29]

Change committed as 213368

[not at google - send to devlin, 2013-07-25 14:34:01]

https://chromiumcodereview.appspot.com/19672022/diff/1/chrome/common/extensio...
File chrome/common/extensions/api/experimental_system_info_storage.idl (right):

https://chromiumcodereview.appspot.com/19672022/diff/1/chrome/common/extensio...
chrome/common/extensions/api/experimental_system_info_storage.idl:93: static
void onAvailableCapacityChanged(StorageFreeSpaceChangeInfo info);
what about this event?