summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/views/toolbar_view.cc
diff options
context:
space:
mode:
authorxiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-05-10 16:35:54 +0000
committerxiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-05-10 16:35:54 +0000
commit658e883290c87808964c8fe245a4755c67b98eab (patch)
treefd2e461e2c189a6542d1c5780d0b6378c2b8399c /chrome/browser/ui/views/toolbar_view.cc
parent78750dd0542424fdca642c03bbed8841f1d129e0 (diff)
downloadchromium_src-658e883290c87808964c8fe245a4755c67b98eab.zip
chromium_src-658e883290c87808964c8fe245a4755c67b98eab.tar.gz
chromium_src-658e883290c87808964c8fe245a4755c67b98eab.tar.bz2
[ChromeOS] Move help and feedback button into wrench menu.
- Remove feedback button from toolbar; - Remove feedback entry in "Tools" submenu and put it into top-tier wrench menu for ChromeOS; - Remove help app icon by removing "brwoser_action" from its manifest; - Add an icon for help menu; BUG=chromium-os:11435 TEST=Verify fix for chromium-os:11435 Review URL: http://codereview.chromium.org/6992001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84793 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/views/toolbar_view.cc')
-rw-r--r--chrome/browser/ui/views/toolbar_view.cc43
1 files changed, 0 insertions, 43 deletions
diff --git a/chrome/browser/ui/views/toolbar_view.cc b/chrome/browser/ui/views/toolbar_view.cc
index 58d35f5..3e62081 100644
--- a/chrome/browser/ui/views/toolbar_view.cc
+++ b/chrome/browser/ui/views/toolbar_view.cc
@@ -86,9 +86,6 @@ ToolbarView::ToolbarView(Browser* browser)
back_(NULL),
forward_(NULL),
reload_(NULL),
-#if defined(OS_CHROMEOS)
- feedback_(NULL),
-#endif
home_(NULL),
location_bar_(NULL),
browser_actions_(NULL),
@@ -166,17 +163,6 @@ void ToolbarView::Init(Profile* profile) {
reload_->SetAccessibleName(l10n_util::GetStringUTF16(IDS_ACCNAME_RELOAD));
reload_->SetID(VIEW_ID_RELOAD_BUTTON);
-#if defined(OS_CHROMEOS)
- feedback_ = new views::ImageButton(this);
- feedback_->set_tag(IDC_FEEDBACK);
- feedback_->set_triggerable_event_flags(ui::EF_LEFT_BUTTON_DOWN |
- ui::EF_MIDDLE_BUTTON_DOWN);
- feedback_->set_tag(IDC_FEEDBACK);
- feedback_->SetTooltipText(
- UTF16ToWide(l10n_util::GetStringUTF16(IDS_TOOLTIP_FEEDBACK)));
- feedback_->SetID(VIEW_ID_FEEDBACK_BUTTON);
-#endif
-
home_ = new views::ImageButton(this);
home_->set_triggerable_event_flags(ui::EF_LEFT_BUTTON_DOWN |
ui::EF_MIDDLE_BUTTON_DOWN);
@@ -210,9 +196,6 @@ void ToolbarView::Init(Profile* profile) {
AddChildView(home_);
AddChildView(location_bar_);
AddChildView(browser_actions_);
-#if defined(OS_CHROMEOS)
- AddChildView(feedback_);
-#endif
AddChildView(app_menu_);
location_bar_->Init();
@@ -426,9 +409,6 @@ gfx::Size ToolbarView::GetPreferredSize() {
(show_home_button_.GetValue() ?
(home_->GetPreferredSize().width() + kButtonSpacing) : 0) +
browser_actions_->GetPreferredSize().width() +
-#if defined(OS_CHROMEOS)
- feedback_->GetPreferredSize().width() + kButtonSpacing +
-#endif
app_menu_->GetPreferredSize().width() + kEdgeSpacing;
static SkBitmap normal_background;
@@ -495,16 +475,9 @@ void ToolbarView::Layout() {
}
int browser_actions_width = browser_actions_->GetPreferredSize().width();
-#if defined(OS_CHROMEOS)
- int feedback_menu_width = feedback_->GetPreferredSize().width() +
- kButtonSpacing;
-#endif
int app_menu_width = app_menu_->GetPreferredSize().width();
int location_x = home_->x() + home_->width() + kStandardSpacing;
int available_width = width() - kEdgeSpacing - app_menu_width -
-#if defined(OS_CHROMEOS)
- feedback_menu_width -
-#endif
browser_actions_width - location_x;
location_bar_->SetBounds(location_x, child_y, std::max(available_width, 0),
@@ -525,15 +498,8 @@ void ToolbarView::Layout() {
// we extend the back button to the left edge.
if (maximized)
app_menu_width += kEdgeSpacing;
-#if defined(OS_CHROMEOS)
- feedback_->SetBounds(browser_actions_->x() + browser_actions_width, child_y,
- feedback_->GetPreferredSize().width(), child_height);
- app_menu_->SetBounds(feedback_->x() + feedback_->width() + kButtonSpacing,
- child_y, app_menu_width, child_height);
-#else
app_menu_->SetBounds(browser_actions_->x() + browser_actions_width, child_y,
app_menu_width, child_height);
-#endif
}
void ToolbarView::OnPaint(gfx::Canvas* canvas) {
@@ -673,15 +639,6 @@ void ToolbarView::LoadImages() {
reload_->SetToggledImage(views::CustomButton::BS_DISABLED,
tp->GetBitmapNamed(IDR_STOP_D));
-#if defined(OS_CHROMEOS)
- feedback_->SetImage(views::CustomButton::BS_NORMAL,
- tp->GetBitmapNamed(IDR_FEEDBACK));
- feedback_->SetImage(views::CustomButton::BS_HOT,
- tp->GetBitmapNamed(IDR_FEEDBACK_H));
- feedback_->SetImage(views::CustomButton::BS_PUSHED,
- tp->GetBitmapNamed(IDR_FEEDBACK_P));
-#endif
-
home_->SetImage(views::CustomButton::BS_NORMAL, tp->GetBitmapNamed(IDR_HOME));
home_->SetImage(views::CustomButton::BS_HOT, tp->GetBitmapNamed(IDR_HOME_H));
home_->SetImage(views::CustomButton::BS_PUSHED,