diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-17 20:37:42 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-17 20:37:42 +0000 |
| commit | 7133f3639754e0189d58b90c07835e8d165ce3e1 (patch) | |
| tree | 25571ce31beecdbd1d8e97feb8b5ec28de7a79b5 /chrome/browser | |
| parent | 5fffbcda3e27cb91ff3dbe4353592e74b4d176ed (diff) | |
| download | chromium_src-7133f3639754e0189d58b90c07835e8d165ce3e1.zip chromium_src-7133f3639754e0189d58b90c07835e8d165ce3e1.tar.gz chromium_src-7133f3639754e0189d58b90c07835e8d165ce3e1.tar.bz2 | |
Fixes bug where creating new tabs on linux would momentarily flash the
new tab on the left edge of the strip. We don't have this bug on
windows because the tab strip isn't painted until after the animation
progresses.
BUG=none
TEST=on linux create new tabs by way of control-t (or plus button),
make sure you don't get the new tab to momentarily appear on the
left edge.
Review URL: http://codereview.chromium.org/126240
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18650 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rwxr-xr-x | chrome/browser/gtk/tabs/tab_strip_gtk.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/browser/gtk/tabs/tab_strip_gtk.cc b/chrome/browser/gtk/tabs/tab_strip_gtk.cc index 89a8fdc..87d1d9d 100755 --- a/chrome/browser/gtk/tabs/tab_strip_gtk.cc +++ b/chrome/browser/gtk/tabs/tab_strip_gtk.cc @@ -681,6 +681,10 @@ void TabStripGtk::TabInsertedAt(TabContents* contents, // Don't animate the first tab; it looks weird. if (GetTabCount() > 1) { StartInsertTabAnimation(index); + // We added the tab at 0x0, we need to force an animation step otherwise + // if GTK paints before the animation event the tab is painted at 0x0 + // which is most likely not where it should be positioned. + active_animation_->AnimationProgressed(NULL); } else { Layout(); } |