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

chrome/browser/extensions/extension_gcm_app_handler_unittest.cc

Index: chrome/browser/extensions/extension_gcm_app_handler_unittest.cc
diff --git a/chrome/browser/extensions/extension_gcm_app_handler_unittest.cc b/chrome/browser/extensions/extension_gcm_app_handler_unittest.cc
index cee0ea9592b579db6b5f577d5dd812d5b3af1045..b327824a8ffc11a2a91266d2a44a89a78699a5ed 100644
--- a/chrome/browser/extensions/extension_gcm_app_handler_unittest.cc
+++ b/chrome/browser/extensions/extension_gcm_app_handler_unittest.cc
@@ -56,6 +56,7 @@
 #include "chrome/browser/chromeos/login/users/scoped_test_user_manager.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/settings/device_settings_service.h"
+#include "chromeos/dbus/dbus_thread_manager.h"
 #endif
 
 namespace extensions {
@@ -220,6 +221,8 @@ class ExtensionGCMAppHandlerTest : public testing::Test {
     // This is needed to create extension service under CrOS.
 #if defined(OS_CHROMEOS)
     test_user_manager_.reset(new chromeos::ScopedTestUserManager());
+    // Create a DBus thread manager setter for its side effect.
+    (void) chromeos::DBusThreadManager::GetSetterForTesting();

[Yoyo Zhou, 2014/09/13 00:50:47]

What is this (void) doing here?

[Luigi Semenzato, 2014/09/13 01:02:48]

The function returns a value which I ignore, as I thought was clear from the
comment (obviously it is not).  Should I omit the (void)?

[Yoyo Zhou, 2014/09/13 01:09:43]

The usage looks pretty uncommon in src/chrome, so I would say you can omit it.

 #endif
 
     // Create a new profile.