diff options
author | tdanderson@chromium.org <tdanderson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-28 05:46:13 +0000 |
---|---|---|
committer | tdanderson@chromium.org <tdanderson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-28 05:46:13 +0000 |
commit | 17850017385dbc0567a8a5bf056c818574e3f934 (patch) | |
tree | a42183eb12dcfa9888c35af8fcdb4b5ea21d8d06 /ui/views/view_targeter_unittest.cc | |
parent | b89fa1316d60fbca9031ab770aee7e17f5091243 (diff) | |
download | chromium_src-17850017385dbc0567a8a5bf056c818574e3f934.zip chromium_src-17850017385dbc0567a8a5bf056c818574e3f934.tar.gz chromium_src-17850017385dbc0567a8a5bf056c818574e3f934.tar.bz2 |
View should store a ViewTargeter instead of an EventTargeter
Change |targeter_| in View to be a ViewTargeter instead
of an EventTargeter. This will allow us to avoid casting
when calling into ViewTargeter-specific methods in
upcoming CLs.
BUG=388838
TEST=none
Review URL: https://codereview.chromium.org/354063003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@280501 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/views/view_targeter_unittest.cc')
-rw-r--r-- | ui/views/view_targeter_unittest.cc | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/ui/views/view_targeter_unittest.cc b/ui/views/view_targeter_unittest.cc index 8fd4eba..7b9b194 100644 --- a/ui/views/view_targeter_unittest.cc +++ b/ui/views/view_targeter_unittest.cc @@ -7,6 +7,7 @@ #include "ui/events/event_targeter.h" #include "ui/events/event_utils.h" #include "ui/views/test/views_test_base.h" +#include "ui/views/view_targeter.h" #include "ui/views/widget/root_view.h" namespace views { @@ -61,10 +62,11 @@ TEST_F(ViewTargeterTest, ViewTargeterForKeyEvents) { grandchild->SetFocusable(true); grandchild->RequestFocus(); - ui::EventTargeter* targeter = new ViewTargeter(); + ViewTargeter* view_targeter = new ViewTargeter(); + ui::EventTargeter* targeter = view_targeter; internal::RootView* root_view = static_cast<internal::RootView*>(widget.GetRootView()); - root_view->SetEventTargeter(make_scoped_ptr(targeter)); + root_view->SetEventTargeter(make_scoped_ptr(view_targeter)); ui::KeyEvent key_event(ui::ET_KEY_PRESSED, ui::VKEY_A, 0, true); @@ -109,10 +111,11 @@ TEST_F(ViewTargeterTest, ViewTargeterForScrollEvents) { content->AddChildView(child); child->AddChildView(grandchild); - ui::EventTargeter* targeter = new ViewTargeter(); + ViewTargeter* view_targeter = new ViewTargeter(); + ui::EventTargeter* targeter = view_targeter; internal::RootView* root_view = static_cast<internal::RootView*>(widget.GetRootView()); - root_view->SetEventTargeter(make_scoped_ptr(targeter)); + root_view->SetEventTargeter(make_scoped_ptr(view_targeter)); // The event falls within the bounds of |child| and |content| but not // |grandchild|, so |child| should be the initial target for the event. @@ -161,7 +164,7 @@ TEST_F(ViewTargeterTest, SubtreeShouldBeExploredForEvent) { params.bounds = gfx::Rect(0, 0, 650, 650); widget.Init(params); - ui::EventTargeter* targeter = new ViewTargeter(); + ViewTargeter* targeter = new ViewTargeter(); internal::RootView* root_view = static_cast<internal::RootView*>(widget.GetRootView()); root_view->SetEventTargeter(make_scoped_ptr(targeter)); @@ -226,10 +229,11 @@ TEST_F(ViewTargeterTest, CanProcessEventsWithinSubtree) { params.bounds = gfx::Rect(0, 0, 650, 650); widget.Init(params); - ui::EventTargeter* targeter = new ViewTargeter(); + ViewTargeter* view_targeter = new ViewTargeter(); + ui::EventTargeter* targeter = view_targeter; internal::RootView* root_view = static_cast<internal::RootView*>(widget.GetRootView()); - root_view->SetEventTargeter(make_scoped_ptr(targeter)); + root_view->SetEventTargeter(make_scoped_ptr(view_targeter)); // The coordinates used for SetBounds() are in the parent coordinate space. TestingView v1, v2, v3; |