Polish the SystemInfoProvider template code and refactor StorageInfoProvider based on it

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

Index: chrome/browser/extensions/api/system_info_storage/storage_info_provider_linux.cc
diff --git a/chrome/browser/extensions/api/system_info_storage/storage_info_provider_linux.cc b/chrome/browser/extensions/api/system_info_storage/storage_info_provider_linux.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4862b41801729dae2931e66a2d08e5a8e131f814
--- /dev/null
+++ b/chrome/browser/extensions/api/system_info_storage/storage_info_provider_linux.cc
@@ -0,0 +1,40 @@
+// 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"
+
+namespace extensions {
+
+namespace {
+
+using api::experimental_system_info_storage::StorageInfo;
+using api::experimental_system_info_storage::StorageUnitInfo;
+
+// StorageInfoProvider implementation on Linux platform.
+// TODO(hongbo): not implemented yet.
+class StorageInfoProviderLinux : public StorageInfoProvider {
+ public:
+  StorageInfoProviderLinux() {}
+  virtual ~StorageInfoProviderLinux() {}
+
+  virtual bool QueryInfo(StorageInfo* info) OVERRIDE;
+  virtual bool QueryUnitInfo(StorageUnitInfo* info) OVERRIDE;
+};
+
+bool StorageInfoProviderLinux::QueryInfo(StorageInfo* info) {
+  return false;
+}
+
+bool StorageInfoProviderLinux::QueryUnitInfo(StorageUnitInfo* info) {

[Mihai Parparita -not on Chrome, 2012/08/24 00:57:29]

In the header file this was declared as taking two params (applies to Mac
implementation too).

+  return false;
+}
+
+}   // namespace
+
+// static
+StorageInfoProvider* StorageInfoProvider::Get() {
+  return StorageInfoProvider::GetInstance<StorageInfoProviderLinux>();
+}
+
+}  // namespace extensions