Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1835)

Unified Diff: ui/views/view_targeter_unittest.cc

Issue 2930803002: Remove flag --disable-views-rect-based-targeting (Closed)
Patch Set: restore enum entry Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/flag_descriptions.cc ('k') | ui/views/views_switches.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/view_targeter_unittest.cc
diff --git a/ui/views/view_targeter_unittest.cc b/ui/views/view_targeter_unittest.cc
index d72a26a1ab113f95f73efaa674bffbe13b8f4dea..1427abd47826fe1a8e21d7f45aa927a37fd84e26 100644
--- a/ui/views/view_targeter_unittest.cc
+++ b/ui/views/view_targeter_unittest.cc
@@ -13,7 +13,6 @@
#include "ui/views/test/views_test_base.h"
#include "ui/views/view_targeter.h"
#include "ui/views/view_targeter_delegate.h"
-#include "ui/views/views_switches.h"
#include "ui/views/widget/root_view.h"
namespace views {
@@ -399,13 +398,7 @@ TEST_F(ViewTargeterTest, TargetContentsAndRootView) {
details.set_bounding_box(bounding_box);
tap = GestureEventForTest(details);
- // This only applies if rect-based targeting is enabled.
- if (views::switches::IsRectBasedTargetingEnabled()) {
- EXPECT_EQ(content, targeter->FindTargetForEvent(root_view, &tap));
- } else {
- EXPECT_EQ(widget.GetRootView(),
- targeter->FindTargetForEvent(root_view, &tap));
- }
+ EXPECT_EQ(content, targeter->FindTargetForEvent(root_view, &tap));
// A gesture event not overlapping the contents view by at least
// 60% and not having its center within the contents view should
« no previous file with comments | « chrome/browser/flag_descriptions.cc ('k') | ui/views/views_switches.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698