Open Bug 1843881 Opened 2 years ago Updated 2 years ago

Simplify BookmarkingUI._newTabURI / BookmarkingUI._newTabURICache

Categories

(Firefox :: Bookmarks & History, task, P3)

Product:
Firefox
Component:
Bookmarks & History
Type:
task

Tracking

()

Status:
ASSIGNED
Tracking Flags:
Tracking Status
firefox117 --- affected

People

(Reporter: dao, Assigned: Gijs)

References

Details

From https://phabricator.services.mozilla.com/D183493#inline-1018493:

Hm, this is needed because AboutNewTab.newTabURL can change, and because it doesn't provide an nsIURI getter, I guess? That is annoying.

Would you mind filing a follow-up to tidy this up? We shouldn't need a map here, just a list (+ whatever AboutNewTab.newTabURIObject returns), and so the lookup overhead is unnecessary. I can probably find time to do that work.

Flags: needinfo?(gijskruitbosch+bugs)
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Flags: needinfo?(gijskruitbosch+bugs)
Severity: -- → N/A
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.