Linux: really hides the tab close button when it should not be painted.

Linux: really hides the tab close button when it should not be painted.

[James Hawkins, 2009-05-14 17:19:21]

http://codereview.chromium.org/113161/diff/3/4
File chrome/browser/gtk/tabs/tab_gtk.cc (right):

http://codereview.chromium.org/113161/diff/3/4#newcode234
Line 234: if (bounds.width() > 0 && bounds.height() > 0) {
Should be (!bounds.IsEmpty()).

[Vincent Zanotti, 2009-05-14 19:05:23]

http://codereview.chromium.org/113161/diff/3/4
File chrome/browser/gtk/tabs/tab_gtk.cc (right):

http://codereview.chromium.org/113161/diff/3/4#newcode234
Line 234: if (bounds.width() > 0 && bounds.height() > 0) {
On 2009/05/14 17:19:21, James Hawkins wrote:
> Should be (!bounds.IsEmpty()).

Done.

[James Hawkins, 2009-05-18 21:00:09]

On 2009/05/14 19:05:23, Vincent Zanotti wrote:
> http://codereview.chromium.org/113161/diff/3/4
> File chrome/browser/gtk/tabs/tab_gtk.cc (right):
> 
> http://codereview.chromium.org/113161/diff/3/4#newcode234
> Line 234: if (bounds.width() > 0 && bounds.height() > 0) {
> On 2009/05/14 17:19:21, James Hawkins wrote:
> > Should be (!bounds.IsEmpty()).
> 
> Done.

LGTM. Will commit.

[James Hawkins, 2009-05-18 21:31:22]

On 2009/05/18 21:00:09, James Hawkins wrote:
> On 2009/05/14 19:05:23, Vincent Zanotti wrote:
> > http://codereview.chromium.org/113161/diff/3/4
> > File chrome/browser/gtk/tabs/tab_gtk.cc (right):
> > 
> > http://codereview.chromium.org/113161/diff/3/4#newcode234
> > Line 234: if (bounds.width() > 0 && bounds.height() > 0) {
> > On 2009/05/14 17:19:21, James Hawkins wrote:
> > > Should be (!bounds.IsEmpty()).
> > 
> > Done.
> 
> LGTM. Will commit.

http://codereview.chromium.org/113551