diff options
author | sadrul <sadrul@chromium.org> | 2015-12-09 13:41:51 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-12-09 21:43:30 +0000 |
commit | 0ec0b802541c1b21e66bff3943a1bfcac9b5711e (patch) | |
tree | 209b05eb8a1f34909a9b0135740a3762fddff0b3 /components/mus/ws/window_tree_unittest.cc | |
parent | 70bcae439e4adfdecbc508787d2334a88f7f7984 (diff) | |
download | chromium_src-0ec0b802541c1b21e66bff3943a1bfcac9b5711e.zip chromium_src-0ec0b802541c1b21e66bff3943a1bfcac9b5711e.tar.gz chromium_src-0ec0b802541c1b21e66bff3943a1bfcac9b5711e.tar.bz2 |
mus: Use ui::Event in DisplayManagerDelegate.
The window server in mus needs to create a mojom::Event out of a ui::Event before
it can be dispatched to the client. However, internally, the window server can
still use ui::Events it receives from the PlatformWindow. This patch delays the
conversion from ui::Event to mojom::Event a little, and so DisplayManager can
send a ui::Event to its delegate.
BUG=none
Review URL: https://codereview.chromium.org/1512003002
Cr-Commit-Position: refs/heads/master@{#364161}
Diffstat (limited to 'components/mus/ws/window_tree_unittest.cc')
-rw-r--r-- | components/mus/ws/window_tree_unittest.cc | 69 |
1 files changed, 23 insertions, 46 deletions
diff --git a/components/mus/ws/window_tree_unittest.cc b/components/mus/ws/window_tree_unittest.cc index 153cd86..71fd138 100644 --- a/components/mus/ws/window_tree_unittest.cc +++ b/components/mus/ws/window_tree_unittest.cc @@ -25,6 +25,8 @@ #include "mojo/converters/geometry/geometry_type_converters.h" #include "mojo/services/network/public/interfaces/url_loader.mojom.h" #include "testing/gtest/include/gtest/gtest.h" +#include "ui/events/event.h" +#include "ui/events/event_utils.h" #include "ui/gfx/geometry/rect.h" using mojo::Array; @@ -278,56 +280,31 @@ class TestDisplayManagerFactory : public DisplayManagerFactory { DISALLOW_COPY_AND_ASSIGN(TestDisplayManagerFactory); }; -EventPtr CreatePointerDownEvent(int x, int y) { - EventPtr event(Event::New()); - event->action = mus::mojom::EVENT_TYPE_POINTER_DOWN; - event->pointer_data = PointerData::New(); - event->pointer_data->pointer_id = 1u; - event->pointer_data->location = LocationData::New(); - event->pointer_data->location->x = x; - event->pointer_data->location->y = y; - event->pointer_data->kind = mojom::POINTER_KIND_TOUCH; - return event; +ui::TouchEvent CreatePointerDownEvent(int x, int y) { + return ui::TouchEvent(ui::ET_TOUCH_PRESSED, gfx::Point(x, y), 1, + ui::EventTimeForNow()); } -EventPtr CreatePointerUpEvent(int x, int y) { - EventPtr event(Event::New()); - event->action = mus::mojom::EVENT_TYPE_POINTER_UP; - event->pointer_data = PointerData::New(); - event->pointer_data->pointer_id = 1u; - event->pointer_data->location = LocationData::New(); - event->pointer_data->location->x = x; - event->pointer_data->location->y = y; - event->pointer_data->kind = mojom::POINTER_KIND_TOUCH; - return event; +ui::TouchEvent CreatePointerUpEvent(int x, int y) { + return ui::TouchEvent(ui::ET_TOUCH_RELEASED, gfx::Point(x, y), 1, + ui::EventTimeForNow()); } -EventPtr CreateMouseMoveEvent(int x, int y) { - EventPtr event(Event::New()); - event->action = mus::mojom::EVENT_TYPE_POINTER_MOVE; - event->pointer_data = PointerData::New(); - event->pointer_data->pointer_id = std::numeric_limits<uint32_t>::max(); - event->pointer_data->location = LocationData::New(); - event->pointer_data->location->x = x; - event->pointer_data->location->y = y; - event->pointer_data->kind = mojom::POINTER_KIND_MOUSE; - return event; +ui::MouseEvent CreateMouseMoveEvent(int x, int y) { + return ui::MouseEvent(ui::ET_MOUSE_MOVED, gfx::Point(x, y), gfx::Point(x, y), + ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE); } -EventPtr CreateMouseDownEvent(int x, int y) { - EventPtr event = CreatePointerDownEvent(x, y); - event->flags = static_cast<mus::mojom::EventFlags>( - event->flags | mojom::EVENT_FLAGS_LEFT_MOUSE_BUTTON); - event->pointer_data->kind = mojom::POINTER_KIND_MOUSE; - return event; +ui::MouseEvent CreateMouseDownEvent(int x, int y) { + return ui::MouseEvent(ui::ET_MOUSE_PRESSED, gfx::Point(x, y), + gfx::Point(x, y), ui::EventTimeForNow(), + ui::EF_LEFT_MOUSE_BUTTON, ui::EF_LEFT_MOUSE_BUTTON); } -EventPtr CreateMouseUpEvent(int x, int y) { - EventPtr event = CreatePointerUpEvent(x, y); - event->flags = static_cast<mus::mojom::EventFlags>( - event->flags | mojom::EVENT_FLAGS_LEFT_MOUSE_BUTTON); - event->pointer_data->kind = mojom::POINTER_KIND_MOUSE; - return event; +ui::MouseEvent CreateMouseUpEvent(int x, int y) { + return ui::MouseEvent(ui::ET_MOUSE_RELEASED, gfx::Point(x, y), + gfx::Point(x, y), ui::EventTimeForNow(), + ui::EF_LEFT_MOUSE_BUTTON, ui::EF_LEFT_MOUSE_BUTTON); } } // namespace @@ -362,8 +339,8 @@ class WindowTreeTest : public testing::Test { TestWindowTreeHostConnection* host_connection() { return host_connection_; } - void DispatchEventWithoutAck(mojom::EventPtr event) { - host_connection()->window_tree_host()->OnEvent(std::move(event)); + void DispatchEventWithoutAck(const ui::Event& event) { + host_connection()->window_tree_host()->OnEvent(event); } void AckPreviousEvent() { @@ -372,8 +349,8 @@ class WindowTreeTest : public testing::Test { ->tree_awaiting_input_ack_->OnWindowInputEventAck(0); } - void DispatchEventAndAckImmediately(mojom::EventPtr event) { - DispatchEventWithoutAck(std::move(event)); + void DispatchEventAndAckImmediately(const ui::Event& event) { + DispatchEventWithoutAck(event); AckPreviousEvent(); } |