Open Bug 1942758 Opened 23 days ago Updated 14 days ago

`BrowserStoreBinding` doesn't appear to be used

Categories

(Fenix :: General, defect)

All
Android
defect

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: willdurand, Assigned: matt-tighe)

References

Details

(Whiteboard: [fxdroid][group1])

Attachments

(1 file)

The BrowserStoreBinding class, which is responsible for sync'ing the Fenix's appStore with the A-C's browserStore doesn't appear to be used at all. Do we know why?

The blame/log output is hard to follow and I am not sure what happened here. :matt-tighe any idea?

Yes, this was part of an ultimately failed effort to consolidate some of our core state properties into the Store pattern - specifically private browsing mode. After running into an edge case that we didn't have a clean solution for I ended up abandoning the stack. This binding was actually deleted in one of the abandoned patches so we should be safe to remove it.

Let's just remove it. Adding it to our board.

Whiteboard: [fxdroid][group1]
Severity: -- → S3
Assignee: nobody → mtighe
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: