Adding this as a follow-up to bug 1857005. In order to ensure bookmarked open tabs are indicated in Fx View properly, we'll need to create a `BookmarkList` class that internally stays up-to-date (it may be nice to implement it in Places so others could reuse it). This class should query all the open tabs URLs at start or at first access, use observers (`bookmark-added`, `bookmark-removed`, `bookmark-url-changed` at least) to keep the list up-to-date, and query for new entries as new tabs will be added to the list In doing so, we avoid having to use `fetch` to check each individual URL which would become costly with 100+ tabs.
Bug 1876165 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Adding this as a follow-up to bug 1857005. In order to ensure bookmarked open tabs are indicated in Fx View properly, we'll need to create a `BookmarkList` class that internally stays up-to-date (it may be nice to implement it in Places so others could reuse it). This class should query all the open tabs URLs at start or at first access, use observers (`bookmark-added`, `bookmark-removed`, `bookmark-url-changed` at least) to keep the list up-to-date, and query for new entries as new tabs will be added to the list In doing so, we avoid having to use `fetch` to check each individual URL which would become costly with 100+ tabs. We should also consider only checking URLs for items rendered within the `virtual-list` component rather than the entire list of open tabs.