1444823 - Execute messageManager.sendAsyncMessage after runTest in bulklog and objectexpand

Status: RESOLVED FIXED

(DevTools :: General, enhancement, P3)

Description

[Julian Descottes [:jdescottes]]

From comment on Bug 1441703:

> ::: testing/talos/talos/tests/devtools/addon/content/tests/webconsole/bulklog.js:51
> (Diff revision 4)
> > +  ) + ")()", true);
> > +
> > +  // Kick off the logging
> > +  messageManager.sendAsyncMessage("do-logs");
> > +
> > +  let test = runTest("console.bulklog");
> 
> I didn't realized but `messageManager.sendAsyncMessage("do-logs");` should 
> probably be execute after `runTest` call.
>

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1444823 - Execute messageManager.sendAsyncMessage after runTest; .

Review commit: https://reviewboard.mozilla.org/r/227668/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/227668/

Comment 2

[Julian Descottes [:jdescottes]]

Comment on attachment 8958745 [details]
Bug 1444823 - Execute messageManager.sendAsyncMessage after runTest; .

https://reviewboard.mozilla.org/r/227668/#review233410

Looks good to me thanks!

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

TRY is green with a g2 job. Let's land this

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3a7a26b914ac
Execute messageManager.sendAsyncMessage after runTest; r=jdescottes.

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/3a7a26b914ac