Add systemInfo.cpu.onUpdated event implementation.

chrome/browser/extensions/system_info_event_router.h

 // 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_SYSTEM_INFO_EVENT_ROUTER_H_
 #define CHROME_BROWSER_EXTENSIONS_SYSTEM_INFO_EVENT_ROUTER_H_
 
 #include <set>
 
 #include "base/file_path.h"
 #include "base/memory/singleton.h"
 #include "base/values.h"
 
 namespace extensions {
 
+namespace api {
+namespace experimental_system_info_cpu {

[benwells, 2012/09/21 08:19:50]

Nit: extra blank lines after each namespace statement and before closing
namespace brace.

[Hongbo Min, 2012/09/21 09:00:23]

Done.

+struct CpuUpdateInfo;
+}
+}
+
 // Event router for systemInfo API. It is a singleton instance shared by
 // multiple profiles.
 // TODO(hongbo): It should derive from SystemMonitor::DevicesChangedObserver.
 // Since the system_monitor will be refactored along with media_gallery, once
 // http://crbug.com/145400 is fixed, we need to update SystemInfoEventRouter
 // accordingly.
 class SystemInfoEventRouter {
  public:
   static SystemInfoEventRouter* GetInstance();
 
@@ skipping 13 matching lines @@
                                   const string16& name,
                                   const FilePath::StringType& location);
   void OnRemovableStorageDetached(const std::string& id);
 
  private:
   friend struct DefaultSingletonTraits<SystemInfoEventRouter>;
 
   SystemInfoEventRouter();
   virtual ~SystemInfoEventRouter();
 
-  // Called on the UI thread to dispatch the systemInfo event to all extension
+  // Called from any thread to dispatch the systemInfo event to all extension
   // processes cross multiple profiles.
   void DispatchEvent(const std::string& event_name,
       scoped_ptr<base::ListValue> args);
 
+  // The callback for CPU sampling cycle. Can be called from any thread.

[benwells, 2012/09/21 08:19:50]

Is this just called from the FILE thread now?

[Hongbo Min, 2012/09/21 09:00:23]

Yes.

[benwells, 2012/09/25 17:32:13]

Can you updated the comment then?

+  void OnNextCpuSampling(
+      scoped_ptr<api::experimental_system_info_cpu::CpuUpdateInfo> info);
+
   // Used to record the event names being watched.
   std::multiset<std::string> watching_event_set_;
 
   DISALLOW_COPY_AND_ASSIGN(SystemInfoEventRouter);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_SYSTEM_INFO_EVENT_ROUTER_H_