From a578606f5e15b4db72928f955995e71a92b683f9 Mon Sep 17 00:00:00 2001 From: "alekseys@chromium.org" Date: Thu, 9 Sep 2010 18:11:57 +0000 Subject: 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. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58964 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/app/nibs/BrowserWindow.xib | 108 ++++++++++++++++++++++++++++++++++---- chrome/app/nibs/TabContents.xib | 102 ++++------------------------------- 2 files changed, 109 insertions(+), 101 deletions(-) (limited to 'chrome/app/nibs') diff --git a/chrome/app/nibs/BrowserWindow.xib b/chrome/app/nibs/BrowserWindow.xib index 30b1c8f..022c2b2 100644 --- a/chrome/app/nibs/BrowserWindow.xib +++ b/chrome/app/nibs/BrowserWindow.xib @@ -12,6 +12,7 @@ YES + YES @@ -48,19 +49,41 @@ {1.79769e+308, 1.79769e+308} {400, 250} - + 256 YES 274 - {750, 586} + + YES + + + 274 + + YES + + + 268 + {750, 600} + + YES + 2 + + + {750, 600} + + 2 + + + {750, 600} FastResizeView {750, 600} + {{0, 0}, {1440, 878}} {400, 272} @@ -157,6 +180,22 @@ 90 + + + devToolsContainer_ + + + + 94 + + + + contentsContainer_ + + + + 98 + @@ -229,14 +268,32 @@ + 89 + + + + 84 + + YES + + - 89 - - + 91 + + + YES + + + + + + 95 + + @@ -268,15 +325,17 @@ 84.IBPluginDependency 89.IBEditorWindowLastContentRect 89.IBPluginDependency + 91.IBPluginDependency + 95.IBPluginDependency YES com.apple.InterfaceBuilder.CocoaPlugin com.apple.InterfaceBuilder.CocoaPlugin com.apple.InterfaceBuilder.CocoaPlugin - {{267, 167}, {750, 600}} + {{319, 325}, {750, 600}} com.apple.InterfaceBuilder.CocoaPlugin - {{267, 167}, {750, 600}} + {{319, 325}, {750, 600}} {196, 240} {{357, 418}, {480, 270}} @@ -298,6 +357,8 @@ com.apple.InterfaceBuilder.CocoaPlugin {{89, 611}, {236, 393}} com.apple.InterfaceBuilder.CocoaPlugin + com.apple.InterfaceBuilder.CocoaPlugin + com.apple.InterfaceBuilder.CocoaPlugin @@ -316,7 +377,7 @@ - 90 + 98 @@ -337,6 +398,14 @@ + ChromeBrowserWindow + ChromeEventProcessingWindow + + IBProjectSource + browser/cocoa/chrome_browser_window.h + + + ChromeEventProcessingWindow NSWindow @@ -373,7 +442,7 @@ FramedBrowserWindow - ChromeEventProcessingWindow + ChromeBrowserWindow IBProjectSource browser/cocoa/framed_browser_window.h @@ -467,12 +536,16 @@ YES YES + contentsContainer_ + devToolsContainer_ sideTabStripView_ tabContentArea_ topTabStripView_ YES + NSSplitView + NSSplitView TabStripView FastResizeView TabStripView @@ -482,6 +555,8 @@ YES YES + contentsContainer_ + devToolsContainer_ sideTabStripView_ tabContentArea_ topTabStripView_ @@ -489,6 +564,14 @@ YES + contentsContainer_ + NSSplitView + + + devToolsContainer_ + NSSplitView + + sideTabStripView_ TabStripView @@ -785,7 +868,7 @@ NSObject - + IBFrameworkSource AppKit.framework/Headers/NSSplitView.h @@ -1114,6 +1197,11 @@ + NSSplitView + NSView + + + NSView IBFrameworkSource diff --git a/chrome/app/nibs/TabContents.xib b/chrome/app/nibs/TabContents.xib index d93dcbb..2419109 100644 --- a/chrome/app/nibs/TabContents.xib +++ b/chrome/app/nibs/TabContents.xib @@ -12,7 +12,7 @@ YES - + YES @@ -57,27 +57,6 @@ 274 - - YES - - - 274 - - YES - - - 268 - {480, 360} - - YES - 2 - - - {480, 360} - - 2 - - {480, 360} @@ -97,22 +76,6 @@ 156 - - - devToolsContainer_ - - - - 212 - - - - contentsContainer_ - - - - 216 - @@ -150,7 +113,6 @@ YES - @@ -160,23 +122,6 @@ Application - - 195 - - - YES - - - - - - 213 - - - YES - - - @@ -184,8 +129,6 @@ YES -3.IBPluginDependency - 195.IBPluginDependency - 213.IBPluginDependency 93.IBEditorWindowLastContentRect 93.IBPluginDependency 93.IBViewEditorWindowController.showingLayoutRectangles @@ -199,12 +142,10 @@ YES com.apple.InterfaceBuilder.CocoaPlugin - com.apple.InterfaceBuilder.CocoaPlugin - com.apple.InterfaceBuilder.CocoaPlugin - {{529, 843}, {480, 360}} + {{1428, 763}, {480, 360}} com.apple.InterfaceBuilder.CocoaPlugin - {{529, 843}, {480, 360}} + {{1428, 763}, {480, 360}} {480, 360} @@ -228,7 +169,7 @@ - 216 + 221 @@ -291,7 +232,7 @@ NSWindow IBProjectSource - browser/cocoa/chrome_browser_window.h + browser/cocoa/framed_browser_window.h @@ -305,35 +246,14 @@ TabContentsController NSViewController - YES - - YES - contentsContainer_ - devToolsContainer_ - - - YES - NSSplitView - NSSplitView - + contentsContainer_ + NSSplitView - YES - - YES - contentsContainer_ - devToolsContainer_ - - - YES - - contentsContainer_ - NSSplitView - - - devToolsContainer_ - NSSplitView - + contentsContainer_ + + contentsContainer_ + NSSplitView -- cgit v1.1