Bug 1368100 - Fix batchMessageAdd for test-perf.html; r=bgrins. draft
authorNicolas Chevobbe <nchevobbe@mozilla.com>
Fri, 17 Nov 2017 15:14:32 +0100
changeset 699705 fb8531a3c97e6439b9a4f48675edeccd3c3486e2
parent 699617 a77c628829b389ed6ac608eadc88b09eb5115bef
child 699706 cb1f611219b3aea41db110e05084c4732e717430
push id89636
push userbmo:nchevobbe@mozilla.com
push dateFri, 17 Nov 2017 14:29:44 +0000
reviewersbgrins
bugs1368100
milestone59.0a1
Bug 1368100 - Fix batchMessageAdd for test-perf.html; r=bgrins. MozReview-Commit-ID: KvEriUXRzve
devtools/client/webconsole/new-console-output/new-console-output-wrapper.js
--- a/devtools/client/webconsole/new-console-output/new-console-output-wrapper.js
+++ b/devtools/client/webconsole/new-console-output/new-console-output-wrapper.js
@@ -198,40 +198,43 @@ NewConsoleOutputWrapper.prototype = {
           childComponent
       ));
       this.body = ReactDOM.render(provider, this.parentNode);
 
       this.jsterm.focus();
     });
   },
 
-  dispatchMessageAdd: function (message, waitForResponse) {
+  dispatchMessageAdd: function (packet, waitForResponse) {
     // Wait for the message to render to resolve with the DOM node.
     // This is just for backwards compatibility with old tests, and should
     // be removed once it's not needed anymore.
     // Can only wait for response if the action contains a valid message.
     let promise;
     if (waitForResponse) {
+      const timeStampToMatch = packet.message
+        ? packet.message.timeStamp
+        : packet.timestamp;
       promise = new Promise(resolve => {
         let jsterm = this.jsterm;
         jsterm.hud.on("new-messages", function onThisMessage(e, messages) {
           for (let m of messages) {
-            if (m.timeStamp === message.timestamp) {
+            if (m.timeStamp === timeStampToMatch) {
               resolve(m.node);
               jsterm.hud.off("new-messages", onThisMessage);
               return;
             }
           }
         });
       });
     } else {
       promise = Promise.resolve();
     }
 
-    this.batchedMessagesAdd(message);
+    this.batchedMessagesAdd(packet);
     return promise;
   },
 
   dispatchMessagesAdd: function (messages) {
     store.dispatch(actions.messagesAdd(messages));
   },
 
   dispatchMessagesClear: function () {