Clean Up: Gather all ibus constants to ibus_constants.h

BUG=None
TEST=ran chromeos_unittests


Review URL: https://chromiumcodereview.appspot.com/10704118

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146052 0039d316-1c4b-4281-b951-d872f2087c98

chromeos/dbus/ibus/ibus_client.cc

@@ -16,14 +16,13 @@
 namespace chromeos {
 
 namespace {
-const char kIBusBusRegisterComponentMethod[] = "RegisterComponent";
 
 // The IBusClient implementation.
 class IBusClientImpl : public IBusClient {
  public:
   explicit IBusClientImpl(dbus::Bus* bus)
-      : proxy_(bus->GetObjectProxy(kIBusServiceName,
-                                   dbus::ObjectPath(kIBusServicePath))),
+      : proxy_(bus->GetObjectProxy(ibus::kServiceName,
+                                   dbus::ObjectPath(ibus::bus::kServicePath))),
         weak_ptr_factory_(this) {
   }
 
@@ -36,8 +35,8 @@ class IBusClientImpl : public IBusClient {
       const ErrorCallback& error_callback) OVERRIDE {
     DCHECK(!callback.is_null());
     DCHECK(!error_callback.is_null());
-    dbus::MethodCall method_call(kIBusServiceInterface,
-                                 kIBusBusCreateInputContextMethod);
+    dbus::MethodCall method_call(ibus::bus::kServiceInterface,
+                                 ibus::bus::kCreateInputContextMethod);
     dbus::MessageWriter writer(&method_call);
     writer.AppendString(client_name);
     proxy_->CallMethodWithErrorCallback(
@@ -59,8 +58,8 @@ class IBusClientImpl : public IBusClient {
       const ErrorCallback& error_callback) OVERRIDE {
     DCHECK(!callback.is_null());
     DCHECK(!error_callback.is_null());
-    dbus::MethodCall method_call(kIBusServiceInterface,
-                                 kIBusBusRegisterComponentMethod);
+    dbus::MethodCall method_call(ibus::bus::kServiceInterface,
+                                 ibus::bus::kRegisterComponentMethod);
     dbus::MessageWriter writer(&method_call);
     ibus::AppendIBusComponent(ibus_component, &writer);
     proxy_->CallMethodWithErrorCallback(

chromeos/dbus/ibus/ibus_client_unittest.cc

@@ -76,10 +76,12 @@ class IBusClientTest : public testing::Test {
     dbus::Bus::Options options;
     mock_bus_ = new dbus::MockBus(options);
     mock_proxy_ = new dbus::MockObjectProxy(mock_bus_.get(),
-                                            kIBusServiceName,
-                                            dbus::ObjectPath(kIBusServicePath));
-    EXPECT_CALL(*mock_bus_, GetObjectProxy(kIBusServiceName,
-                                           dbus::ObjectPath(kIBusServicePath)))
+                                            ibus::kServiceName,
+                                            dbus::ObjectPath(
+                                                ibus::bus::kServicePath));
+    EXPECT_CALL(*mock_bus_, GetObjectProxy(ibus::kServiceName,
+                                           dbus::ObjectPath(
+                                               ibus::bus::kServicePath)))
         .WillOnce(Return(mock_proxy_.get()));
 
     EXPECT_CALL(*mock_bus_, ShutdownAndBlock());

chromeos/dbus/ibus/ibus_constants.h

@@ -2,17 +2,38 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROMEOS_DBUS_IBUS_IBUS_CONST_H_
-#define CHROMEOS_DBUS_IBUS_IBUS_CONST_H_
+#ifndef CHROMEOS_DBUS_IBUS_IBUS_CONSTANTS_H_
+#define CHROMEOS_DBUS_IBUS_IBUS_CONSTANTS_H_
 
 namespace chromeos {
 
-const char kIBusServiceName[] = "org.freedesktop.IBus";
-const char kIBusServicePath[] = "/org/freedesktop/IBus";
-const char kIBusServiceInterface[] = "org.freedesktop.IBus";
+namespace ibus {
 
-const char kIBusBusCreateInputContextMethod[] = "CreateInputContext";
+const char kServiceName[] = "org.freedesktop.IBus";
 
+namespace bus {
+const char kServicePath[] = "/org/freedesktop/IBus";
+const char kServiceInterface[] = "org.freedesktop.IBus";
+const char kCreateInputContextMethod[] = "CreateInputContext";
+const char kRegisterComponentMethod[] = "RegisterComponent";
+}  // namespace bus
+
+namespace input_context {
+const char kServiceInterface[] = "org.freedesktop.IBus.InputContext";
+const char kCommitTextSignal[] = "CommitText";
+const char kForwardKeyEventSignal[] = "ForwardKeyEvent";
+const char kHidePreeditTextSignal[] = "HidePreeditText";
+const char kShowPreeditTextSignal[] = "ShowPreeditText";
+const char kUpdatePreeditTextSignal[] = "UpdatePreeditText";
+const char kFocusInMethod[] = "FocusIn";
+const char kFocusOutMethod[] = "FocusOut";
+const char kResetMethod[] = "Reset";
+const char kSetCapabilitiesMethod[] = "SetCapabilities";
+const char kSetCursorLocationMethod[] = "SetCursorLocation";
+const char kProcessKeyEventMethod[] = "ProcessKeyEvent";
+}  // namespace input_context
+
+}  // namespace ibus
 }  // namespace chromeos
 
-#endif  // CHROMEOS_DBUS_IBUS_IBUS_CONST_H_
+#endif  // CHROMEOS_DBUS_IBUS_IBUS_CONSTANTS_H_

chromeos/dbus/ibus/ibus_input_context_client.cc

@@ -20,22 +20,6 @@ namespace chromeos {
 using chromeos::ibus::IBusText;
 
 namespace {
-const char kIBusInputContextInterface[] = "org.freedesktop.IBus.InputContext";
-
-// Signal names.
-const char kCommitTextSignal[] = "CommitText";
-const char kForwardKeyEventSignal[] = "ForwardKeyEvent";
-const char kHidePreeditTextSignal[] = "HidePreeditText";
-const char kShowPreeditTextSignal[] = "ShowPreeditText";
-const char kUpdatePreeditTextSignal[] = "UpdatePreeditText";
-
-// Method names.
-const char kFocusInMethod[] = "FocusIn";
-const char kFocusOutMethod[] = "FocusOut";
-const char kResetMethod[] = "Reset";
-const char kSetCapabilitiesMethod[] = "SetCapabilities";
-const char kSetCursorLocationMethod[] = "SetCursorLocation";
-const char kProcessKeyEventMethod[] = "ProcessKeyEvent";
 
 // The IBusInputContextClient implementation.
 class IBusInputContextClientImpl : public IBusInputContextClient {
@@ -55,7 +39,7 @@ class IBusInputContextClientImpl : public IBusInputContextClient {
       LOG(ERROR) << "IBusInputContextClient is already initialized.";
       return;
     }
-    proxy_ = bus->GetObjectProxy(kIBusServiceName, object_path);
+    proxy_ = bus->GetObjectProxy(ibus::kServiceName, object_path);
 
     ConnectSignals();
   }
@@ -133,42 +117,47 @@ class IBusInputContextClientImpl : public IBusInputContextClient {
 
   // IBusInputContextClient override.
   virtual void SetCapabilities(uint32 capabilities) OVERRIDE {
-    dbus::MethodCall method_call(kIBusInputContextInterface,
-                                 kSetCapabilitiesMethod);
+    dbus::MethodCall method_call(ibus::input_context::kServiceInterface,
+                                 ibus::input_context::kSetCapabilitiesMethod);
     dbus::MessageWriter writer(&method_call);
     writer.AppendUint32(capabilities);
-    CallNoResponseMethod(&method_call, kSetCapabilitiesMethod);
+    CallNoResponseMethod(&method_call,
+                         ibus::input_context::kSetCapabilitiesMethod);
   }
 
   // IBusInputContextClient override.
   virtual void FocusIn() OVERRIDE {
-    dbus::MethodCall method_call(kIBusInputContextInterface, kFocusInMethod);
-    CallNoResponseMethod(&method_call, kFocusInMethod);
+    dbus::MethodCall method_call(ibus::input_context::kServiceInterface,
+                                 ibus::input_context::kFocusInMethod);
+    CallNoResponseMethod(&method_call, ibus::input_context::kFocusInMethod);
   }
 
   // IBusInputContextClient override.
   virtual void FocusOut() OVERRIDE {
-    dbus::MethodCall method_call(kIBusInputContextInterface, kFocusOutMethod);
-    CallNoResponseMethod(&method_call, kFocusOutMethod);
+    dbus::MethodCall method_call(ibus::input_context::kServiceInterface,
+                                 ibus::input_context::kFocusOutMethod);
+    CallNoResponseMethod(&method_call, ibus::input_context::kFocusOutMethod);
   }
 
   // IBusInputContextClient override.
   virtual void Reset() OVERRIDE {
-    dbus::MethodCall method_call(kIBusInputContextInterface, kResetMethod);
-    CallNoResponseMethod(&method_call, kResetMethod);
+    dbus::MethodCall method_call(ibus::input_context::kServiceInterface,
+                                 ibus::input_context::kResetMethod);
+    CallNoResponseMethod(&method_call, ibus::input_context::kResetMethod);
   }
 
   // IBusInputContextClient override.
   virtual void SetCursorLocation(int32 x, int32 y, int32 width,
                                  int32 height) OVERRIDE {
-    dbus::MethodCall method_call(kIBusInputContextInterface,
-                                 kSetCursorLocationMethod);
+    dbus::MethodCall method_call(ibus::input_context::kServiceInterface,
+                                 ibus::input_context::kSetCursorLocationMethod);
     dbus::MessageWriter writer(&method_call);
     writer.AppendInt32(x);
     writer.AppendInt32(y);
     writer.AppendInt32(width);
     writer.AppendInt32(height);
-    CallNoResponseMethod(&method_call, kSetCursorLocationMethod);
+    CallNoResponseMethod(&method_call,
+                         ibus::input_context::kSetCursorLocationMethod);
   }
 
   // IBusInputContextClient override.
@@ -178,8 +167,8 @@ class IBusInputContextClientImpl : public IBusInputContextClient {
       uint32 state,
       const ProcessKeyEventCallback& callback,
       const ErrorCallback& error_callback) OVERRIDE {
-    dbus::MethodCall method_call(kIBusInputContextInterface,
-                                 kProcessKeyEventMethod);
+    dbus::MethodCall method_call(ibus::input_context::kServiceInterface,
+                                 ibus::input_context::kProcessKeyEventMethod);
     dbus::MessageWriter writer(&method_call);
     writer.AppendUint32(keyval);
     writer.AppendUint32(keycode);
@@ -313,40 +302,40 @@ class IBusInputContextClientImpl : public IBusInputContextClient {
   // Connects signals to signal handlers.
   void ConnectSignals() {
     proxy_->ConnectToSignal(
-        kIBusInputContextInterface,
-        kCommitTextSignal,
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kCommitTextSignal,
         base::Bind(&IBusInputContextClientImpl::OnCommitText,
                    weak_ptr_factory_.GetWeakPtr()),
         base::Bind(&IBusInputContextClientImpl::OnSignalConnected,
                    weak_ptr_factory_.GetWeakPtr()));
 
     proxy_->ConnectToSignal(
-        kIBusInputContextInterface,
-        kForwardKeyEventSignal,
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kForwardKeyEventSignal,
         base::Bind(&IBusInputContextClientImpl::OnForwardKeyEvent,
                    weak_ptr_factory_.GetWeakPtr()),
         base::Bind(&IBusInputContextClientImpl::OnSignalConnected,
                    weak_ptr_factory_.GetWeakPtr()));
 
     proxy_->ConnectToSignal(
-        kIBusInputContextInterface,
-        kUpdatePreeditTextSignal,
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kUpdatePreeditTextSignal,
         base::Bind(&IBusInputContextClientImpl::OnUpdatePreeditText,
                    weak_ptr_factory_.GetWeakPtr()),
         base::Bind(&IBusInputContextClientImpl::OnSignalConnected,
                    weak_ptr_factory_.GetWeakPtr()));
 
     proxy_->ConnectToSignal(
-        kIBusInputContextInterface,
-        kShowPreeditTextSignal,
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kShowPreeditTextSignal,
         base::Bind(&IBusInputContextClientImpl::OnShowPreeditText,
                    weak_ptr_factory_.GetWeakPtr()),
         base::Bind(&IBusInputContextClientImpl::OnSignalConnected,
                    weak_ptr_factory_.GetWeakPtr()));
 
     proxy_->ConnectToSignal(
-        kIBusInputContextInterface,
-        kHidePreeditTextSignal,
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kHidePreeditTextSignal,
         base::Bind(&IBusInputContextClientImpl::OnHidePreeditText,
                    weak_ptr_factory_.GetWeakPtr()),
         base::Bind(&IBusInputContextClientImpl::OnSignalConnected,

chromeos/dbus/ibus/ibus_input_context_client_unittest.cc

@@ -7,6 +7,7 @@
 #include <map>
 #include <string>
 #include "base/message_loop.h"
+#include "chromeos/dbus/ibus/ibus_constants.h"
 #include "chromeos/dbus/ibus/ibus_text.h"
 #include "dbus/message.h"
 #include "dbus/mock_bus.h"
@@ -24,25 +25,8 @@ namespace chromeos {
 using chromeos::ibus::IBusText;
 
 namespace {
-const char kServiceName[] = "org.freedesktop.IBus";
-const char kInputContextInterface[] = "org.freedesktop.IBus.InputContext";
 const char kObjectPath[] = "/org/freedesktop/IBus/InputContext_1010";
 
-// Signal names.
-const char kCommitTextSignal[] = "CommitText";
-const char kForwardKeyEventSignal[] = "ForwardKeyEvent";
-const char kHidePreeditTextSignal[] = "HidePreeditText";
-const char kShowPreeditTextSignal[] = "ShowPreeditText";
-const char kUpdatePreeditTextSignal[] = "UpdatePreeditText";
-
-// Method names.
-const char kFocusInMethod[] = "FocusIn";
-const char kFocusOutMethod[] = "FocusOut";
-const char kResetMethod[] = "Reset";
-const char kSetCapabilitiesMethod[] = "SetCapabilities";
-const char kSetCursorLocationMethod[] = "SetCursorLocation";
-const char kProcessKeyEventMethod[] = "ProcessKeyEvent";
-
 // Following variables are used in callback expectations.
 const uint32 kCapabilities = 12345;
 const int32 kCursorX = 30;
@@ -109,7 +93,7 @@ class IBusInputContextClientTest : public testing::Test {
 
     // Create a mock proxy.
     mock_proxy_ = new dbus::MockObjectProxy(mock_bus_.get(),
-                                            kServiceName,
+                                            ibus::kServiceName,
                                             dbus::ObjectPath(kObjectPath));
 
     // Create a client.
@@ -119,7 +103,7 @@ class IBusInputContextClientTest : public testing::Test {
     // Set an expectation so mock_bus's GetObjectProxy() for the given service
     // name and the object path will return mock_proxy_. The GetObjectProxy
     // function is called in Initialized function.
-    EXPECT_CALL(*mock_bus_, GetObjectProxy(kServiceName,
+    EXPECT_CALL(*mock_bus_, GetObjectProxy(ibus::kServiceName,
                                            dbus::ObjectPath(kObjectPath)))
         .WillOnce(Return(mock_proxy_.get()));
 
@@ -127,23 +111,28 @@ class IBusInputContextClientTest : public testing::Test {
     // OnConnectToSignal() to run the callback. The ConnectToSignal is called in
     // Initialize function.
     EXPECT_CALL(*mock_proxy_, ConnectToSignal(
-        kInputContextInterface, kCommitTextSignal, _, _))
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kCommitTextSignal, _, _))
         .WillRepeatedly(
             Invoke(this, &IBusInputContextClientTest::OnConnectToSignal));
     EXPECT_CALL(*mock_proxy_, ConnectToSignal(
-        kInputContextInterface, kForwardKeyEventSignal, _, _))
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kForwardKeyEventSignal, _, _))
         .WillRepeatedly(
             Invoke(this, &IBusInputContextClientTest::OnConnectToSignal));
     EXPECT_CALL(*mock_proxy_, ConnectToSignal(
-        kInputContextInterface, kHidePreeditTextSignal, _, _))
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kHidePreeditTextSignal, _, _))
         .WillRepeatedly(
             Invoke(this, &IBusInputContextClientTest::OnConnectToSignal));
     EXPECT_CALL(*mock_proxy_, ConnectToSignal(
-        kInputContextInterface, kShowPreeditTextSignal, _, _))
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kShowPreeditTextSignal, _, _))
         .WillRepeatedly(
             Invoke(this, &IBusInputContextClientTest::OnConnectToSignal));
     EXPECT_CALL(*mock_proxy_, ConnectToSignal(
-        kInputContextInterface, kUpdatePreeditTextSignal, _, _))
+        ibus::input_context::kServiceInterface,
+        ibus::input_context::kUpdatePreeditTextSignal, _, _))
         .WillRepeatedly(
             Invoke(this, &IBusInputContextClientTest::OnConnectToSignal));
 
@@ -162,8 +151,9 @@ class IBusInputContextClientTest : public testing::Test {
                  int timeout_ms,
                  const dbus::ObjectProxy::ResponseCallback& callback,
                  const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kFocusInMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kFocusInMethod, method_call->GetMember());
     dbus::MessageReader reader(method_call);
     EXPECT_FALSE(reader.HasMoreData());
 
@@ -175,8 +165,9 @@ class IBusInputContextClientTest : public testing::Test {
                   int timeout_ms,
                   const dbus::ObjectProxy::ResponseCallback& callback,
                   const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kFocusOutMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kFocusOutMethod, method_call->GetMember());
     dbus::MessageReader reader(method_call);
     EXPECT_FALSE(reader.HasMoreData());
 
@@ -188,8 +179,9 @@ class IBusInputContextClientTest : public testing::Test {
                int timeout_ms,
                const dbus::ObjectProxy::ResponseCallback& callback,
                const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kResetMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kResetMethod, method_call->GetMember());
     dbus::MessageReader reader(method_call);
     EXPECT_FALSE(reader.HasMoreData());
 
@@ -202,8 +194,10 @@ class IBusInputContextClientTest : public testing::Test {
       int timeout_ms,
       const dbus::ObjectProxy::ResponseCallback& callback,
       const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kSetCursorLocationMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kSetCursorLocationMethod,
+              method_call->GetMember());
     dbus::MessageReader reader(method_call);
     int32 x, y, width, height;
     EXPECT_TRUE(reader.PopInt32(&x));
@@ -221,8 +215,10 @@ class IBusInputContextClientTest : public testing::Test {
       int timeout_ms,
       const dbus::ObjectProxy::ResponseCallback& callback,
       const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kSetCapabilitiesMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kSetCapabilitiesMethod,
+              method_call->GetMember());
     uint32 capabilities;
     dbus::MessageReader reader(method_call);
     EXPECT_TRUE(reader.PopUint32(&capabilities));
@@ -238,8 +234,10 @@ class IBusInputContextClientTest : public testing::Test {
       int timeout_ms,
       const dbus::ObjectProxy::ResponseCallback& callback,
       const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kProcessKeyEventMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kProcessKeyEventMethod,
+              method_call->GetMember());
     uint32 keyval, keycode, state;
     dbus::MessageReader reader(method_call);
     EXPECT_TRUE(reader.PopUint32(&keyval));
@@ -255,8 +253,10 @@ class IBusInputContextClientTest : public testing::Test {
       int timeout_ms,
       const dbus::ObjectProxy::ResponseCallback& callback,
       const dbus::ObjectProxy::ErrorCallback& error_callback) {
-    EXPECT_EQ(kInputContextInterface, method_call->GetInterface());
-    EXPECT_EQ(kProcessKeyEventMethod, method_call->GetMember());
+    EXPECT_EQ(ibus::input_context::kServiceInterface,
+              method_call->GetInterface());
+    EXPECT_EQ(ibus::input_context::kProcessKeyEventMethod,
+              method_call->GetMember());
     uint32 keyval, keycode, state;
     dbus::MessageReader reader(method_call);
     EXPECT_TRUE(reader.PopUint32(&keyval));
@@ -312,15 +312,17 @@ TEST_F(IBusInputContextClientTest, CommitTextHandler) {
   message_loop_.RunAllPending();
 
   // Emit signal.
-  dbus::Signal signal(kInputContextInterface, kCommitTextSignal);
+  dbus::Signal signal(ibus::input_context::kServiceInterface,
+                      ibus::input_context::kCommitTextSignal);
   dbus::MessageWriter writer(&signal);
   AppendIBusText(ibus_text, &writer);
-  ASSERT_FALSE(signal_callback_map_[kCommitTextSignal].is_null());
-  signal_callback_map_[kCommitTextSignal].Run(&signal);
+  ASSERT_FALSE(
+      signal_callback_map_[ibus::input_context::kCommitTextSignal].is_null());
+  signal_callback_map_[ibus::input_context::kCommitTextSignal].Run(&signal);
 
   // Unset the handler so expect not calling handler.
   client_->UnsetCommitTextHandler();
-  signal_callback_map_[kCommitTextSignal].Run(&signal);
+  signal_callback_map_[ibus::input_context::kCommitTextSignal].Run(&signal);
 }
 
 TEST_F(IBusInputContextClientTest, ForwardKeyEventHandlerTest) {
@@ -333,17 +335,22 @@ TEST_F(IBusInputContextClientTest, ForwardKeyEventHandlerTest) {
   message_loop_.RunAllPending();
 
   // Emit signal.
-  dbus::Signal signal(kInputContextInterface, kForwardKeyEventSignal);
+  dbus::Signal signal(ibus::input_context::kServiceInterface,
+                      ibus::input_context::kForwardKeyEventSignal);
   dbus::MessageWriter writer(&signal);
   writer.AppendUint32(kKeyval);
   writer.AppendUint32(kKeycode);
   writer.AppendUint32(kState);
-  ASSERT_FALSE(signal_callback_map_[kForwardKeyEventSignal].is_null());
-  signal_callback_map_[kForwardKeyEventSignal].Run(&signal);
+  ASSERT_FALSE(
+      signal_callback_map_[ibus::input_context::kForwardKeyEventSignal]
+          .is_null());
+  signal_callback_map_[ibus::input_context::kForwardKeyEventSignal].Run(
+      &signal);
 
   // Unset the handler so expect not calling handler.
   client_->UnsetForwardKeyEventHandler();
-  signal_callback_map_[kForwardKeyEventSignal].Run(&signal);
+  signal_callback_map_[ibus::input_context::kForwardKeyEventSignal].Run(
+      &signal);
 }
 
 TEST_F(IBusInputContextClientTest, HidePreeditTextHandlerTest) {
@@ -356,13 +363,18 @@ TEST_F(IBusInputContextClientTest, HidePreeditTextHandlerTest) {
   message_loop_.RunAllPending();
 
   // Emit signal.
-  dbus::Signal signal(kInputContextInterface, kHidePreeditTextSignal);
-  ASSERT_FALSE(signal_callback_map_[kHidePreeditTextSignal].is_null());
-  signal_callback_map_[kHidePreeditTextSignal].Run(&signal);
+  dbus::Signal signal(ibus::input_context::kServiceInterface,
+                      ibus::input_context::kHidePreeditTextSignal);
+  ASSERT_FALSE(
+      signal_callback_map_[ibus::input_context::kHidePreeditTextSignal]
+          .is_null());
+  signal_callback_map_[ibus::input_context::kHidePreeditTextSignal].Run(
+      &signal);
 
   // Unset the handler so expect not calling handler.
   client_->UnsetHidePreeditTextHandler();
-  signal_callback_map_[kHidePreeditTextSignal].Run(&signal);
+  signal_callback_map_[ibus::input_context::kHidePreeditTextSignal].Run(
+      &signal);
 }
 
 TEST_F(IBusInputContextClientTest, ShowPreeditTextHandlerTest) {
@@ -375,13 +387,18 @@ TEST_F(IBusInputContextClientTest, ShowPreeditTextHandlerTest) {
   message_loop_.RunAllPending();
 
   // Emit signal.
-  dbus::Signal signal(kInputContextInterface, kShowPreeditTextSignal);
-  ASSERT_FALSE(signal_callback_map_[kShowPreeditTextSignal].is_null());
-  signal_callback_map_[kShowPreeditTextSignal].Run(&signal);
+  dbus::Signal signal(ibus::input_context::kServiceInterface,
+                      ibus::input_context::kShowPreeditTextSignal);
+  ASSERT_FALSE(
+      signal_callback_map_[ibus::input_context::kShowPreeditTextSignal]
+          .is_null());
+  signal_callback_map_[ibus::input_context::kShowPreeditTextSignal].Run(
+      &signal);
 
   // Unset the handler so expect not calling handler.
   client_->UnsetShowPreeditTextHandler();
-  signal_callback_map_[kShowPreeditTextSignal].Run(&signal);
+  signal_callback_map_[ibus::input_context::kShowPreeditTextSignal].Run(
+      &signal);
 }
 
 TEST_F(IBusInputContextClientTest, UpdatePreeditTextHandlerTest) {
@@ -400,17 +417,22 @@ TEST_F(IBusInputContextClientTest, UpdatePreeditTextHandlerTest) {
   message_loop_.RunAllPending();
 
   // Emit signal.
-  dbus::Signal signal(kInputContextInterface, kUpdatePreeditTextSignal);
+  dbus::Signal signal(ibus::input_context::kServiceInterface,
+                      ibus::input_context::kUpdatePreeditTextSignal);
   dbus::MessageWriter writer(&signal);
   AppendIBusText(ibus_text, &writer);
   writer.AppendUint32(kCursorPos);
   writer.AppendBool(kVisible);
-  ASSERT_FALSE(signal_callback_map_[kUpdatePreeditTextSignal].is_null());
-  signal_callback_map_[kUpdatePreeditTextSignal].Run(&signal);
+  ASSERT_FALSE(
+      signal_callback_map_[ibus::input_context::kUpdatePreeditTextSignal]
+          .is_null());
+  signal_callback_map_[ibus::input_context::kUpdatePreeditTextSignal].Run(
+      &signal);
 
   // Unset the handler so expect not calling handler.
   client_->UnsetUpdatePreeditTextHandler();
-  signal_callback_map_[kUpdatePreeditTextSignal].Run(&signal);
+  signal_callback_map_[ibus::input_context::kUpdatePreeditTextSignal].Run(
+      &signal);
 }
 
 TEST_F(IBusInputContextClientTest, FocusInTest) {