From e57a192292925559d992c7565aac3a4fdc1cfdbf Mon Sep 17 00:00:00 2001
From: AlexeyBarabash <alexey@brave.com>
Date: Wed, 11 Nov 2020 19:18:40 +0200
Subject: [PATCH 1/3] Get back `Tabs from other devices` history section; fixes
 brave-browser#12391;

---
 ...e-browser-resources-history-app.html.patch | 20 -------------------
 ...wser-resources-history-side_bar.html.patch | 18 -----------------
 ...s-history-synced_device_manager.html.patch | 20 +++++++++++++++++++
 3 files changed, 20 insertions(+), 38 deletions(-)
 delete mode 100644 patches/chrome-browser-resources-history-app.html.patch
 delete mode 100644 patches/chrome-browser-resources-history-side_bar.html.patch
 create mode 100644 patches/chrome-browser-resources-history-synced_device_manager.html.patch

diff --git a/patches/chrome-browser-resources-history-app.html.patch b/patches/chrome-browser-resources-history-app.html.patch
deleted file mode 100644
index 861c8777f478..000000000000
--- a/patches/chrome-browser-resources-history-app.html.patch
+++ /dev/null
@@ -1,20 +0,0 @@
-diff --git a/chrome/browser/resources/history/app.html b/chrome/browser/resources/history/app.html
-index 0afd8eb41341f3e10aa216dc00599078fc8f5df0..1360df4b5e903f3eca956d32e9765d34173309d5 100644
---- a/chrome/browser/resources/history/app.html
-+++ b/chrome/browser/resources/history/app.html
-@@ -70,6 +70,7 @@
-             query-result="[[queryResult_]]"
-             path="history">
-         </history-list>
-+<!--
-         <template is="dom-if" if="[[syncedTabsSelected_(selectedPage_)]]">
-           <history-synced-device-manager id="synced-devices"
-               session-list="[[queryResult_.sessionList]]"
-@@ -78,6 +79,7 @@
-               path="syncedTabs">
-           </history-synced-device-manager>
-         </template>
-+-->
-       </iron-pages>
-     </div>
- 
diff --git a/patches/chrome-browser-resources-history-side_bar.html.patch b/patches/chrome-browser-resources-history-side_bar.html.patch
deleted file mode 100644
index 80849daf210f..000000000000
--- a/patches/chrome-browser-resources-history-side_bar.html.patch
+++ /dev/null
@@ -1,18 +0,0 @@
-diff --git a/chrome/browser/resources/history/side_bar.html b/chrome/browser/resources/history/side_bar.html
-index 372d46cf8efc72e91bb9473a59b762cbfc868090..d0cfe9cca0e895713302f4e410d1a45bef4a94c6 100644
---- a/chrome/browser/resources/history/side_bar.html
-+++ b/chrome/browser/resources/history/side_bar.html
-@@ -101,11 +101,13 @@
-         $i18n{historyMenuItem}
-         <paper-ripple></paper-ripple>
-       </a>
-+<!--
-       <a href="/syncedTabs" class="page-item" path="syncedTabs"
-           on-click="onItemClick_">
-         $i18n{openTabsMenuItem}
-         <paper-ripple></paper-ripple>
-       </a>
-+-->
-       <div class="separator"></div>
-       <a id="clear-browsing-data"
-           href="chrome://settings/clearBrowserData"
diff --git a/patches/chrome-browser-resources-history-synced_device_manager.html.patch b/patches/chrome-browser-resources-history-synced_device_manager.html.patch
new file mode 100644
index 000000000000..2b37e6e8b643
--- /dev/null
+++ b/patches/chrome-browser-resources-history-synced_device_manager.html.patch
@@ -0,0 +1,20 @@
+diff --git a/chrome/browser/resources/history/synced_device_manager.html b/chrome/browser/resources/history/synced_device_manager.html
+index d03a367a7cb1ea6ccad6478a2300606b4c20829a..bfa8a2debda7e0c35b217cea85df668232c5507b 100644
+--- a/chrome/browser/resources/history/synced_device_manager.html
++++ b/chrome/browser/resources/history/synced_device_manager.html
+@@ -73,6 +73,7 @@
+                                        guestSession_)]]">
+       [[noSyncedTabsMessage(fetchingSyncedTabs_, searchTerm)]]
+     </div>
++<!--
+     <div id="sign-in-guide"
+         hidden="[[!showSignInGuide(signInState, guestSession_)]]">
+       <div id="illustration"></div>
+@@ -83,6 +84,7 @@
+         $i18n{signInButton}
+       </cr-button>
+     </div>
++-->
+ 
+     <cr-lazy-render id="menu">
+       <template>

