Remove some TODOs related to event targeting in views

Removes some obsolete TODOs from the code.

BUG=355425
TEST=none

Review URL: https://codereview.chromium.org/395223002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283796 0039d316-1c4b-4281-b951-d872f2087c98
parent e9738e76
...@@ -551,11 +551,6 @@ class VIEWS_EXPORT View : public ui::LayerDelegate, ...@@ -551,11 +551,6 @@ class VIEWS_EXPORT View : public ui::LayerDelegate,
// The points, rects, mouse locations, and touch locations in the following // The points, rects, mouse locations, and touch locations in the following
// functions are in the view's coordinates, except for a RootView. // functions are in the view's coordinates, except for a RootView.
// TODO(tdanderson): GetEventHandlerForPoint() and GetEventHandlerForRect()
// will be removed once their logic is moved into
// ViewTargeter and its derived classes. See
// crbug.com/355425.
// Convenience functions which calls into GetEventHandler() with // Convenience functions which calls into GetEventHandler() with
// a 1x1 rect centered at |point|. // a 1x1 rect centered at |point|.
View* GetEventHandlerForPoint(const gfx::Point& point); View* GetEventHandlerForPoint(const gfx::Point& point);
......
...@@ -267,9 +267,6 @@ TEST_F(ViewTargeterTest, SubtreeShouldBeExploredForEvent) { ...@@ -267,9 +267,6 @@ TEST_F(ViewTargeterTest, SubtreeShouldBeExploredForEvent) {
EXPECT_FALSE(targeter->SubtreeShouldBeExploredForEvent(&v2, event)); EXPECT_FALSE(targeter->SubtreeShouldBeExploredForEvent(&v2, event));
v1.ConvertEventToTarget(&v2, &event); v1.ConvertEventToTarget(&v2, &event);
EXPECT_FALSE(targeter->SubtreeShouldBeExploredForEvent(&v3, event)); EXPECT_FALSE(targeter->SubtreeShouldBeExploredForEvent(&v3, event));
// TODO(tdanderson): Move the hit-testing unit tests out of view_unittest
// and into here. See crbug.com/355425.
} }
// Tests that FindTargetForEvent() returns the correct target when some // Tests that FindTargetForEvent() returns the correct target when some
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment