Refactoring the ownership of ui::InputMethod.

ui/base/accelerators/accelerator_manager_unittest.cc

Index: ui/base/accelerators/accelerator_manager_unittest.cc
diff --git a/ui/base/accelerators/accelerator_manager_unittest.cc b/ui/base/accelerators/accelerator_manager_unittest.cc
index 1c3402d73e2b161fa4d018a5658424c451ad8b5a..1f8db027b139c0bfc3e3d251a6b0b4818ba049e0 100644
--- a/ui/base/accelerators/accelerator_manager_unittest.cc
+++ b/ui/base/accelerators/accelerator_manager_unittest.cc
@@ -151,12 +151,8 @@ TEST_F(AcceleratorManagerTest, Process) {
     // The non-registered accelerators are not processed.
     accelerator.set_type(ET_UNKNOWN);
     EXPECT_FALSE(manager_.Process(accelerator)) << text;  // different type
-    accelerator.set_type(ET_TRANSLATED_KEY_PRESS);
-    EXPECT_FALSE(manager_.Process(accelerator)) << text;  // different type

[James Cook, 2015/06/01 22:21:00]

Just curious, why aren't these needed anymore? It seems odd that there's a
KEY_RELEASED but not a KEY_PRESSED.

[Shu Chen, 2015/06/02 04:11:16]

The line 148 is to verify EXPECT_TRUE for KEY_PRESSED.
These lines are to verify EXPECT_FALSE's.

     accelerator.set_type(ET_KEY_RELEASED);
     EXPECT_FALSE(manager_.Process(accelerator)) << text;  // different type
-    accelerator.set_type(ET_TRANSLATED_KEY_RELEASE);
-    EXPECT_FALSE(manager_.Process(accelerator)) << text;  // different type
 
     EXPECT_FALSE(manager_.Process(GetAccelerator(VKEY_UNKNOWN, mask)))
         << text;  // different vkey