Changeset 92461 in webkit

Timestamp:

Aug 5, 2011 12:51:18 AM (13 years ago)

Author:

commit-queue@webkit.org

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 <​pfeldman@google.com> on 2011-08-05
Reviewed by Yury Semikhatsky.

• inspector/front-end/TextViewer.js:

(WebInspector.TextViewer.prototype._syncScroll):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/TextViewer.js (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-08-05  Pavel Feldman  <pfeldman@google.com>
+
+        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  <pilgrim@chromium.org>
 

trunk/Source/WebCore/inspector/front-end/TextViewer.js

@@ -201 +201 @@
     _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;
     },