From 2feaeccbaed3aaf878259263cafe6741d56fdccf Mon Sep 17 00:00:00 2001 From: "Brian R. Bondy" Date: Wed, 14 Jan 2026 07:11:28 -0500 Subject: [PATCH] Fix URL bar icon for Brave Origin brave:// pages (#33186) The reason this is needed is because there is no way to do multi-brand chromium_src overrides right now. The chromium_src/components/omnibox/browser/vector_icons/product_chrome_refresh.icon file contains the standard Brave lion logo. --- browser/ui/BUILD.gn | 1 + .../ui/toolbar/brave_location_bar_model_delegate.cc | 12 ++++++++++++ components/vector_icons/BUILD.gn | 1 + 3 files changed, 14 insertions(+) diff --git a/browser/ui/BUILD.gn b/browser/ui/BUILD.gn index e43bf502f27..5891af9589d 100644 --- a/browser/ui/BUILD.gn +++ b/browser/ui/BUILD.gn @@ -1214,6 +1214,7 @@ source_set("ui") { # "//brave/browser/extensions", "//brave/components/brave_extension:generated_resources", "//brave/components/brave_extension:static_resources", + "//brave/components/brave_origin/buildflags", "//brave/components/resources:static_resources", "//brave/components/resources:strings_grit", "//chrome/browser/extensions", diff --git a/browser/ui/toolbar/brave_location_bar_model_delegate.cc b/browser/ui/toolbar/brave_location_bar_model_delegate.cc index b21dee3096d..47d805688b9 100644 --- a/browser/ui/toolbar/brave_location_bar_model_delegate.cc +++ b/browser/ui/toolbar/brave_location_bar_model_delegate.cc @@ -10,6 +10,7 @@ #include "brave/browser/brave_shields/brave_shields_tab_helper.h" #include "brave/browser/ui/brave_scheme_utils.h" #include "brave/browser/ui/page_info/features.h" +#include "brave/components/brave_origin/buildflags/buildflags.h" #include "brave/components/constants/url_constants.h" #include "brave/components/constants/webui_url_constants.h" #include "brave/components/vector_icons/vector_icons.h" @@ -19,6 +20,7 @@ #include "components/prefs/pref_service.h" #include "content/public/browser/navigation_entry.h" #include "content/public/browser/web_contents.h" +#include "content/public/common/url_constants.h" #include "extensions/buildflags/buildflags.h" #if !BUILDFLAG(IS_ANDROID) @@ -86,6 +88,16 @@ bool BraveLocationBarModelDelegate::GetURL(GURL* url) const { const gfx::VectorIcon* BraveLocationBarModelDelegate::GetVectorIconOverride() const { +#if BUILDFLAG(IS_BRAVE_ORIGIN_BRANDED) + // For Brave Origin branded builds, use the branded product icon for + // chrome:// URLs instead of the omnibox product icon which uses the + // regular Brave branding. + GURL url; + if (GetURL(&url) && url.SchemeIs(content::kChromeUIScheme)) { + return &kLeoBraveIconOnlyFaceIcon; + } +#endif + // Defer to the Chromium implementation first. const gfx::VectorIcon* parent_icon = BrowserLocationBarModelDelegate::GetVectorIconOverride(); diff --git a/components/vector_icons/BUILD.gn b/components/vector_icons/BUILD.gn index f2c0ad5f71c..b1ff7a86bfa 100644 --- a/components/vector_icons/BUILD.gn +++ b/components/vector_icons/BUILD.gn @@ -20,6 +20,7 @@ aggregate_vector_icons("brave_components_vector_icons") { "leo_arrow_right.icon", "leo_arrow_small_right.icon", "leo_brave_icon_monochrome.icon", + "leo_brave_icon_only_face.icon", "leo_browser_add.icon", "leo_browser_bookmark_add.icon", "leo_browser_bookmark_normal.icon",