Fix bookmarks tab not reflecting group changes on tab switch

[diveshpatil9104]

Description

Problem

When a user adds, deletes, or renames a bookmark group on the Groups tab and then switches to the Bookmarks tab, the changes don't appear. The bookmarks tab keeps showing the old group state until the user moves a bookmark, which happens to trigger a form reload internally.

Root Cause

toggleControllers() was missing two steps when switching from Groups → Bookmarks:

1. Group changes weren't persisted to userDataStore — that only happened on modal dismiss.
2. The bookmarks form was never rebuilt — it was built once in viewDidLoad and never refreshed.

Fix

• Persist group changes (updateModelState()) before switching away from the Groups tab.
• Reload the bookmarks form (reloadFormFromStore()) when switching to the Bookmarks tab.
• Extract reloadFormFromStore() to deduplicate the deferred reload pattern already used by moveRowAt (see Index Out of Bounds When Reordering Bookmarks in ManageBookmarksViewController #922).

Files Changed

File	Change
ManageBookmarksAndGroupsViewController.swift	Persist groups + trigger reload on tab switch
ManageBookmarksViewController.swift	Add reloadFormFromStore(), deduplicate moveRowAt

Before (Bug) | After (Fixed)

Before (Bug)	After (Fixed)
issue.mov	fix.mov
Bookmarks tab shows stale group sections after adding/deleting/renaming groups	Bookmarks tab immediately reflects current group state on tab switch

[aaronbrethorst]

Hey Divesh — sharp diagnosis on this one. The root cause analysis is spot on: group changes weren't being persisted until viewWillDisappear, so the Bookmarks tab had no way to see them on a mid-session tab switch. Calling updateModelState() before the swap and then reloadFormFromStore() after is exactly the right fix. Extracting the deferred reload into its own method also cleans up the existing moveRowAt code nicely. This looks good to merge!