1401944 - Migrate browser_bug_638949_copy_link_location.js to the new console

Status: RESOLVED FIXED

(DevTools :: Console, enhancement, P1)

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1401944 - Delete browser_webconsole_copy_link_location in new console frontend; .

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

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Not sure why it was disabled since it was here before "the grand migration". Anyway, refactoring this a bit and putting it in the clipboard subsuite

Comment 3

[Patrick Brosset <:pbro>]

I'm confused, there are 2 tests with almost the same name: browser_webconsole_copy_link_location.js and browser_webconsole_context_menu_copy_link_location.js and it looks like you enable one in browser.ini but changed the other.
Should one be deleted?

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

oh, guess i was confused too

Comment 5

[Nicolas Chevobbe [:nchevobbe]]

Comment on attachment 8934445 [details]
Bug 1401944 - Delete browser_webconsole_copy_link_location in new console frontend; .

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

Comment 6

[Julian Descottes [:jdescottes]]

Comment on attachment 8934445 [details]
Bug 1401944 - Delete browser_webconsole_copy_link_location in new console frontend; .

https://reviewboard.mozilla.org/r/205388/#review211398

LGTM, thanks Nicolas.

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ea00ea69cb7e
Delete browser_webconsole_copy_link_location in new console frontend; r=jdescottes.

Comment 8

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/ea00ea69cb7e