Index: content/browser/renderer_host/render_widget_host_view_aura_unittest.cc |
diff --git a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc |
index e4b6181b7471f9eb93c9914e7b06c0a79d9f230e..7c866103709c51d48f251259c564ae68ec8c9764 100644 |
--- a/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc |
+++ b/content/browser/renderer_host/render_widget_host_view_aura_unittest.cc |
@@ -180,19 +180,22 @@ TEST_F(RenderWidgetHostViewAuraTest, TouchEventState) { |
ui::TouchEvent release(ui::ET_TOUCH_RELEASED, gfx::Point(20, 20), 0, |
base::Time::NowFromSystemTime() - base::Time()); |
- EXPECT_EQ(ui::ER_UNHANDLED, view_->OnTouchEvent(&press)); |
+ view_->OnTouchEvent(&press); |
+ EXPECT_FALSE(press.handled()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchStart, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StatePressed, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_UNHANDLED, view_->OnTouchEvent(&move)); |
+ view_->OnTouchEvent(&move); |
+ EXPECT_FALSE(move.handled()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchMove, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StateMoved, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_UNHANDLED, view_->OnTouchEvent(&release)); |
+ view_->OnTouchEvent(&release); |
+ EXPECT_FALSE(release.handled()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchEnd, view_->touch_event_.type); |
EXPECT_EQ(0U, view_->touch_event_.touchesLength); |
@@ -202,24 +205,28 @@ TEST_F(RenderWidgetHostViewAuraTest, TouchEventState) { |
widget_host_->OnMessageReceived(ViewHostMsg_HasTouchEventHandlers(0, true)); |
EXPECT_TRUE(widget_host_->ShouldForwardTouchEvent()); |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&press)); |
+ view_->OnTouchEvent(&press); |
+ EXPECT_TRUE(press.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchStart, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StatePressed, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&move)); |
+ view_->OnTouchEvent(&move); |
+ EXPECT_TRUE(move.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchMove, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StateMoved, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&release)); |
+ view_->OnTouchEvent(&release); |
+ EXPECT_TRUE(release.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchEnd, view_->touch_event_.type); |
EXPECT_EQ(0U, view_->touch_event_.touchesLength); |
// Now start a touch event, and remove the event-handlers before the release. |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&press)); |
+ view_->OnTouchEvent(&press); |
+ EXPECT_TRUE(press.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchStart, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StatePressed, |
@@ -228,13 +235,19 @@ TEST_F(RenderWidgetHostViewAuraTest, TouchEventState) { |
widget_host_->OnMessageReceived(ViewHostMsg_HasTouchEventHandlers(0, false)); |
EXPECT_FALSE(widget_host_->ShouldForwardTouchEvent()); |
- EXPECT_EQ(ui::ER_UNHANDLED, view_->OnTouchEvent(&move)); |
+ ui::TouchEvent move2(ui::ET_TOUCH_MOVED, gfx::Point(20, 20), 0, |
+ base::Time::NowFromSystemTime() - base::Time()); |
+ view_->OnTouchEvent(&move2); |
+ EXPECT_FALSE(move2.handled()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchMove, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StateMoved, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_UNHANDLED, view_->OnTouchEvent(&release)); |
+ ui::TouchEvent release2(ui::ET_TOUCH_RELEASED, gfx::Point(20, 20), 0, |
+ base::Time::NowFromSystemTime() - base::Time()); |
+ view_->OnTouchEvent(&release2); |
+ EXPECT_FALSE(release2.handled()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchEnd, view_->touch_event_.type); |
EXPECT_EQ(0U, view_->touch_event_.touchesLength); |
} |
@@ -255,13 +268,15 @@ TEST_F(RenderWidgetHostViewAuraTest, TouchEventSyncAsync) { |
ui::TouchEvent release(ui::ET_TOUCH_RELEASED, gfx::Point(20, 20), 0, |
base::Time::NowFromSystemTime() - base::Time()); |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&press)); |
+ view_->OnTouchEvent(&press); |
+ EXPECT_TRUE(press.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchStart, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StatePressed, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&move)); |
+ view_->OnTouchEvent(&move); |
+ EXPECT_TRUE(move.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchMove, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StateMoved, |
@@ -269,13 +284,15 @@ TEST_F(RenderWidgetHostViewAuraTest, TouchEventSyncAsync) { |
// Send the same move event. Since the point hasn't moved, it won't affect the |
// queue. However, the view should consume the event. |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&move)); |
+ view_->OnTouchEvent(&move); |
+ EXPECT_TRUE(move.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchMove, view_->touch_event_.type); |
EXPECT_EQ(1U, view_->touch_event_.touchesLength); |
EXPECT_EQ(WebKit::WebTouchPoint::StateMoved, |
view_->touch_event_.touches[0].state); |
- EXPECT_EQ(ui::ER_CONSUMED, view_->OnTouchEvent(&release)); |
+ view_->OnTouchEvent(&release); |
+ EXPECT_TRUE(release.stopped_propagation()); |
EXPECT_EQ(WebKit::WebInputEvent::TouchEnd, view_->touch_event_.type); |
EXPECT_EQ(0U, view_->touch_event_.touchesLength); |
} |