diff options
author | jcivelli@chromium.org <jcivelli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-14 21:29:26 +0000 |
---|---|---|
committer | jcivelli@chromium.org <jcivelli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-14 21:29:26 +0000 |
commit | b7969209a311ca384eceacef7d67743f2ffd792d (patch) | |
tree | d64c91438914c09f4389c9edf99ff29f1a8c7dbf /chrome/browser/browser.cc | |
parent | 0805a9b30d8cf935a572fe093a74f6784c6e4689 (diff) | |
download | chromium_src-b7969209a311ca384eceacef7d67743f2ffd792d.zip chromium_src-b7969209a311ca384eceacef7d67743f2ffd792d.tar.gz chromium_src-b7969209a311ca384eceacef7d67743f2ffd792d.tar.bz2 |
Removing the app launcher button in the ChromeOS browser and making the app launcher pinned to the location bar.
BUG=3190
TEST=The app launcher icon on the top-left corner of the browser should
be gone. Opening a new tab by pressing the + button or typing
CTRL-T should bring the app launcher, it should be located on top
of the location bar.
Review URL: http://codereview.chromium.org/2060004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47314 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser.cc')
-rw-r--r-- | chrome/browser/browser.cc | 27 |
1 files changed, 22 insertions, 5 deletions
diff --git a/chrome/browser/browser.cc b/chrome/browser/browser.cc index 492ebfb..3d94ed5 100644 --- a/chrome/browser/browser.cc +++ b/chrome/browser/browser.cc @@ -111,6 +111,10 @@ #include "chrome/browser/cocoa/find_pasteboard.h" #endif +#if defined(OS_CHROMEOS) +#include "chrome/browser/views/app_launcher.h" +#endif + using base::TimeDelta; // How long we wait before updating the browser chrome while loading a page. @@ -1045,6 +1049,21 @@ void Browser::UpdateCommandsForFullscreenMode(bool is_fullscreen) { command_updater_.UpdateCommandEnabled(IDC_SHOW_PAGE_MENU, show_main_ui); } +bool Browser::OpenAppsPanelAsNewTab() { +#if defined(OS_CHROMEOS) || defined(OS_WIN) + CommandLine* command_line = CommandLine::ForCurrentProcess(); + if (command_line->HasSwitch(switches::kDisableAppsPanel) || + (!browser_defaults::kShowAppsPanelForNewTab && + !command_line->HasSwitch(switches::kAppsPanel))) { + return false; + } + AppLauncher::ShowForNewTab(this, std::string()); + return true; +#endif // OS_CHROMEOS || OS_WIN + + return false; +} + /////////////////////////////////////////////////////////////////////////////// // Browser, Assorted browser commands: @@ -1248,12 +1267,10 @@ void Browser::CloseWindow() { void Browser::NewTab() { UserMetrics::RecordAction(UserMetricsAction("NewTab"), profile_); -#if defined(OS_WIN) - if (CommandLine::ForCurrentProcess()->HasSwitch(switches::kAppsPanel)) { - AppLauncher::ShowForNewTab(this, std::string()); + + if (OpenAppsPanelAsNewTab()) return; - } -#endif + if (type() == TYPE_NORMAL) { AddBlankTab(true); } else { |