Title: [92461] trunk/Source/WebCore
Revision
92461
Author
[email protected]
Date
2011-08-05 00:51:18 -0700 (Fri, 05 Aug 2011)

Log Message

Web Inspector: do not delay scroll event handling - scroll is already async.
https://bugs.webkit.org/show_bug.cgi?id=65693

Patch by Pavel Feldman <[email protected]> on 2011-08-05
Reviewed by Yury Semikhatsky.

* inspector/front-end/TextViewer.js:
(WebInspector.TextViewer.prototype._syncScroll):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (92460 => 92461)


--- trunk/Source/WebCore/ChangeLog	2011-08-05 07:48:48 UTC (rev 92460)
+++ trunk/Source/WebCore/ChangeLog	2011-08-05 07:51:18 UTC (rev 92461)
@@ -1,3 +1,13 @@
+2011-08-05  Pavel Feldman  <[email protected]>
+
+        Web Inspector: do not delay scroll event handling - scroll is already async.
+        https://bugs.webkit.org/show_bug.cgi?id=65693
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/front-end/TextViewer.js:
+        (WebInspector.TextViewer.prototype._syncScroll):
+
 2011-08-05  Mark Pilgrim  <[email protected]>
 
         Remove LegacyDefaultOptionalArguments flag from web audio API

Modified: trunk/Source/WebCore/inspector/front-end/TextViewer.js (92460 => 92461)


--- trunk/Source/WebCore/inspector/front-end/TextViewer.js	2011-08-05 07:48:48 UTC (rev 92460)
+++ trunk/Source/WebCore/inspector/front-end/TextViewer.js	2011-08-05 07:51:18 UTC (rev 92461)
@@ -200,14 +200,11 @@
 
     _syncScroll: function()
     {
-        // Async call due to performance reasons.
-        setTimeout(function() {
-            var mainElement = this._mainPanel.element;
-            var gutterElement = this._gutterPanel.element;
-            // Handle horizontal scroll bar at the bottom of the main panel.
-            this._gutterPanel.syncClientHeight(mainElement.clientHeight);
-            gutterElement.scrollTop = mainElement.scrollTop;
-        }.bind(this), 0);
+        var mainElement = this._mainPanel.element;
+        var gutterElement = this._gutterPanel.element;
+        // Handle horizontal scroll bar at the bottom of the main panel.
+        this._gutterPanel.syncClientHeight(mainElement.clientHeight);
+        gutterElement.scrollTop = mainElement.scrollTop;
     },
 
     _syncDecorationsForLine: function(lineNumber)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to