Fixes the Web Inspector flashing white while resizing after
authortimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Aug 2008 22:36:30 +0000 (22:36 +0000)
committertimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 19 Aug 2008 22:36:30 +0000 (22:36 +0000)
        highlighting a page element. Calling disableScreenUpdatesUntilFlush
        when attaching and detaching the Inspector page highlight is bad,
        since the browser window might not flush again for a while. So screen
        updates could be disabled for long periods of time, causing backing store
        flashing while resizing. There is no need to call disableScreenUpdatesUntilFlush
        when attaching or detaching the child window.

        Reviewed by John Sullivan and Kevin McCullough.

        * WebInspector/WebNodeHighlight.m:
        (-[WebNodeHighlight attach]): Remove the call to disableScreenUpdatesUntilFlush.
        (-[WebNodeHighlight detach]): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@35850 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/mac/ChangeLog
WebKit/mac/WebInspector/WebNodeHighlight.m

index 90003c4..95e54ba 100644 (file)
@@ -1,5 +1,21 @@
 2008-08-19  Timothy Hatcher  <timothy@apple.com>
 
+        Fixes the Web Inspector flashing white while resizing after
+        highlighting a page element. Calling disableScreenUpdatesUntilFlush
+        when attaching and detaching the Inspector page highlight is bad,
+        since the browser window might not flush again for a while. So screen
+        updates could be disabled for long periods of time, causing backing store
+        flashing while resizing. There is no need to call disableScreenUpdatesUntilFlush
+        when attaching or detaching the child window.
+
+        Reviewed by John Sullivan and Kevin McCullough.
+
+        * WebInspector/WebNodeHighlight.m:
+        (-[WebNodeHighlight attach]): Remove the call to disableScreenUpdatesUntilFlush.
+        (-[WebNodeHighlight detach]): Ditto.
+
+2008-08-19  Timothy Hatcher  <timothy@apple.com>
+
         Correctly remembers the attached state of the Web Inspector so
         it opens in that state for the next window, or next launch.
 
index 11a8a37..bf4b0de 100644 (file)
@@ -84,8 +84,6 @@ using namespace WebCore;
     if (!_highlightWindow || !_targetView || ![_targetView window])
         return;
 
-    // Disable screen updates so the highlight moves in sync with the view.
-    [[_targetView window] disableScreenUpdatesUntilFlush];
     [[_targetView window] addChildWindow:_highlightWindow ordered:NSWindowAbove];
 
     // Observe both frame-changed and bounds-changed notifications because either one could leave
@@ -117,9 +115,6 @@ using namespace WebCore;
     if (_delegate && [_delegate respondsToSelector:@selector(willDetachWebNodeHighlight:)])
         [_delegate willDetachWebNodeHighlight:self];
 
-    // FIXME: is this necessary while detaching? Should test.
-    [[_targetView window] disableScreenUpdatesUntilFlush];
-
     NSNotificationCenter *notificationCenter = [NSNotificationCenter defaultCenter];
     [notificationCenter removeObserver:self name:NSViewFrameDidChangeNotification object:nil];
     [notificationCenter removeObserver:self name:NSViewBoundsDidChangeNotification object:nil];