Changeset 134028 in webkit

Timestamp:

Nov 9, 2012, 1:01:04 AM (13 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Navigating around NMI snapshot grid with keys breaks the grid
​https://bugs.webkit.org/show_bug.cgi?id=101611

Patch by Alexei Filippov <​alph@chromium.org> on 2012-11-09
Reviewed by Yury Semikhatsky.

Keys navigation cause populate message sent to the node. The node should
not react on populate messages but the first one.

• inspector/front-end/NativeMemorySnapshotView.js:

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-11-09  Alexei Filippov  <alph@chromium.org>
+
+        Web Inspector: Navigating around NMI snapshot grid with keys breaks the grid
+        https://bugs.webkit.org/show_bug.cgi?id=101611
+
+        Reviewed by Yury Semikhatsky.
+
+        Keys navigation cause populate message sent to the node. The node should
+        not react on populate messages but the first one.
+
+        * inspector/front-end/NativeMemorySnapshotView.js:
+
 2012-11-08  Eugene Klyuchnikov  <eustas.bug@gmail.com>
 

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

@@ -147 +147 @@
 
     _populate: function() {
+        this.removeEventListener("populate", this._populate, this);
         function comparator(a, b) {
             return b.size - a.size;