(No comment was entered for this change.)

(No comment was entered for this change.)

My best guess from the screenshot is that this is for Linux?  I can't repro for OSX.

yes for linux, sorry forgot to mention that. it Ubu 10.04.1 Just updated this morning and got this issue. I tend to update once a week from the daily ppa. the more I look at it, the more it seems the toolbar is slightly over sized as well, but I don't have any recent screenies to compare it.

I should have thought of this sooner, I switched back to "use classic theme" and there is the button. The toolbar is also reduced to normal, but reload/stop button is having issues. Reload is a little messed up and when stop is active its drawing ontop of the reload button.

I've tried a number of gtk/icon themes, all with the same result when using gtk in chromium. last weeks build was fine.

(No comment was entered for this change.)

downgraded to 6.0.472.14 for comparison here is a screenshot.

	home-ico-there-chromium.png 363 KB   View   Download

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=55159 

------------------------------------------------------------------------
r55159 | estade@chromium.org | 2010-08-05 17:13:10 -0700 (Thu, 05 Aug 2010) | 6 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_toolbar_gtk.cc?r1=55159&r2=55158

[GTK] undo more toolbar destruction - unbreak GTK+ theme mode home button

BUG=51325
TEST=showing home button in gtk+ should work

Review URL: http://codereview.chromium.org/3083018
------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=55161 

------------------------------------------------------------------------
r55161 | estade@chromium.org | 2010-08-05 17:24:27 -0700 (Thu, 05 Aug 2010) | 9 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/472/src/chrome/browser/gtk/browser_toolbar_gtk.cc?r1=55161&r2=55160

Merge 55159 - [GTK] undo more toolbar destruction - unbreak GTK+ theme mode home button

BUG=51325
TEST=showing home button in gtk+ should work

Review URL: http://codereview.chromium.org/3083018

TBR=estade@chromium.org
Review URL: http://codereview.chromium.org/3041053
------------------------------------------------------------------------

Cool that rev. fixed the home button, but the omnibox is still over sized. Does a separate bug need to be filed?

Also the stop/reload button is still weird when using chrome's builtin decos, again separate bug??

hmmm, when I glanced at it I thought the large omnibox was due to your theme. I'll take a look.

Nope see my second screenie, same theme. I'm trying to get a screencast for chromium builin stop/reload issue but never done it before.

Its quick and dirty but here is a short screencast of what I mean.

chromium-buttons-1.ogv 906 KB   Download

Tony is also getting the oversize omnibox, if you need him to do any local debugging.

no, I found the problem. It was caused by swapping around the widget
hierarchy, disrupting some sizing code. Hopefully my change here:
http://codereview.chromium.org/3086021/show will make it more
future-proof. The best future-proofing would come from unit testing,
it's just that in this case, unit testing would be hard because of the
required interaction with the system setup.

The stop/reload issue arose because the old stop/reload code depended
on the fact that custom buttons didn't draw subwidgets, but I had to
add that capability to get the wrench menu right. Again, I'm not sure
how to add a unit test to avoid this sort of regression (unless we
added a pixel test, which obviously has its own problems).

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=55253 

------------------------------------------------------------------------
r55253 | estade@chromium.org | 2010-08-06 10:40:53 -0700 (Fri, 06 Aug 2010) | 9 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_toolbar_gtk.cc?r1=55253&r2=55252
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/reload_button_gtk.cc?r1=55253&r2=55252

GTK toolbar -

- fix vertical height in gtk theme mode
- fix reload button showing two images when switching from gtk to chrome theme mode

BUG=51325
TEST=1) switch from gtk to chrome theme mode, look at stop/reload 2) change desktop document font size from 10 to 18 in gtk theme mode, the omnibox should size appropriately for each font.

Review URL: http://codereview.chromium.org/3086021
------------------------------------------------------------------------

The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=55256 

------------------------------------------------------------------------
r55256 | estade@chromium.org | 2010-08-06 10:48:04 -0700 (Fri, 06 Aug 2010) | 12 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/472/src/chrome/browser/gtk/browser_toolbar_gtk.cc?r1=55256&r2=55255
   M http://src.chromium.org/viewvc/chrome/branches/472/src/chrome/browser/gtk/reload_button_gtk.cc?r1=55256&r2=55255

Merge 55253 - GTK toolbar -

- fix vertical height in gtk theme mode
- fix reload button showing two images when switching from gtk to chrome theme mode

BUG=51325
TEST=1) switch from gtk to chrome theme mode, look at stop/reload 2) change desktop document font size from 10 to 18 in gtk theme mode, the omnibox should size appropriately for each font.

Review URL: http://codereview.chromium.org/3086021

TBR=erg
Review URL: http://codereview.chromium.org/3097001
------------------------------------------------------------------------

 Issue 51406  has been merged into this issue.

(No comment was entered for this change.)

(No comment was entered for this change.)

 Issue 51489  has been merged into this issue.

 Issue 51443  has been merged into this issue.

 Issue 51492  has been merged into this issue.

more general title

 Issue 51605  has been merged into this issue.

 Issue 51649  has been merged into this issue.

 Issue 51867  has been merged into this issue.

Issue 52013 has been merged into this issue.

 Issue 51765  has been merged into this issue.

Chrome Version       : 6.0.486.0
URLs (if applicable) : N/A
Other browsers tested: N/A
<b>Add OK or FAIL after other browsers where you have tested this issue:</b>
<b>Safari 4:</b>
<b>Firefox 3.x:</b>
<b>IE 7:</b>
<b>IE 8:</b>

<b>What steps will reproduce the problem?</b>
1. open chromium
2. look at toolbar
<b>3.</b>

<b>What is the expected result?</b>
I expect to see back, forward, reload and the home button before the omnibox

<b>What happens instead?</b>
Home button icon is missing, but the button is still there when you mouse over the area. Its about 3 pixels wide.

<b>Please provide any additional information below. Attach a screenshot if</b>
<b>possible.</b>
see screenshot