1437845 - Enable browser_console_dead_objects.js in new frontend

Status: RESOLVED FIXED

(DevTools :: Console, enhancement, P1)

Comment 1

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Attachment: Bug 1437845 - Enable browser_console_dead_objects.js in new frontend

We can't access dead objects in an E10S build so this has to be non-E10s only for now... if "1442226 - Investigate creation of dead objects in E10S mode" bears any fruit we can always come back and add the E10S version.

This test used to click a link to open the Variables View but the link now points to MDN instead so it can't be clicked inside a test (external links cause app crashes because they are not allowed).

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

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

We can use this to get a dead object:
var s = Cu.Sandbox(null); Cu.nukeSandbox(s); s;

Comment 3

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Comment on attachment 8955114 [details]
Bug 1437845 - Enable browser_console_dead_objects.js in new frontend

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/224270/diff/1-2/

Comment 4

[Brian Grinstead [:bgrins]]

Comment on attachment 8955114 [details]
Bug 1437845 - Enable browser_console_dead_objects.js in new frontend

https://reviewboard.mozilla.org/r/224270/#review230304

Thanks, this is much nicer (and glad it's now working in e10s)!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/98f2913853d4
Enable browser_console_dead_objects.js in new frontend r=bgrins

Comment 6

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/98f2913853d4