summaryrefslogtreecommitdiffstats
path: root/chrome/browser/browser.cc
diff options
context:
space:
mode:
authorjcivelli@chromium.org <jcivelli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-05-14 21:29:26 +0000
committerjcivelli@chromium.org <jcivelli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-05-14 21:29:26 +0000
commitb7969209a311ca384eceacef7d67743f2ffd792d (patch)
treed64c91438914c09f4389c9edf99ff29f1a8c7dbf /chrome/browser/browser.cc
parent0805a9b30d8cf935a572fe093a74f6784c6e4689 (diff)
downloadchromium_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.cc27
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 {