I looked at this a bit last week, and it's harder to fix than I expected, because we add/remove/resize views in 
many different Observe() methods, split between BWC, TabStripController, and InfobarContainerController.  
These all happen sequentially and in arbitrary order, so there may not be a good way to simply disable view 
layout until we're done with the tab switch.

I think I've seen something similar on Linux.  I'll see if I can make it happen on Windows.

Punting to Mstone-5.  We now correctly draw the web content anchored at the top left of the view, so this has 
gone from really awful to a flicker at the bottom of the screen.  Not nearly as bad anymore.

You may as well mark this fixed. It LGTM.

Open gmail in one tab and the NTP in another.  Turn off "Always show bookmarks bar," so you get the NTP-only 
version.  Now, when you switch from the NTP to gmail, you should see a flicker at the bottom of the screen.

I don't want to close this bug because I do still see the problem.  It's not all that noticeable anymore, so I don't 
think it's a high priority.  On the other hand, it would be nice to not trigger two unnecessary renderer resizes for 
each tab switch.

Updating labels:
Area-UI-Features replaces Area-BrowserUI

(No comment was entered for this change.)

(No comment was entered for this change.)

(No comment was entered for this change.)

 Issue 53776  has been merged into this issue.

(Dupe has the same analysis. I think the TabStripController should do this explicitly instead of having 3 independent observers. As a workaround, you could make sure the TabStripController is always the first of the 3 observers, then it gets the event first, but that's hacky.)

 Issue 54346  has been merged into this issue.

(No comment was entered for this change.)

Verified label updated by AutoAllocator, contact AmolK or KrisR for details

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

------------------------------------------------------------------------
r58964 | alekseys@chromium.org | Thu Sep 09 11:11:57 PDT 2010

Changed paths:
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/sidebar_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_window_controller.h?r1=58964&r2=58963&pathrev=58964
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/dev_tools_controller.h?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/infobar_container_controller_unittest.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/infobar_container_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_contents_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/nibs/BrowserWindow.xib?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_strip_controller_unittest.mm?r1=58964&r2=58963&pathrev=58964
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/sidebar_controller.h?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_strip_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_controller.h?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/side_tab_strip_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_window_controller.mm?r1=58964&r2=58963&pathrev=58964
 A http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/dev_tools_controller.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/app/nibs/TabContents.xib?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/infobar_container_controller.h?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_contents_controller.h?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_controller_private.mm?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/tab_strip_controller.h?r1=58964&r2=58963&pathrev=58964
 M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/chrome_browser.gypi?r1=58964&r2=58963&pathrev=58964

Move devtools and sidebar handling to BWC and ensure the order of layout adjustments
during tab change updates to eliminate UI flickering.

TabContents.xib changes: devtools and sidebar NSSplitViews removed (moved to BrowserWindow.xib),
contentsContainer_ view removed (now it's inserted at runtime).
BrowserWindow.xib chnages: devtools and sidebar NSSplitViews added into FastResizeView.

BUG=22717, 31633
TEST=Run unit_tests, browser_tests and interactive_ui_tests.

Open google.com in one tab (it's easy to spot content relayout on google.com), trigger an infobar
and content scroll bar on another tab (ru.wikipedia.org with translation infobar will do), switch
tabs back and forth. Content should not relayout, scroll bar should not "jump".
Open new tab and make sure bookmark bar is not always visible, open any site which triggers infobar
and big enough to show content scroll bar on another tab, switch tabs back and forth. Content should
not relayout, scroolbar should not "jump".
Repeat the tests above with sidebar opened on any of these tabs, page and sidebar content should not
relayout, sidebar should stay the same width, scrollbar should not "jump".
Repeat the tests above with devtools docked.

------------------------------------------------------------------------

Verified in 7.0.517.5 (Official Build 59018) dev

This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.

(No comment was entered for this change.)