Add systeminfo.storage API implementation skeleton

chrome/browser/extensions/api/system_info_storage/storage_info_provider.cc

Index: chrome/browser/extensions/api/system_info_storage/storage_info_provider.cc
diff --git a/chrome/browser/extensions/api/system_info_storage/storage_info_provider.cc b/chrome/browser/extensions/api/system_info_storage/storage_info_provider.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bc8385ac03bf7fedd6c94aa95385c7f06590abf2
--- /dev/null
+++ b/chrome/browser/extensions/api/system_info_storage/storage_info_provider.cc
@@ -0,0 +1,13 @@
+// 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.
+
+#include "chrome/browser/extensions/api/system_info_storage/storage_info_provider.h"

[Mihai Parparita -not on Chrome, 2012/08/14 00:07:31]

For the sake of brevity, you may want to start with all of the system_info APIs
being in the same directory ("system_info"), and only move them out into
separate ones if it gets overwhelming.

The directories are meant to be per-API, and I see all of SystemInfo as being
one API.

[Hongbo Min, 2012/08/14 01:37:23]

If the namespace specified in IDL is systemInfo.X, the generated_api.h generated
from IDL parser assumes the X APIs are in system_info_x directory.

So each SystemInfo api category has to be in a separate directory, not in the
same dir ("system_info").

If the IDL allows to add nested namespaces specified IDL file, it could get
fixed, like
namespace experimental.systemInfo {
  namespace cpu {
  }
  namespace storage {
  }
}

[Hongbo Min, 2012/08/14 04:25:57]

If using JSON format, all systemInfo API could be in the same one directory. But
currently the IDL compiler can't support such usage model.

Is it necessary to use JSON format instead? or fire a bug to IDL compiler?

[Mihai Parparita -not on Chrome, 2012/08/15 00:14:05]

It's not worth switching to the JSON format just for this. You can file a bug.

[Hongbo Min, 2012/08/15 13:43:07]

The bug is filed at http://crbug.com/142841

+
+namespace extensions {
+namespace systeminfo {
+const char kStorageTypeUnknown[] = "unknown";
+const char kStorageTypeHardDisk[] = "harddisk";
+const char kStorageTypeRemovable[] = "removable";
+}  // namespace systeminfo
+}  // namespace extensions