From 347e2b75504afff6ac4608768b07746a60d46bea Mon Sep 17 00:00:00 2001
From: AlexeyBarabash <alexey@brave.com>
Date: Mon, 16 Nov 2020 16:50:38 +0200
Subject: [PATCH 2/3] History sub menu item for all tabs from other synced
 devices

---
 app/brave_generated_resources.grd             |  3 +
 browser/ui/BUILD.gn                           |  1 +
 .../brave_recent_tabs_sub_menu_model.h        | 30 +++++++++
 .../browser/ui/toolbar/app_menu_model.cc      | 12 ++++
 .../ui/toolbar/recent_tabs_sub_menu_model.cc  | 66 +++++++++++++++++++
 .../ui/toolbar/recent_tabs_sub_menu_model.h   | 18 +++++
 patches/chrome-VERSION.patch                  |  2 +-
 ...oolbar-recent_tabs_sub_menu_model.cc.patch | 12 ++++
 ...toolbar-recent_tabs_sub_menu_model.h.patch | 12 ++++
 9 files changed, 155 insertions(+), 1 deletion(-)
 create mode 100644 browser/ui/toolbar/brave_recent_tabs_sub_menu_model.h
 create mode 100644 chromium_src/chrome/browser/ui/toolbar/app_menu_model.cc
 create mode 100644 chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
 create mode 100644 chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
 create mode 100644 patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.cc.patch
 create mode 100644 patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.h.patch

diff --git a/app/brave_generated_resources.grd b/app/brave_generated_resources.grd
index 8ada7008fe2f..676d78ac1f7f 100644
--- a/app/brave_generated_resources.grd
+++ b/app/brave_generated_resources.grd
@@ -172,6 +172,9 @@
       <message name="IDS_OPEN_GUEST_PROFILE" desc="The app menu item to create a new User Profile">
         Open Guest Window
       </message>
+      <message name="IDS_OPEN_MORE_OTHER_DEVICES_SESSIONS" desc="The history sub-menu item to show sessions from other devices">
+        More…
+      </message>
       <message name="IDS_EXTENSION_CANT_INSTALL_ON_BRAVE" desc="Error message when user tries to install an extension that is not allowed in Brave.">
         <ph name="EXTENSION_NAME">$1</ph> (extension ID "<ph name="EXTENSION_ID">$2<ex>abacabadabacabaeabacabadabacabaf</ex></ph>") is not allowed in Brave.
       </message>
