1408893 - Enable browser_webconsole_eval_in_debugger_stackframe2.js in the new console frontend

Status: RESOLVED FIXED

(DevTools :: Console, enhancement, P1)

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1408893 - Enable browser_webconsole_eval_in_debugger_stackframe2.js in the new console frontend; .

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

Comment 2

[Brian Grinstead [:bgrins]]

Comment on attachment 8940185 [details]
Bug 1408893 - Enable browser_webconsole_eval_in_debugger_stackframe2.js in the new console frontend; .

https://reviewboard.mozilla.org/r/210496/#review216240

::: devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_eval_in_debugger_stackframe2.js:57
(Diff revision 1)
> -                          '"globalFooBug783499foo2SecondCall"');
> +  jsterm.execute("foo + foo2");
> +  message = await onMessageReceived;
> +  ok(message, "`foo + foo2` was evaluated as expected with debugger paused");
> +
> +  info("Checking the first command, which is the last to resolve since it paused");
> +  ok(firstCallEvaluationResult === unresolvedSymbol, "firstCall was not evaluated yet");

Nit: you could use `is` here

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8940185 [details]
Bug 1408893 - Enable browser_webconsole_eval_in_debugger_stackframe2.js in the new console frontend; .

https://reviewboard.mozilla.org/r/210496/#review216240

> Nit: you could use `is` here

I tried but the test was failing : 
```
  TEST-UNEXPECTED-FAIL | devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_eval_in_debugger_stackframe2.js | Uncaught exception - at chrome://mochikit/content/browser-test.js:1269 - TypeError: can't convert symbol to string
Stack trace:
    test_is@chrome://mochikit/content/browser-test.js:1269:5
    @chrome://mochitests/content/browser/devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_eval_in_debugger_stackframe2.js:57:3
    Async*Tester_execTest/<@chrome://mochikit/content/browser-test.js:1065:21
    Tester_execTest@chrome://mochikit/content/browser-test.js:1056:9
    Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:956:9
    SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:795:59
```

Maybe I should file a bug against browser-test.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/253e9d9481f5
Enable browser_webconsole_eval_in_debugger_stackframe2.js in the new console frontend; r=bgrins.

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/253e9d9481f5