[Bedrock] Moving `TabMenuModelDelegate` into `BrowserWindowFeatures`.

This CL moves `TabMenuModelDelegate` off `Browser` and into
`BrowserWindowFeatures`.

`BrowserTabMenuModelDelegate` takes `SessionID`, `Profile*` and
`web_app::AppBrowserController*` as arguments to replace `Browser*` as
member.

`SessionID` is later used to find the correct browser which is necessary
for `GetOtherBrowserWindows`.

Also made the following code hygiene changes:

1. Explicitly made `BrowserTabMenuModelDelegate` non-copyable &
non-movable.

2. In `existing_window_sub_menu_model_browsertest.cc` changed `vector`s
to `initializer_list`/`span` to remove the unnecessary memory
allocation.

3. Also changed the variable name `kExpectedMRUOrder` to
`expected_mru_order` to reflect the fact that it is "not" a compile time
constant.

Bug: 418856264
Change-Id: I3e8e86b474ba9ea755d961995d29529a3cc4967e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6564309
Reviewed-by: Tom Lukaszewicz <tluk@chromium.org>
Commit-Queue: Tom Lukaszewicz <tluk@chromium.org>
Reviewed-by: Keren Zhu <kerenzhu@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1463639}