From 4c5decf05e3ae1fffdb4864132968d84acf08eff Mon Sep 17 00:00:00 2001 From: Max Karolinskiy <41635752+mkarolin@users.noreply.github.com> Date: Wed, 13 Nov 2024 02:06:27 -0500 Subject: [PATCH] Upgrade from Chromium 131.0.6778.39 to Chromium 131.0.6778.69 (1.73.x). (#26515) * Upgrade from Chromium 131.0.6778.39 to Chromium 131.0.6778.69 * Update pins list timestamp * Recent tabs menu model has been updated. The upstream code has been updated to only show history clusters menu item when history clusters are enabled. We disable history clusters and don't need to remove that menu item manually any more. Chromium change: https://source.chromium.org/chromium/chromium/src/+/088cf43bda717ed6db36ae23388f58b2cac5ac0c commit 088cf43bda717ed6db36ae23388f58b2cac5ac0c Author: Toni Barzic Date: Fri Nov 8 16:53:49 2024 +0000 [Merge to 131] Don't show history clusters menu item if history clusters is disabled. Prior to this CL, history clusters could be disabled but Chrome would continue to show the history clusters menu item. Bug: 374013418, 376751203 * Disables failing upstream unit tests. --------- Co-authored-by: brave-builds --- .../ui/tabs/recent_tabs_sub_menu_model.cc | 8 +------- .../recent_tabs_sub_menu_model_unittest.cc | 19 +++++++++++++++---- .../input_file_parsers.cc | 4 ++-- package.json | 2 +- patches/chrome-browser-BUILD.gn.patch | 6 +++--- patches/chrome-browser-about_flags.cc.patch | 2 +- ...i-tabs-recent_tabs_sub_menu_model.cc.patch | 4 ++-- ...cent_tabs_sub_menu_model_unittest.cc.patch | 4 ++-- patches/chrome-test-BUILD.gn.patch | 2 +- patches/content-browser-BUILD.gn.patch | 4 ++-- ...-renderer_host-navigation_request.cc.patch | 2 +- patches/content-public-browser-BUILD.gn.patch | 4 ++-- patches/net-BUILD.gn.patch | 4 ++-- test/filters/unit_tests.filter | 8 ++++++++ 14 files changed, 43 insertions(+), 30 deletions(-) diff --git a/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc b/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc index 4c95dbba511b..9831da879aec 100644 --- a/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc +++ b/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc @@ -53,13 +53,7 @@ constexpr char kBraveSyncedTabsUrl[] = "brave://history/syncedTabs"; BraveRecentTabsSubMenuModel::BraveRecentTabsSubMenuModel( ui::AcceleratorProvider* accelerator_provider, Browser* browser) - : RecentTabsSubMenuModel(accelerator_provider, browser) { - // We disable history clusters feature, so this command won't work. - std::optional show_history_clusters_index = - GetIndexOfCommandId(IDC_SHOW_HISTORY_CLUSTERS_SIDE_PANEL); - CHECK(show_history_clusters_index); - RemoveItemAt(show_history_clusters_index.value()); -} + : RecentTabsSubMenuModel(accelerator_provider, browser) {} BraveRecentTabsSubMenuModel::~BraveRecentTabsSubMenuModel() {} diff --git a/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc b/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc index 83779f090906..d9b1264ff562 100644 --- a/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc +++ b/chromium_src/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc @@ -5,6 +5,7 @@ #include +#include "base/containers/to_vector.h" #include "base/strings/string_util.h" // Disabling these tests because they refer to g_brave_browser_process which is @@ -34,6 +35,12 @@ #define RecentlyClosedTabsAndWindowsFromLastSessionWithRefresh \ DISABLED_RecentlyClosedTabsAndWindowsFromLastSessionWithRefresh +// Disabling these tests because upstream code won't execute +// IDC_SHOW_HISTORY_CLUSTERS_SIDE_PANEL command when history clusters aren't +// enabled but the test doesn't check for it. +#define LogMenuMetricsForShowGroupedHistory \ + DISABLED_LogMenuMetricsForShowGroupedHistory + #define BRAVE_RECENT_TABS_SUB_MENU_MODEL_TEST \ void VerifyModel(const RecentTabsSubMenuModel& model, \ base::span data); \ @@ -47,6 +54,7 @@ #undef BRAVE_RECENT_TABS_SUB_MENU_MODEL_TEST +#undef LogMenuMetricsForShowGroupedHistory #undef RecentlyClosedTabsAndWindowsFromLastSessionWithRefresh #undef MaxTabsPerSessionAndRecency #undef MaxSessionsAndRecency @@ -58,18 +66,21 @@ // expectations void RecentTabsSubMenuModelTest::VerifyModel( const RecentTabsSubMenuModel& model, - base::span data) { - std::vector v_data{data.begin(), data.end()}; + base::span input) { + // We have to copy it over as we can not modify the input. + auto data = base::ToVector(input); // We replace the "Sign in to see tabs from other devices" menu command with // the non-command string "No tabs from other devices" and need to adjust the // data - auto& item_data = v_data.back(); + auto& item_data = data.back(); if (item_data.type == ui::MenuModel::TYPE_COMMAND) { item_data.enabled = false; } - ::VerifyModel(model, base::make_span(v_data.begin(), v_data.size())); + // The first two commands are History and History Clusters, but we disable + // History Clusters and upstream won't show it, so we should skip one command. + ::VerifyModel(model, base::make_span(data).subspan(1)); } void RecentTabsSubMenuModelTest::VerifyModel(const ui::MenuModel* model, diff --git a/chromium_src/net/tools/transport_security_state_generator/input_file_parsers.cc b/chromium_src/net/tools/transport_security_state_generator/input_file_parsers.cc index 02c8358c7fee..8df40d0c21ad 100644 --- a/chromium_src/net/tools/transport_security_state_generator/input_file_parsers.cc +++ b/chromium_src/net/tools/transport_security_state_generator/input_file_parsers.cc @@ -561,9 +561,9 @@ bool ParseCertificatesFile(std::string_view certs_input, Pinsets* pinsets, base::Time* timestamp) { constexpr std::string_view brave_certs = R"brave_certs( -# Last updated: Wed Nov 6 23:23:12 UTC 2024 +# Last updated: Tue Nov 12 20:40:33 UTC 2024 PinsListTimestamp -1730935392 +1731444033 # =====BEGIN BRAVE ROOTS ASC===== #From https://www.amazontrust.com/repository/ diff --git a/package.json b/package.json index afa46609a6d6..8fde39978ac1 100644 --- a/package.json +++ b/package.json @@ -68,7 +68,7 @@ "projects": { "chrome": { "dir": "src", - "tag": "131.0.6778.39", + "tag": "131.0.6778.69", "repository": { "url": "https://github.com/brave/chromium" } diff --git a/patches/chrome-browser-BUILD.gn.patch b/patches/chrome-browser-BUILD.gn.patch index 091d5fa0a7be..363c5690c948 100644 --- a/patches/chrome-browser-BUILD.gn.patch +++ b/patches/chrome-browser-BUILD.gn.patch @@ -1,8 +1,8 @@ diff --git a/chrome/browser/BUILD.gn b/chrome/browser/BUILD.gn -index 634b9a15c670a891d0edb5cb983b27133258f71e..941345fc99e30e6ca86a5b00ec86f18ea5461641 100644 +index 54279426733f7eeec00ee2ccf7c82771ae424c65..6cb0f560164665f7744986a2a425be4d30858632 100644 --- a/chrome/browser/BUILD.gn +++ b/chrome/browser/BUILD.gn -@@ -2514,6 +2514,7 @@ static_library("browser") { +@@ -2517,6 +2517,7 @@ static_library("browser") { "performance_monitor/metric_evaluator_helper_posix.h", ] } @@ -10,7 +10,7 @@ index 634b9a15c670a891d0edb5cb983b27133258f71e..941345fc99e30e6ca86a5b00ec86f18e if (!is_chromeos_ash) { sources += [ -@@ -8601,6 +8602,7 @@ static_library("browser_public_dependencies") { +@@ -8604,6 +8605,7 @@ static_library("browser_public_dependencies") { "//rlz:rlz_lib", ] } diff --git a/patches/chrome-browser-about_flags.cc.patch b/patches/chrome-browser-about_flags.cc.patch index 83b981ed1fc7..3453196f86c2 100644 --- a/patches/chrome-browser-about_flags.cc.patch +++ b/patches/chrome-browser-about_flags.cc.patch @@ -1,5 +1,5 @@ diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc -index c5b7d81d7731ef818878e568b9f8c8b277379082..2ff0fc7b34c69101855b8cf0b14aba0cb618c657 100644 +index ad599311ac46db300ec62d8841b8c89e3d6fd1fa..eb3763b158c650165c5dfb4344666431c7f632a4 100644 --- a/chrome/browser/about_flags.cc +++ b/chrome/browser/about_flags.cc @@ -4234,6 +4234,7 @@ const FeatureEntry::FeatureVariation diff --git a/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model.cc.patch b/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model.cc.patch index 4a747a8ddfb9..1d21d8e28d8e 100644 --- a/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model.cc.patch +++ b/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model.cc.patch @@ -1,8 +1,8 @@ diff --git a/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc b/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc -index 159fdebfb81c3ff0f1b2714a700184ebd6f8db8c..738871b2d6457440fb7e95fa95ebcd55a737a118 100644 +index dc1626195b0d06b6aed3d3c4ba0baa30842a9ccc..47f66ef97613abc74837ab6d2a32c75f6f59d682 100644 --- a/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc +++ b/chrome/browser/ui/tabs/recent_tabs_sub_menu_model.cc -@@ -457,6 +457,7 @@ void RecentTabsSubMenuModel::BuildTabsFromOtherDevices() { +@@ -463,6 +463,7 @@ void RecentTabsSubMenuModel::BuildTabsFromOtherDevices() { DCHECK(!session->GetSessionName().empty()); std::unique_ptr device_menu_model = CreateOtherDeviceSubMenu(session, tabs_in_session); diff --git a/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model_unittest.cc.patch b/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model_unittest.cc.patch index f4153d0e0560..cc75103383f7 100644 --- a/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model_unittest.cc.patch +++ b/patches/chrome-browser-ui-tabs-recent_tabs_sub_menu_model_unittest.cc.patch @@ -1,8 +1,8 @@ diff --git a/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc b/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc -index d0ac4166816514cf713d129581135fb8871e63f8..939103710846a867166193b3017359cb66e00c3a 100644 +index b082729482e20314e33ec3185af94ec03e5d914c..c33188f604aa57d4efe478cb3bacbd776fecf76f 100644 --- a/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc +++ b/chrome/browser/ui/tabs/recent_tabs_sub_menu_model_unittest.cc -@@ -165,6 +165,7 @@ class RecentTabsSubMenuModelTest : public BrowserWithTestWindowTest { +@@ -172,6 +172,7 @@ class RecentTabsSubMenuModelTest : public BrowserWithTestWindowTest { helper->VerifyExport(session_sync_service_->GetOpenTabsUIDelegate()); } diff --git a/patches/chrome-test-BUILD.gn.patch b/patches/chrome-test-BUILD.gn.patch index 4342b6bcb1b0..b6fde6eb3f0c 100644 --- a/patches/chrome-test-BUILD.gn.patch +++ b/patches/chrome-test-BUILD.gn.patch @@ -1,5 +1,5 @@ diff --git a/chrome/test/BUILD.gn b/chrome/test/BUILD.gn -index fc8f3418dd4d025c868d695eb63f9adc8e041246..6335658afccb129a043f351f5d655d3236a15be9 100644 +index f79688937c7c9a9ac81bbb4ab6259ee6b9a4e1ea..44bae7fa3af93db5698add7df2211c0d1535d369 100644 --- a/chrome/test/BUILD.gn +++ b/chrome/test/BUILD.gn @@ -581,6 +581,7 @@ static_library("test_support") { diff --git a/patches/content-browser-BUILD.gn.patch b/patches/content-browser-BUILD.gn.patch index cb25d509af30..417b6ee41957 100644 --- a/patches/content-browser-BUILD.gn.patch +++ b/patches/content-browser-BUILD.gn.patch @@ -1,8 +1,8 @@ diff --git a/content/browser/BUILD.gn b/content/browser/BUILD.gn -index 2d5c5f20907719f50663911e21f8eedf016f251f..5776935435d0be1070fa351ea0b11122ad6967ce 100644 +index 9fe4f5cf47fe860688b150290672ae5fc44170d9..484f7aa9af350aeb86c64686b50fe7c6bdc0862e 100644 --- a/content/browser/BUILD.gn +++ b/content/browser/BUILD.gn -@@ -3535,6 +3535,7 @@ source_set("browser") { +@@ -3533,6 +3533,7 @@ source_set("browser") { "profiling_utils.cc", ] } diff --git a/patches/content-browser-renderer_host-navigation_request.cc.patch b/patches/content-browser-renderer_host-navigation_request.cc.patch index 525545e7829f..a7050bd81135 100644 --- a/patches/content-browser-renderer_host-navigation_request.cc.patch +++ b/patches/content-browser-renderer_host-navigation_request.cc.patch @@ -1,5 +1,5 @@ diff --git a/content/browser/renderer_host/navigation_request.cc b/content/browser/renderer_host/navigation_request.cc -index 53bba48293111e74daba50dde617fb78f1bcc7ee..bbf7d08e363677ca0f0b5fd4d2f497afdc0b2adf 100644 +index 6f46f45b415972d94320b7e364b5233a0bcc014f..dc56449c4edb48880f374fa9e84ec499c4be3b47 100644 --- a/content/browser/renderer_host/navigation_request.cc +++ b/content/browser/renderer_host/navigation_request.cc @@ -3467,6 +3467,7 @@ void NavigationRequest::OnRequestRedirected( diff --git a/patches/content-public-browser-BUILD.gn.patch b/patches/content-public-browser-BUILD.gn.patch index 25a1d03e5dcb..8b89ca1c3359 100644 --- a/patches/content-public-browser-BUILD.gn.patch +++ b/patches/content-public-browser-BUILD.gn.patch @@ -1,8 +1,8 @@ diff --git a/content/public/browser/BUILD.gn b/content/public/browser/BUILD.gn -index a89d91b02ae014b5e60bbfa4204ae78d97665707..0ad3f72376be409c3b7999d7c04dc4f9ee3db02f 100644 +index c463b31fcf18ebbade38061baaac1f69230287a7..ca31fec27714aeb80268a2e506435c49c2239a2c 100644 --- a/content/public/browser/BUILD.gn +++ b/content/public/browser/BUILD.gn -@@ -740,6 +740,7 @@ source_set("browser_sources") { +@@ -742,6 +742,7 @@ source_set("browser_sources") { if (is_chromeos_lacros) { deps += [ "//chromeos/startup" ] } diff --git a/patches/net-BUILD.gn.patch b/patches/net-BUILD.gn.patch index a37a1b3d345a..db605f857616 100644 --- a/patches/net-BUILD.gn.patch +++ b/patches/net-BUILD.gn.patch @@ -1,8 +1,8 @@ diff --git a/net/BUILD.gn b/net/BUILD.gn -index 9b3cfa47d48a2510b4694e2b96660f99f30af911..71e15fecc69fd9dd0c13eeeff38a7db18304e48a 100644 +index a1d3a6dbf1eb623a22a6eb833bfcc770746e53c9..aad665d2e040630671d1837a017dc538629e3580 100644 --- a/net/BUILD.gn +++ b/net/BUILD.gn -@@ -1164,6 +1164,7 @@ component("net") { +@@ -1166,6 +1166,7 @@ component("net") { "reporting/reporting_uploader.h", ] } diff --git a/test/filters/unit_tests.filter b/test/filters/unit_tests.filter index 54c5584df6da..19f5200d929f 100644 --- a/test/filters/unit_tests.filter +++ b/test/filters/unit_tests.filter @@ -658,6 +658,14 @@ -SafetyHubHandlerTest.HandleGetSafetyHubEntryPointData_Subheader_OneModule -SafetyHubHandlerTest.HandleGetSafetyHubEntryPointData_Subheader_TwoModulesWithPassword +# These tests fail because our feature override for kSidePanelJourneys doesn't +# seem to work correctly in these tests. Have not diagnosed the specific reason +# why. We also run these tests as part of brave_unit_tests where they seem to +# work correctly. +-RecentTabsSubMenuModelTest.NoTabs +-RecentTabsSubMenuModelTest.OtherDevices +-RecentTabsSubMenuModelTest.OtherDevicesDynamicUpdate + # Tests below this point have not been diagnosed or had issues created yet. -AboutFlagsHistogramTest.* -AboutFlagsTest.*