Index: content/browser/renderer_host/input/input_router_impl_unittest.cc |
diff --git a/content/browser/renderer_host/input/input_router_impl_unittest.cc b/content/browser/renderer_host/input/input_router_impl_unittest.cc |
index 034eb5ba27b2b3331739dd2c0ff9348903f5fa92..203fe99e8a51095b573f9cfa3add600e47a4a587 100644 |
--- a/content/browser/renderer_host/input/input_router_impl_unittest.cc |
+++ b/content/browser/renderer_host/input/input_router_impl_unittest.cc |
@@ -80,12 +80,6 @@ WebInputEvent& GetEventWithType(WebInputEvent::Type type) { |
return *event; |
} |
-bool GetIsShortcutFromHandleInputEventMessage(const IPC::Message* msg) { |
- InputMsg_HandleInputEvent::Schema::Param param; |
- InputMsg_HandleInputEvent::Read(msg, ¶m); |
- return base::get<2>(param); |
-} |
- |
template<typename MSG_T, typename ARG_T1> |
void ExpectIPCMessageWithArg1(const IPC::Message* msg, const ARG_T1& arg1) { |
ASSERT_EQ(MSG_T::ID, msg->type()); |
@@ -182,12 +176,12 @@ class InputRouterImplTest : public testing::Test { |
input_router()->NotifySiteIsMobileOptimized(false); |
} |
- void SimulateKeyboardEvent(WebInputEvent::Type type, bool is_shortcut) { |
+ void SimulateKeyboardEvent(WebInputEvent::Type type) { |
WebKeyboardEvent event = SyntheticWebKeyboardEventBuilder::Build(type); |
NativeWebKeyboardEvent native_event; |
memcpy(&native_event, &event, sizeof(event)); |
NativeWebKeyboardEventWithLatencyInfo key_event(native_event); |
- input_router_->SendKeyboardEvent(key_event, is_shortcut); |
+ input_router_->SendKeyboardEvent(key_event); |
} |
void SimulateWheelEvent(float dX, float dY, int modifiers, bool precise) { |
@@ -617,7 +611,7 @@ TEST_F(InputRouterImplTest, HandledInputEvent) { |
client_->set_filter_state(INPUT_EVENT_ACK_STATE_CONSUMED); |
// Simulate a keyboard event. |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, false); |
+ SimulateKeyboardEvent(WebInputEvent::RawKeyDown); |
// Make sure no input event is sent to the renderer. |
EXPECT_EQ(0u, GetSentMessageCountAndResetSink()); |
@@ -634,7 +628,7 @@ TEST_F(InputRouterImplTest, ClientCanceledKeyboardEvent) { |
client_->set_filter_state(INPUT_EVENT_ACK_STATE_NO_CONSUMER_EXISTS); |
// Simulate a keyboard event that has no consumer. |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, false); |
+ SimulateKeyboardEvent(WebInputEvent::RawKeyDown); |
// Make sure no input event is sent to the renderer. |
EXPECT_EQ(0u, GetSentMessageCountAndResetSink()); |
@@ -643,27 +637,15 @@ TEST_F(InputRouterImplTest, ClientCanceledKeyboardEvent) { |
// Simulate a keyboard event that should be dropped. |
client_->set_filter_state(INPUT_EVENT_ACK_STATE_UNKNOWN); |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, false); |
+ SimulateKeyboardEvent(WebInputEvent::RawKeyDown); |
// Make sure no input event is sent to the renderer, and no ack is sent. |
EXPECT_EQ(0u, GetSentMessageCountAndResetSink()); |
EXPECT_EQ(0U, ack_handler_->GetAndResetAckCount()); |
} |
-TEST_F(InputRouterImplTest, ShortcutKeyboardEvent) { |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, true); |
- EXPECT_TRUE(GetIsShortcutFromHandleInputEventMessage( |
- process_->sink().GetMessageAt(0))); |
- |
- process_->sink().ClearMessages(); |
- |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, false); |
- EXPECT_FALSE(GetIsShortcutFromHandleInputEventMessage( |
- process_->sink().GetMessageAt(0))); |
-} |
- |
TEST_F(InputRouterImplTest, NoncorrespondingKeyEvents) { |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, false); |
+ SimulateKeyboardEvent(WebInputEvent::RawKeyDown); |
SendInputEventACK(WebInputEvent::KeyUp, |
INPUT_EVENT_ACK_STATE_NOT_CONSUMED); |
@@ -674,7 +656,7 @@ TEST_F(InputRouterImplTest, NoncorrespondingKeyEvents) { |
TEST_F(InputRouterImplTest, HandleKeyEventsWeSent) { |
// Simulate a keyboard event. |
- SimulateKeyboardEvent(WebInputEvent::RawKeyDown, false); |
+ SimulateKeyboardEvent(WebInputEvent::RawKeyDown); |
ASSERT_TRUE(input_router_->GetLastKeyboardEvent()); |
EXPECT_EQ(WebInputEvent::RawKeyDown, |
input_router_->GetLastKeyboardEvent()->type); |