diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-15 22:39:27 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-01-15 22:39:27 +0000 |
| commit | 74d95ead1392e952768e659d156ccceb9ef2c792 (patch) | |
| tree | 425b83b22d0ddd4079c185d5da550bd4ef1acd22 /chrome/browser/page_menu_model_unittest.cc | |
| parent | d7a47fff65c763a5485189c16aab2e8ed52bc03c (diff) | |
| download | chromium_src-74d95ead1392e952768e659d156ccceb9ef2c792.zip chromium_src-74d95ead1392e952768e659d156ccceb9ef2c792.tar.gz chromium_src-74d95ead1392e952768e659d156ccceb9ef2c792.tar.bz2 | |
Revert part of r35312 now that ChromeOS is fixed.
The test used to rely on menu item #3, while the change made it use the
last item; the last item on Linux is a submenu so the test was failing.
Go back to menu item #3.
Review URL: http://codereview.chromium.org/545098
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36429 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/page_menu_model_unittest.cc')
| -rw-r--r-- | chrome/browser/page_menu_model_unittest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/page_menu_model_unittest.cc b/chrome/browser/page_menu_model_unittest.cc index f63363e..dbac3aa 100644 --- a/chrome/browser/page_menu_model_unittest.cc +++ b/chrome/browser/page_menu_model_unittest.cc @@ -27,8 +27,8 @@ TEST_F(PageMenuModelTest, Basics) { // delegate is internal, it doesn't use the one we pass in. model.ActivatedAt(0); EXPECT_TRUE(model.IsEnabledAt(0)); - model.ActivatedAt(itemCount - 1); - EXPECT_TRUE(model.IsEnabledAt(itemCount - 1)); + model.ActivatedAt(3); + EXPECT_TRUE(model.IsEnabledAt(3)); EXPECT_EQ(delegate_.execute_count_, 2); EXPECT_EQ(delegate_.enable_count_, 2); |