diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-08 23:20:20 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-08 23:20:20 +0000 |
| commit | a206b446481a9d522958a98e03912aba8c02bb4b (patch) | |
| tree | ef94c7d31020aaa871117fa1d23b352db34f0940 /chrome/browser/browser_commands_unittest.cc | |
| parent | 96a41a0ec6889b5923a92454eafaf39fd63277cf (diff) | |
| download | chromium_src-a206b446481a9d522958a98e03912aba8c02bb4b.zip chromium_src-a206b446481a9d522958a98e03912aba8c02bb4b.tar.gz chromium_src-a206b446481a9d522958a98e03912aba8c02bb4b.tar.bz2 | |
* Add ctrl-shift-d to bookmark all tabs.
* Enable or disable bookmark commands based on whether we're actually going to be able to bookmark anything.
* Rename IDC_STAR to IDC_BOOKMARK_PAGE, which is more obvious.
BUG=2935
TEST=Hit ctrl-shift-d, make sure it bookmarks all tabs
Review URL: http://codereview.chromium.org/266029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28480 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_commands_unittest.cc')
| -rw-r--r-- | chrome/browser/browser_commands_unittest.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/chrome/browser/browser_commands_unittest.cc b/chrome/browser/browser_commands_unittest.cc index 768edd5..144263b 100644 --- a/chrome/browser/browser_commands_unittest.cc +++ b/chrome/browser/browser_commands_unittest.cc @@ -90,10 +90,10 @@ TEST_F(BrowserCommandsTest, BookmarkCurrentPage) { // TODO(beng): remove this once we can use TabContentses directly in testing // instead of the TestTabContents which causes this command not to // be enabled when the tab is added (and selected). - browser()->command_updater()->UpdateCommandEnabled(IDC_STAR, true); + browser()->command_updater()->UpdateCommandEnabled(IDC_BOOKMARK_PAGE, true); // Star it. - browser()->ExecuteCommand(IDC_STAR); + browser()->ExecuteCommand(IDC_BOOKMARK_PAGE); // It should now be bookmarked in the bookmark model. EXPECT_EQ(profile(), browser()->profile()); |