diff --git a/browser/ui/BUILD.gn b/browser/ui/BUILD.gn
index 332a142ffb86..ba68d10548c1 100644
--- a/browser/ui/BUILD.gn
+++ b/browser/ui/BUILD.gn
@@ -75,6 +75,7 @@ source_set("ui") {
       "content_settings/brave_content_setting_image_models.h",
       "omnibox/brave_omnibox_client_impl.cc",
       "omnibox/brave_omnibox_client_impl.h",
+      "toolbar/brave_recent_tabs_sub_menu_model.h",
       "toolbar/brave_app_menu_model.cc",
       "toolbar/brave_app_menu_model.h",
       "webui/brave_settings_ui.cc",
diff --git a/browser/ui/toolbar/brave_recent_tabs_sub_menu_model.h b/browser/ui/toolbar/brave_recent_tabs_sub_menu_model.h
new file mode 100644
index 000000000000..23140a5a7d1d
--- /dev/null
+++ b/browser/ui/toolbar/brave_recent_tabs_sub_menu_model.h
@@ -0,0 +1,30 @@
+/* Copyright (c) 2020 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#ifndef BRAVE_BROWSER_UI_TOOLBAR_BRAVE_RECENT_TABS_SUB_MENU_MODEL_H_
+#define BRAVE_BROWSER_UI_TOOLBAR_BRAVE_RECENT_TABS_SUB_MENU_MODEL_H_
+
+#include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
+
+class Browser;
+
+namespace ui {
+class AcceleratorProvider;
+}
+
+class BraveRecentTabsSubMenuModel : public RecentTabsSubMenuModel {
+ public:
+  BraveRecentTabsSubMenuModel(ui::AcceleratorProvider* accelerator_provider,
+                         Browser* browser);
+
+  ~BraveRecentTabsSubMenuModel() override;
+
+  void ExecuteCommand(int command_id, int event_flags) override;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(BraveRecentTabsSubMenuModel);
+};
+
+#endif  // BRAVE_BROWSER_UI_TOOLBAR_BRAVE_RECENT_TABS_SUB_MENU_MODEL_H_
diff --git a/chromium_src/chrome/browser/ui/toolbar/app_menu_model.cc b/chromium_src/chrome/browser/ui/toolbar/app_menu_model.cc
new file mode 100644
index 000000000000..4cdbd47c2f44
--- /dev/null
+++ b/chromium_src/chrome/browser/ui/toolbar/app_menu_model.cc
@@ -0,0 +1,12 @@
+/* Copyright (c) 2020 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#include "brave/browser/ui/toolbar/brave_recent_tabs_sub_menu_model.h"
+
+#define RecentTabsSubMenuModel BraveRecentTabsSubMenuModel
+
+#include "../../../../../../chrome/browser/ui/toolbar/app_menu_model.cc"
+
+#undef RecentTabsSubMenuModel
diff --git a/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc b/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
new file mode 100644
index 000000000000..e104de66c875
--- /dev/null
+++ b/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
@@ -0,0 +1,66 @@
+/* Copyright (c) 2020 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#include "chrome/browser/ui/singleton_tabs.h"
+
+namespace {
+
+const char kBraveStubSessionTag[] = "brave_stub_more_session_tag";
+const char kBraveSyncedTabsUrl[] = "brave://history/syncedTabs";
+
+}  //  namespace
+
+#define BRAVE_BUILD_TABS_FROM_OTHER_DEVICES                                 \
+  if (tabs_in_session.size() > kMaxTabsPerSessionToShow) {                  \
+    /* Not all the tabs are shown in menu */                                \
+    if (!stub_tab_.get()) {                                                 \
+      stub_tab_.reset(new sessions::SessionTab());                          \
+      sessions::SerializedNavigationEntry stub_nav_entry;                   \
+      stub_nav_entry.set_title(                                             \
+          l10n_util::GetStringUTF16(IDS_OPEN_MORE_OTHER_DEVICES_SESSIONS)); \
+      stub_nav_entry.set_virtual_url(GURL(kBraveSyncedTabsUrl));            \
+      stub_tab_->navigations.push_back(stub_nav_entry);                     \
+      stub_tab_->tab_id = SessionID::NewUnique();                           \
+    }                                                                       \
+    tabs_in_session[kMaxTabsPerSessionToShow] = stub_tab_.get();            \
+    BuildOtherDevicesTabItem(kBraveStubSessionTag,                          \
+                             *tabs_in_session[kMaxTabsPerSessionToShow]);   \
+  }
+
+#include "../../../../../../chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc"
+
+#undef BRAVE_BUILD_TABS_FROM_OTHER_DEVICES
+
+#include "brave/browser/ui/toolbar/brave_recent_tabs_sub_menu_model.h"
+
+// Methods of BraveRecentTabsSubMenuModel are implemented below instead of
+// brave_recent_tabs_sub_menu_model.cc to have the access to functions in
+// anonymous namespace in recent_tabs_sub_menu_model.cc
+
+BraveRecentTabsSubMenuModel::BraveRecentTabsSubMenuModel(
+    ui::AcceleratorProvider* accelerator_provider,
+    Browser* browser)
+    : RecentTabsSubMenuModel(accelerator_provider, browser) {}
+
+BraveRecentTabsSubMenuModel::~BraveRecentTabsSubMenuModel() {}
+
+void BraveRecentTabsSubMenuModel::ExecuteCommand(int command_id,
+                                                 int event_flags) {
+  if (IsTabModelCommandId(command_id)) {
+    TabNavigationItems* tab_items = NULL;
+    int tab_items_idx = CommandIdToTabVectorIndex(command_id, &tab_items);
+    const TabNavigationItem& item = (*tab_items)[tab_items_idx];
+    DCHECK(item.tab_id.is_valid() && item.url.is_valid());
+
+    if (item.session_tag == kBraveStubSessionTag) {
+      ShowSingletonTabOverwritingNTP(
+          browser_,
+          GetSingletonTabNavigateParams(browser_, GURL(kBraveSyncedTabsUrl)));
+      return;
+    }
+  }
+
+  RecentTabsSubMenuModel::ExecuteCommand(command_id, event_flags);
+}
diff --git a/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h b/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
new file mode 100644
index 000000000000..f118beff4639
--- /dev/null
+++ b/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
@@ -0,0 +1,18 @@
+/* Copyright (c) 2020 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+#ifndef BRAVE_CHROMIUM_SRC_CHROME_BROWSER_UI_TOOLBAR_RECENT_TABS_SUB_MENU_MODEL_H_
+#define BRAVE_CHROMIUM_SRC_CHROME_BROWSER_UI_TOOLBAR_RECENT_TABS_SUB_MENU_MODEL_H_
+
+#define BRAVE_RECENT_TABS_SUB_MENU_MODEL_H_ \
+  private: \
+    std::auto_ptr<sessions::SessionTab> stub_tab_; \
+    friend class BraveRecentTabsSubMenuModel; \
+  public:
+#include "../../../../../../chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
+
+#undef BRAVE_RECENT_TABS_SUB_MENU_MODEL_H_
+
+#endif  // BRAVE_CHROMIUM_SRC_CHROME_BROWSER_UI_TOOLBAR_RECENT_TABS_SUB_MENU_MODEL_H_
diff --git a/patches/chrome-VERSION.patch b/patches/chrome-VERSION.patch
index a54994ed567a..3c9fe8e61b15 100644
--- a/patches/chrome-VERSION.patch
+++ b/patches/chrome-VERSION.patch
@@ -1,5 +1,5 @@
 diff --git a/chrome/VERSION b/chrome/VERSION
-index 2c4514a701725958eed9feacc7a43c46ef1f2fba..205283ea59b247e3aa5ac69d0d81f2c05133c5b8 100644
+index 2c4514a701725958eed9feacc7a43c46ef1f2fba..6d423561e1ebcc4ca78cb414346bbb09695b328f 100644
 --- a/chrome/VERSION
 +++ b/chrome/VERSION
 @@ -1,4 +1,4 @@
diff --git a/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.cc.patch b/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.cc.patch
new file mode 100644
index 000000000000..f6c4cc75a72b
--- /dev/null
+++ b/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.cc.patch
@@ -0,0 +1,12 @@
+diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
+index 8cdfc6fbdddcb13bcd5e198cd6ebf53abfb0269d..719d296a16ae05e6d8c897af91684b4efa3527fa 100644
+--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
++++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
+@@ -479,6 +479,7 @@ void RecentTabsSubMenuModel::BuildTabsFromOtherDevices() {
+       BuildOtherDevicesTabItem(session_tag, *tabs_in_session[k]);
+     }  // for all tabs in one session
+ 
++    BRAVE_BUILD_TABS_FROM_OTHER_DEVICES
+     ++num_sessions_added;
+   }  // for all sessions
+ 
diff --git a/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.h.patch b/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.h.patch
new file mode 100644
index 000000000000..7964e3c63a94
--- /dev/null
+++ b/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model.h.patch
@@ -0,0 +1,12 @@
+diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
+index 911131f854b202f2aa8b48377d7e287ce27c652b..46aee818d4872246cc531a78fcace8b2fa25cb15 100644
+--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
++++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
+@@ -75,6 +75,7 @@ class RecentTabsSubMenuModel : public ui::SimpleMenuModel,
+   bool GetURLAndTitleForItemAtIndex(int index,
+                                     std::string* url,
+                                     base::string16* title);
++  BRAVE_RECENT_TABS_SUB_MENU_MODEL_H_
+ 
+  private:
+   struct TabNavigationItem;

From 1f0e0e4d3b8afdd3280151e79851a15b1101d950 Mon Sep 17 00:00:00 2001
From: AlexeyBarabash <alexey@brave.com>
Date: Tue, 17 Nov 2020 19:54:07 +0200
Subject: [PATCH 3/3] Updated test for More... item in History menu

---
 browser/ui/toolbar/BUILD.gn                   | 37 +++++++++++++++++++
 .../recent_tabs_sub_menu_model_unittest.cc    | 32 ++++++++++++++++
 ...cent_tabs_sub_menu_model_unittest.cc.patch | 12 ++++++
 test/BUILD.gn                                 |  4 ++
 4 files changed, 85 insertions(+)
 create mode 100644 browser/ui/toolbar/BUILD.gn
 create mode 100644 chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
 create mode 100644 patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model_unittest.cc.patch

diff --git a/browser/ui/toolbar/BUILD.gn b/browser/ui/toolbar/BUILD.gn
new file mode 100644
index 000000000000..af477bf1f7ee
--- /dev/null
+++ b/browser/ui/toolbar/BUILD.gn
@@ -0,0 +1,37 @@
+# Copyright (c) 2020 The Brave Authors. All rights reserved.
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this file,
+# You can obtain one at http://mozilla.org/MPL/2.0/.
+
+assert(!is_android && !is_ios)
+
+source_set("brave_app_menu_unit_test") {
+  testonly = true
+
+  public_deps = [
+    "//testing/gmock",
+    "//testing/gtest",
+  ]
+
+  sources = [
+    "//chrome/browser/ui/toolbar/recent_tabs_builder_test_helper.cc",
+    "//chrome/browser/ui/toolbar/recent_tabs_builder_test_helper.h",
+    "//chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc",
+    "//chrome/test/base/browser_with_test_window_test.cc",
+    "//chrome/test/base/browser_with_test_window_test.h",
+    "//chrome/test/base/menu_model_test.cc",
+    "//chrome/test/base/menu_model_test.h",
+  ]
+
+  deps = [
+    "//chrome/app:command_ids",
+    "//chrome/browser/ui:ui",
+    "//chrome/test:test_support",
+    "//components/constrained_window:constrained_window",
+    "//components/sessions:test_support",
+    "//components/sync_preferences:sync_preferences",
+    "//components/sync_sessions:sync_sessions",
+    "//content/test:test_support",
+    "//ui/views:test_support",
+  ]
+}
diff --git a/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc b/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
new file mode 100644
index 000000000000..6f104d5584fd
--- /dev/null
+++ b/chromium_src/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
@@ -0,0 +1,32 @@
+/* Copyright (c) 2020 The Brave Authors. All rights reserved.
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+// Disabling these tests because they refer g_brave_browser_process which is not
+// initialized in unit tests, is null and so they are crashing.
+// Not related to change in RecentTabsSubMenuModel for additional `More...`
+// menu item
+#define RecentlyClosedTabsFromCurrentSession \
+        DISABLED_RecentlyClosedTabsFromCurrentSession
+
+#define RecentlyClosedTabsAndWindowsFromLastSession \
+        DISABLED_RecentlyClosedTabsAndWindowsFromLastSession
+
+// Need to expect more items at that place, because Brave has additional item
+// `More...` which redirects to brave://history/syncedTabs
+// The perfect way is to have
+//     EXPECT_EQ(10, num_items)
+// instead
+//     EXPECT_EQ(9, num_items);
+// But in favor to reduce patch, just decrease num_items:
+#define BRAVE_MAX_TABS_PER_SESSION_AND_RECENCY \
+        --num_items;
+
+// The case when number of tabs on other device is <=4 so we do not add
+// `More...` item is tested by RecentTabsSubMenuModelTest.MaxSessionsAndRecency
+
+#include "../../../../../../chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc"
+
+#undef RecentlyClosedTabsAndWindowsFromLastSession
+#undef RecentlyClosedTabsFromCurrentSession
diff --git a/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model_unittest.cc.patch b/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model_unittest.cc.patch
new file mode 100644
index 000000000000..eef51141f05f
--- /dev/null
+++ b/patches/chrome-browser-ui-toolbar-recent_tabs_sub_menu_model_unittest.cc.patch
@@ -0,0 +1,12 @@
+diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
+index 737e6bae0885c335ee8028704827563fc1bd2eea..1fa69f30bf2bd62f7f4aa0faf1ab0beea4ddefcd 100644
+--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
++++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model_unittest.cc
+@@ -675,6 +675,7 @@ TEST_F(RecentTabsSubMenuModelTest, MaxTabsPerSessionAndRecency) {
+ 
+   TestRecentTabsSubMenuModel model(nullptr, browser());
+   int num_items = model.GetItemCount();
++  BRAVE_MAX_TABS_PER_SESSION_AND_RECENCY
+   EXPECT_EQ(9, num_items);
+ 
+   std::vector<base::string16> tab_titles =
diff --git a/test/BUILD.gn b/test/BUILD.gn
index 4a54e98f781e..418a6d5c68cd 100644
--- a/test/BUILD.gn
+++ b/test/BUILD.gn
@@ -258,6 +258,10 @@ test("brave_unit_tests") {
     ]
   }
 
+  if (!is_android && !is_ios) {
+     deps += [ "//brave/browser/ui/toolbar:brave_app_menu_unit_test" ]
+  }
+
   # On Windows, brave_install_static_unittests covers channel test.
   if (is_mac || is_linux) {
     sources += [ "//brave/common/brave_channel_info_unittest.cc" ]