Title: [177309] trunk/Source/WebKit2
Revision
177309
Author
[email protected]
Date
2014-12-15 13:27:17 -0800 (Mon, 15 Dec 2014)

Log Message

Web Inspector: Docked inspector repositioned incorrectly when dragging frame borders
https://bugs.webkit.org/show_bug.cgi?id=139415

Patch by Matt Baker <[email protected]> on 2014-12-15
Reviewed by Joseph Pecoraro.

The _ignoreNextInspectedViewFrameDidChange flag wasn't set back to NO after
being tested by inspectedViewFrameDidChange.

* UIProcess/mac/WebInspectorProxyMac.mm:
(-[WKWebInspectorProxyObjCAdapter inspectedViewFrameDidChange:]):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (177308 => 177309)


--- trunk/Source/WebKit2/ChangeLog	2014-12-15 21:07:30 UTC (rev 177308)
+++ trunk/Source/WebKit2/ChangeLog	2014-12-15 21:27:17 UTC (rev 177309)
@@ -1,3 +1,16 @@
+2014-12-15  Matt Baker  <[email protected]>
+
+        Web Inspector: Docked inspector repositioned incorrectly when dragging frame borders
+        https://bugs.webkit.org/show_bug.cgi?id=139415
+
+        Reviewed by Joseph Pecoraro.
+
+        The _ignoreNextInspectedViewFrameDidChange flag wasn't set back to NO after
+        being tested by inspectedViewFrameDidChange.
+
+        * UIProcess/mac/WebInspectorProxyMac.mm:
+        (-[WKWebInspectorProxyObjCAdapter inspectedViewFrameDidChange:]):
+
 2014-12-15  Gavin Barraclough  <[email protected]>
 
         Simplify tracking of process suppression disabled for PluginProcessManager

Modified: trunk/Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm (177308 => 177309)


--- trunk/Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm	2014-12-15 21:07:30 UTC (rev 177308)
+++ trunk/Source/WebKit2/UIProcess/mac/WebInspectorProxyMac.mm	2014-12-15 21:27:17 UTC (rev 177309)
@@ -145,11 +145,11 @@
 
 - (void)inspectedViewFrameDidChange:(NSNotification *)notification
 {
-    if (_ignoreNextInspectedViewFrameDidChange)
+    if (_ignoreNextInspectedViewFrameDidChange) {
+        _ignoreNextInspectedViewFrameDidChange = NO;
         return;
+    }
 
-    _ignoreNextInspectedViewFrameDidChange = NO;
-
     // Resizing the views while inside this notification can lead to bad results when entering
     // or exiting full screen. To avoid that we need to perform the work after a delay. We only
     // depend on this for enforcing the height constraints, so a small delay isn't terrible. Most
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to