[webkit-reviews] review granted: [Bug 62360] Web Inspector: [Chromium] Detailed heap snapshots window becomes unresponsive if several long-running operations being requested : [Attachment 97264] patch

[bugzilla-daemon at webkit.org]

Pavel Feldman <pfeldman at chromium.org> has granted Mikhail Naganov
<mnaganov at chromium.org>'s request for review:
Bug 62360: Web Inspector: [Chromium] Detailed heap snapshots window becomes
unresponsive if several long-running operations being requested
https://bugs.webkit.org/show_bug.cgi?id=62360

Attachment 97264: patch
https://bugs.webkit.org/attachment.cgi?id=97264&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=97264&action=review

> Source/WebCore/inspector/front-end/PartialQuickSort.js:31
> +function swap(array, i1, i2)

You should move these into the closure within defineProperty.