Closed Bug 1811271 Opened 1 year ago Closed 1 year ago

Consider maintaining the blocked dll's on top of the about:third-party list

Categories

(Firefox :: Launcher Process, enhancement)

All
Windows
enhancement

Tracking

()

VERIFIED FIXED
111 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox109 --- unaffected
firefox110 --- verified
firefox111 --- verified

People

(Reporter: emilghitta, Assigned: gstoll)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Attached image listOrder.gif

Currently, if we:

  1. Block a dll via the about:third-party page.
  2. Restart Firefox.

The blocked dll is going to be displayed as the only dll available (first and only item of the dll list).

If we wait for the "Reload with system info" button to appear and click on it I can notice that a couple of more dll's are added inside the list but the previously blocked dll remains on top of that list (which is fine).

If I then proceed to hit ctrl + o, close the file explorer and refresh the about:third-party page a couple of more dll's are added to the list and the previously blocked dll is moved away from the top of the list.

I think that we might want to consider keeping the blocked dll's as the first items of the about:third-party page dll list so that users can find/spot and interact with the dll's which were manually modified with ease (without having to search for each of them especially if the list is bigger).

This is a good idea, thanks!

Assignee: nobody → gstoll
Status: NEW → ASSIGNED
Pushed by gstoll@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5bcb66dfaaaa
sort blocked DLLs at the top of the list r=Gijs
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 111 Branch

Comment on attachment 9313107 [details]
Bug 1811271 - sort blocked DLLs at the top of the list r=gijs

Beta/Release Uplift Approval Request

  • User impact if declined: DLLs that are blocked will be harder to find in the list on about:third-party, which could be confusing
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: (see steps to reproduce in bug)
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): simply changing the sort order (and adding a test).

Note that there was some weirdness with the patch applying weirdly because of other changes to the file (that will not be uplifted) - I wasn't sure why there was a conflict in the first place, but let me know if this happens and I'll move the patch onto the Beta branch.

  • String changes made/needed: no
  • Is Android affected?: No
Attachment #9313107 - Flags: approval-mozilla-beta?
Flags: qe-verify+
QA Whiteboard: [qa-triaged]

This is verified fixed using Firefox 111.0a1 (BuildId:20230125215201) on Windows 10, 11 & 7.

The previously blocked dll's are maintained on top of the about:third-party dll list while using "normal browsing mode".

Also verified that the previously blocked dll's are on top of the dll list when running Firefox in Troubleshoot mode or with the --disableDynamicBlocklist argument applied.

Comment on attachment 9313107 [details]
Bug 1811271 - sort blocked DLLs at the top of the list r=gijs

Approved for 110 beta 6, thanks.

Attachment #9313107 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

This is verified fixed using Firefox 110.0b7 (BuildId:20230129190147) on Windows 11, 10 & 7.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: