1661186 - Fix showing the related extension for blocked resources

Status: RESOLVED FIXED

(DevTools :: Netmonitor, defect, P1)

Description

[Qwerty]

Bug 1555057 is broken in 80 release. Mozregression points to Bug 1625909.

DEBUG : Found commit message:
Bug 1625909 - Use the ResourceWatcher API to fetch Network Events r=ochameau,Honza,nchevobbe

This patch migrates to using the Resource Watcher API to handle Network events, it adds a legacy listeners for the network events

Differential Revision: https://phabricator.services.mozilla.com/D71543

DEBUG : Did not find a branch, checking all integration branches
INFO : The bisection is done.
INFO : Stopped

What were you doing?

1. Install uBlock Origin
2. Open netmonitor
3. Load some site
4. Look at transferred column

What happened?

Transferred column is saying "Blocked By Extension"

What should have happened?

Transferred column should say "Blocked By uBlock Origin"

Comment 1

[Hubert Boma Manilla (:bomsy)]

Thanks for reporting!

Looking into fixing!

Comment 2

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1661186 - Show the extension name for the blocked request r=honza

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1625909

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cdd6d47f2cd6
Show the extension name for the blocked request r=Honza

Comment 5

[Narcis Beleuzu [:NarcisB]]

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

Comment 6

[Ryan VanderMeulen [:RyanVM]]

Please nominate this for Beta approval when you get a chance.

Comment 7

[Hubert Boma Manilla (:bomsy)]

Comment on attachment 9172112 [details]
Bug 1661186 - Show the extension name for the blocked request r=honza

Beta/Release Uplift Approval Request

• User impact if declined: Small JS Change. Impacts only developers.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: Yes
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Not risky. Small change which show the specific extension name. Impacts only developers.
• String changes made/needed:

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9172112 [details]
Bug 1661186 - Show the extension name for the blocked request r=honza

Approved for 81.0b4.

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/19000ae951c4