Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: chrome/browser/extensions/api/systeminfo_cpu/systeminfo_cpu_api.h

Issue 10830240: Add the implementation skeleton for systeminfo.cpu API (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Updated patch Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/systeminfo_cpu/systeminfo_cpu_api.h
diff --git a/chrome/browser/extensions/api/systeminfo_cpu/systeminfo_cpu_api.h b/chrome/browser/extensions/api/systeminfo_cpu/systeminfo_cpu_api.h
new file mode 100644
index 0000000000000000000000000000000000000000..1260e9a87d49640665810d0ae1fbdf67c1a0ec29
--- /dev/null
+++ b/chrome/browser/extensions/api/systeminfo_cpu/systeminfo_cpu_api.h
@@ -0,0 +1,44 @@
+// 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.
+#ifndef CHROME_BROWSER_EXTENSIONS_API_SYSTEMINFO_CPU_SYSTEMINFO_CPU_API_H_
+#define CHROME_BROWSER_EXTENSIONS_API_SYSTEMINFO_CPU_SYSTEMINFO_CPU_API_H_
+
+#include "chrome/browser/extensions/extension_function.h"
+
+namespace extensions {
+
+class CpuInfoProvider;
+
+class SysteminfoCpuGetFunction : public AsyncExtensionFunction {
+ public:
+ DECLARE_EXTENSION_FUNCTION_NAME("experimental.systeminfo.cpu.get");
Mihai Parparita -not on Chrome 2012/08/10 20:24:56 Can you capitalize the "info" in "systeminfo" (and
Hongbo Min 2012/08/12 14:22:44 Done.
+ SysteminfoCpuGetFunction();
+ virtual ~SysteminfoCpuGetFunction();
+
+ private:
+ virtual bool RunImpl() OVERRIDE;
+ void WorkOnFileThread();
+ void RespondOnUIThread(bool success);
+ void GetCpuInfoOnFileThread();
+
+ // The CpuInfoProvider instance, lives on FILE thread.
+ CpuInfoProvider* provider_;
+};
+
+class SysteminfoCpuSetUpdateIntervalFunction : public AsyncExtensionFunction {
Mihai Parparita -not on Chrome 2012/08/10 20:24:56 If this is a no-op, can you remove this from the C
Hongbo Min 2012/08/12 14:22:44 Remove setUpdateInterval from cc and idl file. Do
+ public:
+ DECLARE_EXTENSION_FUNCTION_NAME("experimental.systeminfo.cpu.get");
+ SysteminfoCpuSetUpdateIntervalFunction() {}
+ virtual ~SysteminfoCpuSetUpdateIntervalFunction() {}
+
+ private:
+ virtual bool RunImpl() OVERRIDE {
+ // TODO(hmin): not implemented yet
+ return false;
+ }
+};
+
+} // namespace extensions
+
+#endif // CHROME_BROWSER_EXTENSIONS_API_SYSTEMINFO_CPU_SYSTEMINFO_CPU_API_H_

Powered by Google App Engine
This is Rietveld 408576698