[email protected] | 93cc27b | 2012-03-21 12:44:32 | [diff] [blame] | 1 | // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
[email protected] | 1a3aba8 | 2010-11-08 23:52:54 | [diff] [blame] | 5 | #include "chrome/app/chrome_command_ids.h" |
[email protected] | b7ca4e6 | 2009-01-23 20:37:29 | [diff] [blame] | 6 | #include "chrome/browser/bookmarks/bookmark_model.h" |
[email protected] | fa50dea0 | 2012-08-01 23:20:37 | [diff] [blame] | 7 | #include "chrome/browser/bookmarks/bookmark_model_factory.h" |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 8 | #include "chrome/browser/ui/browser_command_controller.h" |
[email protected] | a37d4b0 | 2012-06-25 21:56:10 | [diff] [blame] | 9 | #include "chrome/browser/ui/browser_commands.h" |
[email protected] | 0665ebe | 2013-02-13 09:53:19 | [diff] [blame] | 10 | #include "chrome/browser/ui/browser_finder.h" |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 11 | #include "chrome/browser/ui/tabs/tab_strip_model.h" |
[email protected] | 9423d941 | 2009-04-14 22:13:55 | [diff] [blame] | 12 | #include "chrome/common/url_constants.h" |
[email protected] | a4ff9eae | 2011-08-01 19:58:16 | [diff] [blame] | 13 | #include "chrome/test/base/browser_with_test_window_test.h" |
| 14 | #include "chrome/test/base/testing_profile.h" |
[email protected] | 1f9c7419 | 2013-03-25 23:04:35 | [diff] [blame] | 15 | #include "chrome/test/base/ui_test_utils.h" |
[email protected] | cdcb1dee | 2012-01-04 00:46:20 | [diff] [blame] | 16 | #include "content/public/browser/navigation_controller.h" |
[email protected] | 022af74 | 2011-12-28 18:37:25 | [diff] [blame] | 17 | #include "content/public/browser/navigation_entry.h" |
[email protected] | 6acde635 | 2012-01-04 16:52:20 | [diff] [blame] | 18 | #include "content/public/browser/web_contents.h" |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 19 | |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 20 | typedef BrowserWithTestWindowTest BrowserCommandsTest; |
| 21 | |
[email protected] | e5d549d | 2011-12-28 01:29:20 | [diff] [blame] | 22 | using content::OpenURLParams; |
| 23 | using content::Referrer; |
[email protected] | 6acde635 | 2012-01-04 16:52:20 | [diff] [blame] | 24 | using content::WebContents; |
[email protected] | e5d549d | 2011-12-28 01:29:20 | [diff] [blame] | 25 | |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 26 | // Tests IDC_SELECT_TAB_0, IDC_SELECT_NEXT_TAB, IDC_SELECT_PREVIOUS_TAB and |
| 27 | // IDC_SELECT_LAST_TAB. |
[email protected] | 6df64208 | 2009-08-13 22:31:37 | [diff] [blame] | 28 | TEST_F(BrowserCommandsTest, TabNavigationAccelerators) { |
[email protected] | 081dc52 | 2013-05-15 04:59:20 | [diff] [blame] | 29 | GURL about_blank(content::kAboutBlankURL); |
[email protected] | 9423d941 | 2009-04-14 22:13:55 | [diff] [blame] | 30 | |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 31 | // Create three tabs. |
[email protected] | 9423d941 | 2009-04-14 22:13:55 | [diff] [blame] | 32 | AddTab(browser(), about_blank); |
| 33 | AddTab(browser(), about_blank); |
| 34 | AddTab(browser(), about_blank); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 35 | |
| 36 | // Select the second tab. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 37 | browser()->tab_strip_model()->ActivateTabAt(1, false); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 38 | |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 39 | CommandUpdater* updater = browser()->command_controller()->command_updater(); |
| 40 | |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 41 | // Navigate to the first tab using an accelerator. |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 42 | updater->ExecuteCommand(IDC_SELECT_TAB_0); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 43 | ASSERT_EQ(0, browser()->tab_strip_model()->active_index()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 44 | |
| 45 | // Navigate to the second tab using the next accelerators. |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 46 | updater->ExecuteCommand(IDC_SELECT_NEXT_TAB); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 47 | ASSERT_EQ(1, browser()->tab_strip_model()->active_index()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 48 | |
| 49 | // Navigate back to the first tab using the previous accelerators. |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 50 | updater->ExecuteCommand(IDC_SELECT_PREVIOUS_TAB); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 51 | ASSERT_EQ(0, browser()->tab_strip_model()->active_index()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 52 | |
| 53 | // Navigate to the last tab using the select last accelerator. |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 54 | updater->ExecuteCommand(IDC_SELECT_LAST_TAB); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 55 | ASSERT_EQ(2, browser()->tab_strip_model()->active_index()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 56 | } |
| 57 | |
| 58 | // Tests IDC_DUPLICATE_TAB. |
| 59 | TEST_F(BrowserCommandsTest, DuplicateTab) { |
[email protected] | 9423d941 | 2009-04-14 22:13:55 | [diff] [blame] | 60 | GURL url1("http://foo/1"); |
| 61 | GURL url2("http://foo/2"); |
| 62 | GURL url3("http://foo/3"); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 63 | GURL url4("http://foo/4"); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 64 | |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 65 | // Navigate to three urls, plus a pending URL that hasn't committed. |
[email protected] | 9423d941 | 2009-04-14 22:13:55 | [diff] [blame] | 66 | AddTab(browser(), url1); |
[email protected] | 0683cf6 | 2009-04-10 19:31:40 | [diff] [blame] | 67 | NavigateAndCommitActiveTab(url2); |
| 68 | NavigateAndCommitActiveTab(url3); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 69 | content::NavigationController& orig_controller = |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 70 | browser()->tab_strip_model()->GetWebContentsAt(0)->GetController(); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 71 | orig_controller.LoadURL( |
| 72 | url4, content::Referrer(), content::PAGE_TRANSITION_LINK, std::string()); |
| 73 | EXPECT_EQ(3, orig_controller.GetEntryCount()); |
| 74 | EXPECT_TRUE(orig_controller.GetPendingEntry()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 75 | |
[email protected] | 0665ebe | 2013-02-13 09:53:19 | [diff] [blame] | 76 | size_t initial_window_count = chrome::GetTotalBrowserCount(); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 77 | |
| 78 | // Duplicate the tab. |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 79 | chrome::ExecuteCommand(browser(), IDC_DUPLICATE_TAB); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 80 | |
| 81 | // The duplicated tab should not end up in a new window. |
[email protected] | 0665ebe | 2013-02-13 09:53:19 | [diff] [blame] | 82 | size_t window_count = chrome::GetTotalBrowserCount(); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 83 | ASSERT_EQ(initial_window_count, window_count); |
| 84 | |
| 85 | // And we should have a newly duplicated tab. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 86 | ASSERT_EQ(2, browser()->tab_strip_model()->count()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 87 | |
| 88 | // Verify the stack of urls. |
[email protected] | cdcb1dee | 2012-01-04 00:46:20 | [diff] [blame] | 89 | content::NavigationController& controller = |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 90 | browser()->tab_strip_model()->GetWebContentsAt(1)->GetController(); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 91 | EXPECT_EQ(3, controller.GetEntryCount()); |
| 92 | EXPECT_EQ(2, controller.GetCurrentEntryIndex()); |
| 93 | EXPECT_EQ(url1, controller.GetEntryAtIndex(0)->GetURL()); |
| 94 | EXPECT_EQ(url2, controller.GetEntryAtIndex(1)->GetURL()); |
| 95 | EXPECT_EQ(url3, controller.GetEntryAtIndex(2)->GetURL()); |
| 96 | EXPECT_FALSE(controller.GetPendingEntry()); |
| 97 | } |
| 98 | |
| 99 | // Tests IDC_VIEW_SOURCE (See http://crbug.com/138140). |
| 100 | TEST_F(BrowserCommandsTest, ViewSource) { |
| 101 | GURL url1("http://foo/1"); |
| 102 | GURL url2("http://foo/2"); |
| 103 | |
| 104 | // Navigate to a URL, plus a pending URL that hasn't committed. |
| 105 | AddTab(browser(), url1); |
| 106 | content::NavigationController& orig_controller = |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 107 | browser()->tab_strip_model()->GetWebContentsAt(0)->GetController(); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 108 | orig_controller.LoadURL( |
| 109 | url2, content::Referrer(), content::PAGE_TRANSITION_LINK, std::string()); |
| 110 | EXPECT_EQ(1, orig_controller.GetEntryCount()); |
| 111 | EXPECT_TRUE(orig_controller.GetPendingEntry()); |
| 112 | |
[email protected] | 0665ebe | 2013-02-13 09:53:19 | [diff] [blame] | 113 | size_t initial_window_count = chrome::GetTotalBrowserCount(); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 114 | |
| 115 | // View Source. |
| 116 | chrome::ExecuteCommand(browser(), IDC_VIEW_SOURCE); |
| 117 | |
| 118 | // The view source tab should not end up in a new window. |
[email protected] | 0665ebe | 2013-02-13 09:53:19 | [diff] [blame] | 119 | size_t window_count = chrome::GetTotalBrowserCount(); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 120 | ASSERT_EQ(initial_window_count, window_count); |
| 121 | |
| 122 | // And we should have a newly duplicated tab. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 123 | ASSERT_EQ(2, browser()->tab_strip_model()->count()); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 124 | |
| 125 | // Verify we are viewing the source of the last committed entry. |
| 126 | GURL view_source_url("view-source:http://foo/1"); |
| 127 | content::NavigationController& controller = |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 128 | browser()->tab_strip_model()->GetWebContentsAt(1)->GetController(); |
[email protected] | 8fcc7bf | 2012-08-14 18:14:50 | [diff] [blame] | 129 | EXPECT_EQ(1, controller.GetEntryCount()); |
| 130 | EXPECT_EQ(0, controller.GetCurrentEntryIndex()); |
| 131 | EXPECT_EQ(url1, controller.GetEntryAtIndex(0)->GetURL()); |
| 132 | EXPECT_EQ(view_source_url, controller.GetEntryAtIndex(0)->GetVirtualURL()); |
| 133 | EXPECT_FALSE(controller.GetPendingEntry()); |
[email protected] | be3877f | 2009-01-14 15:51:10 | [diff] [blame] | 134 | } |
[email protected] | b7ca4e6 | 2009-01-23 20:37:29 | [diff] [blame] | 135 | |
| 136 | TEST_F(BrowserCommandsTest, BookmarkCurrentPage) { |
| 137 | // We use profile() here, since it's a TestingProfile. |
| 138 | profile()->CreateBookmarkModel(true); |
[email protected] | 1f9c7419 | 2013-03-25 23:04:35 | [diff] [blame] | 139 | |
| 140 | BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile()); |
| 141 | ui_test_utils::WaitForBookmarkModelToLoad(model); |
[email protected] | b7ca4e6 | 2009-01-23 20:37:29 | [diff] [blame] | 142 | |
| 143 | // Navigate to a url. |
[email protected] | 9423d941 | 2009-04-14 22:13:55 | [diff] [blame] | 144 | GURL url1("http://foo/1"); |
| 145 | AddTab(browser(), url1); |
[email protected] | e5d549d | 2011-12-28 01:29:20 | [diff] [blame] | 146 | browser()->OpenURL(OpenURLParams( |
| 147 | url1, Referrer(), CURRENT_TAB, content::PAGE_TRANSITION_TYPED, false)); |
[email protected] | b7ca4e6 | 2009-01-23 20:37:29 | [diff] [blame] | 148 | |
[email protected] | 5d9829491 | 2012-06-27 22:57:40 | [diff] [blame] | 149 | chrome::BookmarkCurrentPage(browser()); |
[email protected] | b7ca4e6 | 2009-01-23 20:37:29 | [diff] [blame] | 150 | |
| 151 | // It should now be bookmarked in the bookmark model. |
| 152 | EXPECT_EQ(profile(), browser()->profile()); |
[email protected] | 1f9c7419 | 2013-03-25 23:04:35 | [diff] [blame] | 153 | EXPECT_TRUE(model->IsBookmarked(url1)); |
[email protected] | b7ca4e6 | 2009-01-23 20:37:29 | [diff] [blame] | 154 | } |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 155 | |
| 156 | // Tests back/forward in new tab (Control + Back/Forward button in the UI). |
| 157 | TEST_F(BrowserCommandsTest, BackForwardInNewTab) { |
| 158 | GURL url1("http://foo/1"); |
| 159 | GURL url2("http://foo/2"); |
| 160 | |
| 161 | // Make a tab with the two pages navigated in it. |
| 162 | AddTab(browser(), url1); |
| 163 | NavigateAndCommitActiveTab(url2); |
| 164 | |
| 165 | // Go back in a new background tab. |
[email protected] | a37d4b0 | 2012-06-25 21:56:10 | [diff] [blame] | 166 | chrome::GoBack(browser(), NEW_BACKGROUND_TAB); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 167 | EXPECT_EQ(0, browser()->tab_strip_model()->active_index()); |
| 168 | ASSERT_EQ(2, browser()->tab_strip_model()->count()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 169 | |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 170 | WebContents* zeroth = browser()->tab_strip_model()->GetWebContentsAt(0); |
[email protected] | e022aca | 2013-07-18 19:10:18 | [diff] [blame^] | 171 | WebContents* first = browser()->tab_strip_model()->GetWebContentsAt(1); |
| 172 | |
| 173 | // The original tab should be unchanged. |
| 174 | EXPECT_EQ(url2, zeroth->GetLastCommittedURL()); |
[email protected] | f5fa20e | 2011-12-21 22:35:56 | [diff] [blame] | 175 | EXPECT_TRUE(zeroth->GetController().CanGoBack()); |
| 176 | EXPECT_FALSE(zeroth->GetController().CanGoForward()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 177 | |
[email protected] | e022aca | 2013-07-18 19:10:18 | [diff] [blame^] | 178 | // The new tab should be like the first one but navigated back. Since we |
| 179 | // didn't wait for the load to complete, we can't use GetLastCommittedURL. |
| 180 | EXPECT_EQ(url1, first->GetActiveURL()); |
[email protected] | f5fa20e | 2011-12-21 22:35:56 | [diff] [blame] | 181 | EXPECT_FALSE(first->GetController().CanGoBack()); |
| 182 | EXPECT_TRUE(first->GetController().CanGoForward()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 183 | |
| 184 | // Select the second tab and make it go forward in a new background tab. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 185 | browser()->tab_strip_model()->ActivateTabAt(1, true); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 186 | // TODO(brettw) bug 11055: It should not be necessary to commit the load here, |
| 187 | // but because of this bug, it will assert later if we don't. When the bug is |
| 188 | // fixed, one of the three commits here related to this bug should be removed |
| 189 | // (to test both codepaths). |
[email protected] | f5fa20e | 2011-12-21 22:35:56 | [diff] [blame] | 190 | CommitPendingLoad(&first->GetController()); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 191 | EXPECT_EQ(1, browser()->tab_strip_model()->active_index()); |
[email protected] | a37d4b0 | 2012-06-25 21:56:10 | [diff] [blame] | 192 | chrome::GoForward(browser(), NEW_BACKGROUND_TAB); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 193 | |
| 194 | // The previous tab should be unchanged and still in the foreground. |
[email protected] | e022aca | 2013-07-18 19:10:18 | [diff] [blame^] | 195 | EXPECT_EQ(url1, first->GetLastCommittedURL()); |
[email protected] | f5fa20e | 2011-12-21 22:35:56 | [diff] [blame] | 196 | EXPECT_FALSE(first->GetController().CanGoBack()); |
| 197 | EXPECT_TRUE(first->GetController().CanGoForward()); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 198 | EXPECT_EQ(1, browser()->tab_strip_model()->active_index()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 199 | |
| 200 | // There should be a new tab navigated forward. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 201 | ASSERT_EQ(3, browser()->tab_strip_model()->count()); |
| 202 | WebContents* second = browser()->tab_strip_model()->GetWebContentsAt(2); |
[email protected] | e022aca | 2013-07-18 19:10:18 | [diff] [blame^] | 203 | // Since we didn't wait for load to complete, we can't use |
| 204 | // GetLastCommittedURL. |
| 205 | EXPECT_EQ(url2, second->GetActiveURL()); |
[email protected] | f5fa20e | 2011-12-21 22:35:56 | [diff] [blame] | 206 | EXPECT_TRUE(second->GetController().CanGoBack()); |
| 207 | EXPECT_FALSE(second->GetController().CanGoForward()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 208 | |
| 209 | // Now do back in a new foreground tab. Don't bother re-checking every sngle |
| 210 | // thing above, just validate that it's opening properly. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 211 | browser()->tab_strip_model()->ActivateTabAt(2, true); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 212 | // TODO(brettw) bug 11055: see the comment above about why we need this. |
[email protected] | f5fa20e | 2011-12-21 22:35:56 | [diff] [blame] | 213 | CommitPendingLoad(&second->GetController()); |
[email protected] | a37d4b0 | 2012-06-25 21:56:10 | [diff] [blame] | 214 | chrome::GoBack(browser(), NEW_FOREGROUND_TAB); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 215 | ASSERT_EQ(3, browser()->tab_strip_model()->active_index()); |
| 216 | ASSERT_EQ(url1, |
[email protected] | e022aca | 2013-07-18 19:10:18 | [diff] [blame^] | 217 | browser()->tab_strip_model()->GetActiveWebContents()-> |
| 218 | GetActiveURL()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 219 | |
| 220 | // Same thing again for forward. |
| 221 | // TODO(brettw) bug 11055: see the comment above about why we need this. |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 222 | CommitPendingLoad(& |
| 223 | browser()->tab_strip_model()->GetActiveWebContents()->GetController()); |
[email protected] | a37d4b0 | 2012-06-25 21:56:10 | [diff] [blame] | 224 | chrome::GoForward(browser(), NEW_FOREGROUND_TAB); |
[email protected] | 5789218 | 2012-12-03 19:15:39 | [diff] [blame] | 225 | ASSERT_EQ(4, browser()->tab_strip_model()->active_index()); |
| 226 | ASSERT_EQ(url2, |
[email protected] | e022aca | 2013-07-18 19:10:18 | [diff] [blame^] | 227 | browser()->tab_strip_model()->GetActiveWebContents()-> |
| 228 | GetActiveURL()); |
[email protected] | e21e8c9 | 2009-04-29 02:42:09 | [diff] [blame] | 229 | } |