Changeset 117057 in webkit

Timestamp:

May 15, 2012 6:08:28 AM (12 years ago)

Author:

yurys@chromium.org

Message:

Web Inspector: remove unnecessary setTimeout in HeapSnapshotGridNodes.js
​https://bugs.webkit.org/show_bug.cgi?id=86464

Reviewed by Pavel Feldman.

• inspector/front-end/HeapSnapshotGridNodes.js:

(WebInspector.HeapSnapshotGridNode.prototype._populateChildren.childrenRetrieved): Posting
a message to worker is already asynchronous, no need to postpone the call via setTimeout.
(WebInspector.HeapSnapshotGridNode.prototype._populateChildren):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/HeapSnapshotGridNodes.js (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-05-15  Yury Semikhatsky  <yurys@chromium.org>
+
+        Web Inspector: remove unnecessary setTimeout in HeapSnapshotGridNodes.js
+        https://bugs.webkit.org/show_bug.cgi?id=86464
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/front-end/HeapSnapshotGridNodes.js:
+        (WebInspector.HeapSnapshotGridNode.prototype._populateChildren.childrenRetrieved): Posting
+        a message to worker is already asynchronous, no need to postpone the call via setTimeout.
+        (WebInspector.HeapSnapshotGridNode.prototype._populateChildren):
+
 2012-05-15  Florin Malita  <fmalita@chromium.org>
 

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

@@ -271 +271 @@
             this._instanceCount += items.length;
             if (firstNotSerializedPosition < toPosition) {
-                setTimeout(serializeNextChunk.bind(this), 0);
+                serializeNextChunk.call(this);
                 return;
             }
@@ -279 +279 @@
             this.dispatchEventToListeners("populate complete");
         }
-        setTimeout(serializeNextChunk.bind(this), 0);
+        serializeNextChunk.call(this);
     },