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

Unified Diff: ash/wm/drag_window_resizer_unittest.cc

Issue 106303005: Fix AdjustBoundsToEnsureWindowVisibility to work with non primary display bounds (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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
Index: ash/wm/drag_window_resizer_unittest.cc
diff --git a/ash/wm/drag_window_resizer_unittest.cc b/ash/wm/drag_window_resizer_unittest.cc
index a9220d5cd33f78b8dae4ce2d412203b82952696c..2d5161505f179cc35d89355fe40aafd803fa7100 100644
--- a/ash/wm/drag_window_resizer_unittest.cc
+++ b/ash/wm/drag_window_resizer_unittest.cc
@@ -243,12 +243,9 @@ TEST_F(DragWindowResizerTest, WindowDragWithMultiDisplays) {
gfx::Rect intersect(window_->GetRootWindow()->GetBoundsInScreen());
intersect.Intersect(window_bounds_in_screen);
- // TODO(oshima): Following condition fails without docked window resizer.
- // Proper fix caused the other failures, so I'm disabling these
- // for m33 (Which is harmless).
- // EXPECT_LE(10, intersect.width());
- // EXPECT_LE(10, intersect.height());
- // EXPECT_TRUE(window_bounds_in_screen.Contains(gfx::Point(800, 10)));
+ EXPECT_LE(10, intersect.width());
+ EXPECT_LE(10, intersect.height());
+ EXPECT_TRUE(window_bounds_in_screen.Contains(gfx::Point(800, 10)));
}
// Dropping a window that is larger than the destination work area

Powered by Google App Engine
This is Rietveld 408576698