1635467 - Add test for ResourceWatcher::unwatch when using several resources

Status: RESOLVED FIXED

(DevTools :: Inspector, task, P3)

Description

[Julian Descottes [:jdescottes]]

Quick followup to Bug 1625961 to avoid potential regressions such as the one found by Nicolas in the review, see https://phabricator.services.mozilla.com/D72854#inline-431055

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1635467 - Test calling ResourceWatcher::unwatch for several resources

Depends on D72854

Comment 2

[Chris Peterson [:cpeterson]]

Tracking Fission DevTools bugs for Fission Nightly (M6) milestone

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1fd6a3754afb
Test calling ResourceWatcher::unwatch for several resources r=nchevobbe

Comment 4

[Dorel Luca [:dluca]]

Backed out 3 changesets (bug 1635467, bug 1625961) for Devtools failures in resources/tests/browser_resources_exceptions.js. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=301557137&repo=autoland&lineNumber=14173

Push with failure:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=1fd6a3754afb1497c1169d7054f5466e38b197a4

Backout:
https://hg.mozilla.org/integration/autoland/rev/494916b74ee5b90dc116b5f84c8f9a8c34c97624

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1d258bab62d7
Test calling ResourceWatcher::unwatch for several resources r=nchevobbe

Comment 6

[Razvan Maries]

Backed out for mochitest perma failures.

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedTaskRun=Aiwu2gqCQU6nkJLB3UDk3g-0&resultStatus=testfailed%2Cbusted%2Cexception&revision=1d258bab62d71f307b8f0e9693d6108c2d7a4839

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=301629272&repo=autoland&lineNumber=8411
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=301629957&repo=autoland&lineNumber=2507

TV failure: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=301629103&repo=autoland&lineNumber=11325

Backout: https://hg.mozilla.org/integration/autoland/rev/f44e64a61ed143bba2ef62dbf42a27943e082bcd

Comment 7

[Julian Descottes [:jdescottes]]

Sorry about that, the last version should fix both failures.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/91130843124c
Test calling ResourceWatcher::unwatch for several resources r=nchevobbe

Comment 9

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/91130843124c