892413 - Followup fixes for bug 887605

Status: RESOLVED FIXED

(DevTools :: Netmonitor, defect, P3)

Comment 1

[Victor Porof [:vporof][:vp]]

Attachment: v1

Comment 2

[Heather Arthur [:harth]]

Comment on attachment 773867 [details] [diff] [review]
v1

Review of attachment 773867 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/netmonitor/test/browser_net_copy_url.js
@@ +21,5 @@
> +      }, function onSuccess() {
> +        ok(true, "Clipboard contains the currently selected item's url.");
> +        cleanUp();
> +      }, function onFailure() {
> +        ok(false, "Copying the currently selected item's url was unsuccessful.");

waitForClipboard does the ok(true) and ok(false) for you, so this is redundant, unless you wanted to add those custom messages.

Comment 3

[Paul Rouget [:paul]]

https://hg.mozilla.org/integration/fx-team/rev/11d1d03a247f

Comment 4

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/11d1d03a247f