Closes #11208: Add tab partitions and groups to BrowserState #11272
Conversation
...ents/browser/state/src/main/java/mozilla/components/browser/state/reducer/TabGroupReducer.kt
Show resolved
Hide resolved
...nents/browser/state/src/main/java/mozilla/components/browser/state/reducer/TabListReducer.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 This looks all great.
...nents/browser/state/src/main/java/mozilla/components/browser/state/reducer/TabListReducer.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good 👍🏽
51c068f
to
1ccb8c1
Compare
@pocmo @Amejia481 OK, added lots of tests and additional validation to make sure we throw an exception if a tab is added that doesn't exist. /cc @rocketsroger @jonalmeida in case you want to take a look as well before this lands, but no changes relative to the RFC. |
1ccb8c1
to
5579442
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took another peek at this today and this looks like a great patch!
Implementation of our RFC. Besides state and actions the only complexity added here is to automatically create the groups/partitions, as needed, and discussed in the RFC to simplify usage for consuming applications. The other piece is to make sure partitions are automatically cleaned up when tabs are removed.
Tests are on the way, but putting up for (early) review.