summaryrefslogtreecommitdiffstats
path: root/chrome/browser/ui/gtk/custom_button.cc
diff options
context:
space:
mode:
authorflackr@chromium.org <flackr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-03-25 17:54:36 +0000
committerflackr@chromium.org <flackr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-03-25 17:54:36 +0000
commit0bf53d21ec308f35a8adcaf0d093a4959deae5c1 (patch)
tree6c8d54b4425fc58392106036f4aeebf2700f59ea /chrome/browser/ui/gtk/custom_button.cc
parentdd204dc2d0055e511cba23bae525829156058ab4 (diff)
downloadchromium_src-0bf53d21ec308f35a8adcaf0d093a4959deae5c1.zip
chromium_src-0bf53d21ec308f35a8adcaf0d093a4959deae5c1.tar.gz
chromium_src-0bf53d21ec308f35a8adcaf0d093a4959deae5c1.tar.bz2
Revert 190431 "Inconsistent use of [x] close panel icon"
> Inconsistent use of [x] close panel icon > The close button in download pane should be the tab close button asset : tab_close_*.png > (this was originally reported as part of BUG=173251). > > By the same logic the same smaller bitmap with red hover background should be used for all infobars > such as translate bar, save password bar, etc. It should also be used for find-in-page bar on all > platforms (Mac was missed with the previous implementation). > > The close button in the chrome UI and Web UI bubbles was left the same as before (this needs to be regression-tested). > > Code was rearranged to make explicit the disrepancy between the IDR_CLOSE_BAR and IDR_CLOSE_BUBBLE. > IDR_TAB_CLOSE was renamed IDR_CLOSE_1. > IDR_CLOSE_BAR was renamed IDR_CLOSE_2. > png files were also renamed. > > BUG=217094 > TEST=Open any page that downloads an attachment such as gmail email with an attachment or a page with images. Download an image or an attachment. Verify that the close button in download bar at the bottom of the screen has a red circular background when mouse hovers over it - same as the close tab button. > > TEST=Open find-in-page (Ctrl+F) box on Mac, Linux and Windows and check that the close button is same as tab close (has a red circular background when mouse hovers over it). > > TEST=Open any infobar (translate / save password) on both ChromeOS and linux and check that the close button is same as tab close (has a red circular background when mouse hovers over it). Opening a non-English news page is usually the easiest way. > > TEST=Trigger chrome to show a toolbar bubble such as extension installed bubble that shows when an extension is installed from chrome web store (see bitmaps in comments attached to BUG=217094). Confirm that the close buttons in those bubbles are NOT changed from before the change (they should NOT have the red circular background when mouse hovers over). > > Review URL: https://codereview.chromium.org/12634025 TBR=flackr@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@190432 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/gtk/custom_button.cc')
-rw-r--r--chrome/browser/ui/gtk/custom_button.cc16
1 files changed, 3 insertions, 13 deletions
diff --git a/chrome/browser/ui/gtk/custom_button.cc b/chrome/browser/ui/gtk/custom_button.cc
index 0c7ce85..2b41351 100644
--- a/chrome/browser/ui/gtk/custom_button.cc
+++ b/chrome/browser/ui/gtk/custom_button.cc
@@ -355,20 +355,10 @@ gboolean CustomDrawButton::OnCustomExpose(GtkWidget* sender,
}
// static
-CustomDrawButton* CustomDrawButton::CloseButtonBar(
+CustomDrawButton* CustomDrawButton::CloseButton(
GtkThemeService* theme_provider) {
- CustomDrawButton* button = new CustomDrawButton(theme_provider,
- IDR_CLOSE_1, IDR_CLOSE_1_P, IDR_CLOSE_1_H, 0,
- GTK_STOCK_CLOSE, GTK_ICON_SIZE_MENU);
- return button;
-}
-
-// static
-CustomDrawButton* CustomDrawButton::CloseButtonBubble(
- GtkThemeService* theme_provider) {
- CustomDrawButton* button = new CustomDrawButton(theme_provider,
- IDR_CLOSE_2, IDR_CLOSE_2_P, IDR_CLOSE_2_H, 0,
- GTK_STOCK_CLOSE, GTK_ICON_SIZE_MENU);
+ CustomDrawButton* button = new CustomDrawButton(theme_provider, IDR_CLOSE_BAR,
+ IDR_CLOSE_BAR_P, IDR_CLOSE_BAR_H, 0, GTK_STOCK_CLOSE, GTK_ICON_SIZE_MENU);
return button;
}