GCM: D-Bus methods for wake-on-packet

chrome/browser/chromeos/chrome_browser_main_chromeos.h

Index: chrome/browser/chromeos/chrome_browser_main_chromeos.h
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.h b/chrome/browser/chromeos/chrome_browser_main_chromeos.h
index cd3b3673ade96fc5b21b81269524fdedb5f8796b..e7bcdfbcac13f6cf557fb9b53974f357ea252fe0 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.h
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.h
@@ -10,6 +10,7 @@
 #include "chrome/browser/chrome_browser_main_linux.h"
 #include "chrome/browser/chromeos/external_metrics.h"
 #include "chrome/browser/chromeos/version_loader.h"
+#include "net/base/ip_endpoint.h"

[Daniel Erat, 2014/08/21 22:17:37]

nit: you don't need to include this in the header; forward-declare
net::IPEndPoint instead

 
 namespace content {
 class PowerSaveBlocker;
@@ -59,6 +60,11 @@ class ChromeBrowserMainPartsChromeos : public ChromeBrowserMainPartsLinux {
   virtual void PostDestroyThreads() OVERRIDE;
 
  private:
+  static void OnGCMConnected(const net::IPEndPoint& ip_endpoint);
+  static void OnGCMDisconnected();
+  static void GCMErrorCallback(const std::string &error_name,

[Daniel Erat, 2014/08/21 22:17:37]

nit: '&' should go to the left of the space on this line and the next

[Luigi Semenzato, 2014/08/22 01:08:19]

Done.

+                               const std::string &error);
+
   scoped_ptr<default_app_order::ExternalLoader> app_order_loader_;
   scoped_ptr<ExtensionSystemEventObserver> extension_system_event_observer_;
   scoped_ptr<PeripheralBatteryObserver> peripheral_battery_observer_;