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

Unified Diff: ui/views/view_model_utils_unittest.cc

Issue 10388036: Adds the option of aligning the launcher to the left or right. There (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile Created 8 years, 7 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 | « ui/views/view_model_utils.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/view_model_utils_unittest.cc
diff --git a/ui/views/view_model_utils_unittest.cc b/ui/views/view_model_utils_unittest.cc
index 289feeed1683fc5802e248e05629714c50986156..92004b709af47ff8c66a1b65ee2bd084dfc1216a 100644
--- a/ui/views/view_model_utils_unittest.cc
+++ b/ui/views/view_model_utils_unittest.cc
@@ -32,15 +32,46 @@ TEST(ViewModelUtils, DetermineMoveIndex) {
model.set_ideal_bounds(1, gfx::Rect(10, 0, 1000, 10));
model.set_ideal_bounds(2, gfx::Rect(1010, 0, 2, 10));
- EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(model, &v1, -10));
- EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(model, &v1, 4));
- EXPECT_EQ(1, ViewModelUtils::DetermineMoveIndex(model, &v1, 506));
- EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(model, &v1, 1010));
- EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(model, &v1, 2000));
-
- EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(model, &v2, -10));
- EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(model, &v2, 4));
- EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(model, &v2, 12));
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::HORIZONTAL, -10, 0));
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::HORIZONTAL, 4, 0));
+ EXPECT_EQ(1, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::HORIZONTAL, 506, 0));
+ EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::HORIZONTAL, 1010, 0));
+ EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::HORIZONTAL, 2000, 0));
+
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v2, ViewModelUtils::HORIZONTAL, -10, 0));
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v2, ViewModelUtils::HORIZONTAL, 4, 0));
+ EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(
+ model, &v2, ViewModelUtils::HORIZONTAL, 12, 0));
+
+ // Try the same when vertical.
+ model.set_ideal_bounds(0, gfx::Rect(0, 0, 10, 10));
+ model.set_ideal_bounds(1, gfx::Rect(0, 10, 10, 1000));
+ model.set_ideal_bounds(2, gfx::Rect(0, 1010, 10, 2));
+
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::VERTICAL, 0, -10));
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::VERTICAL, 0, 4));
+ EXPECT_EQ(1, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::VERTICAL, 0, 506));
+ EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::VERTICAL, 0, 1010));
+ EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(
+ model, &v1, ViewModelUtils::VERTICAL, 0, 2000));
+
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v2, ViewModelUtils::VERTICAL, 0, -10));
+ EXPECT_EQ(0, ViewModelUtils::DetermineMoveIndex(
+ model, &v2, ViewModelUtils::VERTICAL, 0, 4));
+ EXPECT_EQ(2, ViewModelUtils::DetermineMoveIndex(
+ model, &v2, ViewModelUtils::VERTICAL, 0, 12));
}
} // namespace views
« no previous file with comments | « ui/views/view_model_utils.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698