diff options
author | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-10 16:35:54 +0000 |
---|---|---|
committer | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-05-10 16:35:54 +0000 |
commit | 658e883290c87808964c8fe245a4755c67b98eab (patch) | |
tree | fd2e461e2c189a6542d1c5780d0b6378c2b8399c /chrome/browser/ui/views/toolbar_view.cc | |
parent | 78750dd0542424fdca642c03bbed8841f1d129e0 (diff) | |
download | chromium_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.cc | 43 |
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, |