From 6dc7b1aa510eadaa474995a7e4187850b789e651 Mon Sep 17 00:00:00 2001 From: Simon Hong Date: Tue, 20 Oct 2020 15:48:40 +0900 Subject: [PATCH 1/5] Don't show close button for inactive tab If mouse is hovered on tab, shows close button. --- browser/ui/BUILD.gn | 2 ++ browser/ui/views/tabs/brave_tab_strip.cc | 24 +++++++++++++++++++ browser/ui/views/tabs/brave_tab_strip.h | 23 ++++++++++++++++++ .../browser/ui/views/frame/browser_view.cc | 4 ++++ 4 files changed, 53 insertions(+) create mode 100644 browser/ui/views/tabs/brave_tab_strip.cc create mode 100644 browser/ui/views/tabs/brave_tab_strip.h diff --git a/browser/ui/BUILD.gn b/browser/ui/BUILD.gn index 39fdab56b5b0..444577337002 100644 --- a/browser/ui/BUILD.gn +++ b/browser/ui/BUILD.gn @@ -135,6 +135,8 @@ source_set("ui") { "views/tabs/brave_browser_tab_strip_controller.h", "views/tabs/brave_tab_context_menu_contents.cc", "views/tabs/brave_tab_context_menu_contents.h", + "views/tabs/brave_tab_strip.cc", + "views/tabs/brave_tab_strip.h", "views/toolbar/bookmark_button.cc", "views/toolbar/bookmark_button.h", ] diff --git a/browser/ui/views/tabs/brave_tab_strip.cc b/browser/ui/views/tabs/brave_tab_strip.cc new file mode 100644 index 000000000000..ddabe821d2d9 --- /dev/null +++ b/browser/ui/views/tabs/brave_tab_strip.cc @@ -0,0 +1,24 @@ +/* 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/views/tabs/brave_tab_strip.h" + +#include "chrome/browser/ui/views/tabs/tab.h" + +BraveTabStrip::~BraveTabStrip() = default; + +bool BraveTabStrip::ShouldHideCloseButtonForTab(Tab* tab) const { + bool should_hide = TabStrip::ShouldHideCloseButtonForTab(tab); + + // If upstream logic want to hide, follow it. + if (should_hide) + return should_hide; + + if (tab->IsActive()) + return false; + + // Only shows close button on tab when mouse is hovered on tab. + return !tab->mouse_hovered(); +} diff --git a/browser/ui/views/tabs/brave_tab_strip.h b/browser/ui/views/tabs/brave_tab_strip.h new file mode 100644 index 000000000000..e038d8484fdf --- /dev/null +++ b/browser/ui/views/tabs/brave_tab_strip.h @@ -0,0 +1,23 @@ +/* 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_VIEWS_TABS_BRAVE_TAB_STRIP_H_ +#define BRAVE_BROWSER_UI_VIEWS_TABS_BRAVE_TAB_STRIP_H_ + +#include "chrome/browser/ui/views/tabs/tab_strip.h" + +class BraveTabStrip : public TabStrip { + public: + using TabStrip::TabStrip; + ~BraveTabStrip() override; + BraveTabStrip(const BraveTabStrip&) = delete; + BraveTabStrip& operator=(const BraveTabStrip&) = delete; + + private: + // TabStrip overrides: + bool ShouldHideCloseButtonForTab(Tab* tab) const override; +}; + +#endif // BRAVE_BROWSER_UI_VIEWS_TABS_BRAVE_TAB_STRIP_H_ diff --git a/chromium_src/chrome/browser/ui/views/frame/browser_view.cc b/chromium_src/chrome/browser/ui/views/frame/browser_view.cc index bef7c42e0ee8..eff604d1f95a 100644 --- a/chromium_src/chrome/browser/ui/views/frame/browser_view.cc +++ b/chromium_src/chrome/browser/ui/views/frame/browser_view.cc @@ -4,10 +4,14 @@ * You can obtain one at http://mozilla.org/MPL/2.0/. */ #include "brave/browser/ui/views/tabs/brave_browser_tab_strip_controller.h" +#include "brave/browser/ui/views/tabs/brave_tab_strip.h" #include "brave/browser/ui/views/toolbar/brave_toolbar_view.h" +#include "chrome/browser/ui/views/frame/tab_strip_region_view.h" #define ToolbarView BraveToolbarView #define BrowserTabStripController BraveBrowserTabStripController +#define TabStrip BraveTabStrip #include "../../../../../../../chrome/browser/ui/views/frame/browser_view.cc" // NOLINT #undef ToolbarView #undef BrowserTabStripController +#undef TabStrip From 01979ba39b82acc445bafa9de79ca98516ec2754 Mon Sep 17 00:00:00 2001 From: Simon Hong Date: Tue, 20 Oct 2020 16:29:01 +0900 Subject: [PATCH 2/5] Decrease contrast of tab separator * light theme #BEBEBF * dark theme #393838 * no color changes at this time for private window / with tor tab separator and custom theme --- browser/ui/views/tabs/brave_tab_strip.cc | 23 +++++++++++++++++++++++ browser/ui/views/tabs/brave_tab_strip.h | 1 + 2 files changed, 24 insertions(+) diff --git a/browser/ui/views/tabs/brave_tab_strip.cc b/browser/ui/views/tabs/brave_tab_strip.cc index ddabe821d2d9..83bdc322035a 100644 --- a/browser/ui/views/tabs/brave_tab_strip.cc +++ b/browser/ui/views/tabs/brave_tab_strip.cc @@ -5,7 +5,14 @@ #include "brave/browser/ui/views/tabs/brave_tab_strip.h" +#include "brave/browser/profiles/profile_util.h" +#include "brave/browser/themes/brave_dark_mode_utils.h" +#include "chrome/browser/profiles/profile.h" +#include "chrome/browser/themes/theme_service.h" +#include "chrome/browser/themes/theme_service_factory.h" #include "chrome/browser/ui/views/tabs/tab.h" +#include "chrome/browser/ui/views/tabs/tab_strip_controller.h" +#include "third_party/skia/include/core/SkColor.h" BraveTabStrip::~BraveTabStrip() = default; @@ -22,3 +29,19 @@ bool BraveTabStrip::ShouldHideCloseButtonForTab(Tab* tab) const { // Only shows close button on tab when mouse is hovered on tab. return !tab->mouse_hovered(); } + +SkColor BraveTabStrip::GetTabSeparatorColor() const { + Profile* profile = controller()->GetProfile(); + if (!brave::IsRegularProfile(profile)) + return TabStrip::GetTabSeparatorColor(); + + // If custom theme is used, follow upstream separator color. + auto* theme_service = ThemeServiceFactory::GetForProfile(profile); + if (theme_service->GetThemeSupplier()) + return TabStrip::GetTabSeparatorColor(); + + bool dark_mode = dark_mode::GetActiveBraveDarkModeType() == + dark_mode::BraveDarkModeType::BRAVE_DARK_MODE_TYPE_DARK; + return dark_mode ? SkColorSetRGB(0x39, 0x38, 0x38) + : SkColorSetRGB(0xBE, 0xBF, 0xBF); +} diff --git a/browser/ui/views/tabs/brave_tab_strip.h b/browser/ui/views/tabs/brave_tab_strip.h index e038d8484fdf..bae690ba7c0c 100644 --- a/browser/ui/views/tabs/brave_tab_strip.h +++ b/browser/ui/views/tabs/brave_tab_strip.h @@ -18,6 +18,7 @@ class BraveTabStrip : public TabStrip { private: // TabStrip overrides: bool ShouldHideCloseButtonForTab(Tab* tab) const override; + SkColor GetTabSeparatorColor() const override; }; #endif // BRAVE_BROWSER_UI_VIEWS_TABS_BRAVE_TAB_STRIP_H_ From 6cf3dd7fb387d52714ef7ae72a6b7ae4e6610510 Mon Sep 17 00:00:00 2001 From: Simon Hong Date: Tue, 20 Oct 2020 21:42:53 +0900 Subject: [PATCH 3/5] Update tab title font & opacity * Inactive tab title has 70% opacity; on hover or focus show title at 100% opacity * Make active tab font Semibold --- .../browser/ui/views/tabs/tab_style_views.cc | 32 +++++++++++++++ ...ser-ui-views-tabs-tab_style_views.cc.patch | 40 +++++++++++++++++-- 2 files changed, 68 insertions(+), 4 deletions(-) create mode 100644 chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc diff --git a/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc new file mode 100644 index 000000000000..839e795092ee --- /dev/null +++ b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.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/. */ + +#define BRAVE_GM2_TAB_STYLE_H \ + private: \ + gfx::FontList semibold_font_; + +#define BRAVE_GM2_TAB_STYLE \ + semibold_font_ = normal_font_.DeriveWithWeight(gfx::Font::Weight::SEMIBOLD); + +#define BRAVE_CALCULATE_COLORS \ + const SkColor inactive_non_hovered_fg_color = \ + SkColorSetA(foreground_color, \ + gfx::Tween::IntValueBetween(0.7, \ + SK_AlphaTRANSPARENT, \ + SK_AlphaOPAQUE)); \ + const SkColor final_fg_color = \ + (tab_->IsActive() || tab_->mouse_hovered()) ? \ + foreground_color : inactive_non_hovered_fg_color; \ + return { final_fg_color, background_color}; + +#define BRAVE_GET_FONT_LIST \ + return tab_->IsActive() ? semibold_font_ : normal_font_; + +#include "../../../../../../../chrome/browser/ui/views/tabs/tab_style_views.cc" + +#undef BRAVE_GM2_TAB_STYLE_H +#undef BRAVE_GM2_TAB_STYLE +#undef BRAVE_CALCULATE_COLORS +#undef BRAVE_GET_FONT_LIST diff --git a/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch b/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch index 508e9d7863ff..87f9c75846aa 100644 --- a/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch +++ b/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch @@ -1,8 +1,24 @@ diff --git a/chrome/browser/ui/views/tabs/tab_style_views.cc b/chrome/browser/ui/views/tabs/tab_style_views.cc -index 950c967b5f87a6e3f531e3155a420002a7510d3a..a9b5b74a1a5c345727c1deb631fce7fcbb2762dc 100644 +index 950c967b5f87a6e3f531e3155a420002a7510d3a..d488846079f3d0521bd9d1bbe6ffeae158c3393b 100644 --- a/chrome/browser/ui/views/tabs/tab_style_views.cc +++ b/chrome/browser/ui/views/tabs/tab_style_views.cc -@@ -280,6 +280,7 @@ SkPath GM2TabStyle::GetPath(PathType path_type, +@@ -65,6 +65,7 @@ class GM2TabStyle : public TabStyleViews { + void ShowHover(ShowHoverStyle style) override; + void HideHover(HideHoverStyle style) override; + ++ BRAVE_GM2_TAB_STYLE_H + private: + // Gets the bounds for the leading and trailing separators for a tab. + SeparatorBounds GetSeparatorBounds(float scale) const; +@@ -191,6 +192,7 @@ GM2TabStyle::GM2TabStyle(Tab* tab) + views::style::STYLE_PRIMARY)), + heavy_font_(views::style::GetFont(views::style::CONTEXT_BUTTON_MD, + views::style::STYLE_PRIMARY)) { ++ BRAVE_GM2_TAB_STYLE + // TODO(dfried): create a new STYLE_PROMINENT or similar to use instead of + // repurposing CONTEXT_BUTTON_MD. + } +@@ -280,6 +282,7 @@ SkPath GM2TabStyle::GetPath(PathType path_type, const ShapeModifier shape_modifier = GetShapeModifier(path_type); const bool extend_left_to_bottom = shape_modifier & kNoLowerLeftArc; const bool extend_right_to_bottom = shape_modifier & kNoLowerRightArc; @@ -10,7 +26,23 @@ index 950c967b5f87a6e3f531e3155a420002a7510d3a..a9b5b74a1a5c345727c1deb631fce7fc SkPath path; -@@ -533,7 +534,7 @@ TabStyle::SeparatorBounds GM2TabStyle::GetSeparatorBounds(float scale) const { +@@ -459,6 +462,7 @@ TabStyle::TabColors GM2TabStyle::CalculateColors() const { + expected_opacity > 0.5f ? TabActive::kActive : TabActive::kInactive, + background_color); + ++ BRAVE_CALCULATE_COLORS + return {foreground_color, background_color}; + } + +@@ -472,6 +476,7 @@ const gfx::FontList& GM2TabStyle::GetFontList() const { + return heavy_font_; + } + ++ BRAVE_GET_FONT_LIST + return normal_font_; + } + +@@ -533,7 +538,7 @@ TabStyle::SeparatorBounds GM2TabStyle::GetSeparatorBounds(float scale) const { separator_bounds.leading = gfx::RectF(aligned_bounds.x() + corner_radius, aligned_bounds.y() + @@ -19,7 +51,7 @@ index 950c967b5f87a6e3f531e3155a420002a7510d3a..a9b5b74a1a5c345727c1deb631fce7fc separator_size.width(), separator_size.height()); separator_bounds.trailing = separator_bounds.leading; -@@ -888,12 +889,16 @@ void GM2TabStyle::PaintSeparators(gfx::Canvas* canvas) const { +@@ -888,12 +893,16 @@ void GM2TabStyle::PaintSeparators(gfx::Canvas* canvas) const { SK_AlphaOPAQUE)); }; From 9e6db01022d4c3b8bd01f3d63b3a171521a5be3d Mon Sep 17 00:00:00 2001 From: mkarolin Date: Tue, 20 Oct 2020 16:45:37 -0400 Subject: [PATCH 4/5] Reduce number of patches in tab_style_views.cc --- .../browser/ui/views/tabs/tab_style_views.cc | 68 +++++++++++++------ .../browser/ui/views/tabs/tab_style_views.h | 15 ++++ ...ser-ui-views-tabs-tab_style_views.cc.patch | 38 ++--------- 3 files changed, 69 insertions(+), 52 deletions(-) create mode 100644 chromium_src/chrome/browser/ui/views/tabs/tab_style_views.h diff --git a/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc index 839e795092ee..d1eeefef019c 100644 --- a/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc +++ b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc @@ -3,30 +3,56 @@ * 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/. */ -#define BRAVE_GM2_TAB_STYLE_H \ - private: \ +#include "chrome/browser/ui/views/tabs/tab_style_views.h" + +#define BRAVE_GM2_TAB_STYLE_H protected: +#define CreateForTab CreateForTab_ChromiumImpl +#include "../../../../../../../chrome/browser/ui/views/tabs/tab_style_views.cc" +#undef CreateForTab +#undef BRAVE_GM2_TAB_STYLE_H + +namespace { + +class BraveGM2TabStyle : public GM2TabStyle { + public: + explicit BraveGM2TabStyle(Tab* tab); + BraveGM2TabStyle(const GM2TabStyle&) = delete; + BraveGM2TabStyle& operator=(const BraveGM2TabStyle&) = delete; + + protected: + TabStyle::TabColors CalculateColors() const override; + const gfx::FontList& GetFontList() const override; + + private: gfx::FontList semibold_font_; +}; -#define BRAVE_GM2_TAB_STYLE \ - semibold_font_ = normal_font_.DeriveWithWeight(gfx::Font::Weight::SEMIBOLD); +BraveGM2TabStyle::BraveGM2TabStyle(Tab* tab) + : GM2TabStyle(tab), + semibold_font_( + normal_font_.DeriveWithWeight(gfx::Font::Weight::SEMIBOLD)) {} -#define BRAVE_CALCULATE_COLORS \ - const SkColor inactive_non_hovered_fg_color = \ - SkColorSetA(foreground_color, \ - gfx::Tween::IntValueBetween(0.7, \ - SK_AlphaTRANSPARENT, \ - SK_AlphaOPAQUE)); \ - const SkColor final_fg_color = \ - (tab_->IsActive() || tab_->mouse_hovered()) ? \ - foreground_color : inactive_non_hovered_fg_color; \ - return { final_fg_color, background_color}; +TabStyle::TabColors BraveGM2TabStyle::CalculateColors() const { + auto colors = GM2TabStyle::CalculateColors(); + const SkColor inactive_non_hovered_fg_color = SkColorSetA( + colors.foreground_color, + gfx::Tween::IntValueBetween(0.7, SK_AlphaTRANSPARENT, SK_AlphaOPAQUE)); + const SkColor final_fg_color = (tab_->IsActive() || tab_->mouse_hovered()) + ? colors.foreground_color + : inactive_non_hovered_fg_color; + return {final_fg_color, colors.background_color}; +} -#define BRAVE_GET_FONT_LIST \ - return tab_->IsActive() ? semibold_font_ : normal_font_; +const gfx::FontList& BraveGM2TabStyle::GetFontList() const { + const auto& font_list = GM2TabStyle::GetFontList(); + if (&font_list == &normal_font_ && tab_->IsActive()) { + return semibold_font_; + } + return font_list; +} -#include "../../../../../../../chrome/browser/ui/views/tabs/tab_style_views.cc" +} // namespace -#undef BRAVE_GM2_TAB_STYLE_H -#undef BRAVE_GM2_TAB_STYLE -#undef BRAVE_CALCULATE_COLORS -#undef BRAVE_GET_FONT_LIST +std::unique_ptr TabStyleViews::CreateForTab(Tab* tab) { + return std::make_unique(tab); +} diff --git a/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.h b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.h new file mode 100644 index 000000000000..4338258d43a8 --- /dev/null +++ b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.h @@ -0,0 +1,15 @@ +/* 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_VIEWS_TABS_TAB_STYLE_VIEWS_H_ +#define BRAVE_CHROMIUM_SRC_CHROME_BROWSER_UI_VIEWS_TABS_TAB_STYLE_VIEWS_H_ + +#define CreateForTab CreateForTab_ChromiumImpl(Tab* tab); \ + static std::unique_ptr CreateForTab + +#include "../../../../../../../chrome/browser/ui/views/tabs/tab_style_views.h" +#undef CreateForTab + +#endif // BRAVE_CHROMIUM_SRC_CHROME_BROWSER_UI_VIEWS_TABS_TAB_STYLE_VIEWS_H_ diff --git a/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch b/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch index 87f9c75846aa..e4573774c01a 100644 --- a/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch +++ b/patches/chrome-browser-ui-views-tabs-tab_style_views.cc.patch @@ -1,24 +1,16 @@ diff --git a/chrome/browser/ui/views/tabs/tab_style_views.cc b/chrome/browser/ui/views/tabs/tab_style_views.cc -index 950c967b5f87a6e3f531e3155a420002a7510d3a..d488846079f3d0521bd9d1bbe6ffeae158c3393b 100644 +index 950c967b5f87a6e3f531e3155a420002a7510d3a..5e139df9f2061712ec7ed890dc493d5e27a32691 100644 --- a/chrome/browser/ui/views/tabs/tab_style_views.cc +++ b/chrome/browser/ui/views/tabs/tab_style_views.cc -@@ -65,6 +65,7 @@ class GM2TabStyle : public TabStyleViews { - void ShowHover(ShowHoverStyle style) override; +@@ -66,6 +66,7 @@ class GM2TabStyle : public TabStyleViews { void HideHover(HideHoverStyle style) override; -+ BRAVE_GM2_TAB_STYLE_H private: ++ BRAVE_GM2_TAB_STYLE_H // Gets the bounds for the leading and trailing separators for a tab. SeparatorBounds GetSeparatorBounds(float scale) const; -@@ -191,6 +192,7 @@ GM2TabStyle::GM2TabStyle(Tab* tab) - views::style::STYLE_PRIMARY)), - heavy_font_(views::style::GetFont(views::style::CONTEXT_BUTTON_MD, - views::style::STYLE_PRIMARY)) { -+ BRAVE_GM2_TAB_STYLE - // TODO(dfried): create a new STYLE_PROMINENT or similar to use instead of - // repurposing CONTEXT_BUTTON_MD. - } -@@ -280,6 +282,7 @@ SkPath GM2TabStyle::GetPath(PathType path_type, + +@@ -280,6 +281,7 @@ SkPath GM2TabStyle::GetPath(PathType path_type, const ShapeModifier shape_modifier = GetShapeModifier(path_type); const bool extend_left_to_bottom = shape_modifier & kNoLowerLeftArc; const bool extend_right_to_bottom = shape_modifier & kNoLowerRightArc; @@ -26,23 +18,7 @@ index 950c967b5f87a6e3f531e3155a420002a7510d3a..d488846079f3d0521bd9d1bbe6ffeae1 SkPath path; -@@ -459,6 +462,7 @@ TabStyle::TabColors GM2TabStyle::CalculateColors() const { - expected_opacity > 0.5f ? TabActive::kActive : TabActive::kInactive, - background_color); - -+ BRAVE_CALCULATE_COLORS - return {foreground_color, background_color}; - } - -@@ -472,6 +476,7 @@ const gfx::FontList& GM2TabStyle::GetFontList() const { - return heavy_font_; - } - -+ BRAVE_GET_FONT_LIST - return normal_font_; - } - -@@ -533,7 +538,7 @@ TabStyle::SeparatorBounds GM2TabStyle::GetSeparatorBounds(float scale) const { +@@ -533,7 +535,7 @@ TabStyle::SeparatorBounds GM2TabStyle::GetSeparatorBounds(float scale) const { separator_bounds.leading = gfx::RectF(aligned_bounds.x() + corner_radius, aligned_bounds.y() + @@ -51,7 +27,7 @@ index 950c967b5f87a6e3f531e3155a420002a7510d3a..d488846079f3d0521bd9d1bbe6ffeae1 separator_size.width(), separator_size.height()); separator_bounds.trailing = separator_bounds.leading; -@@ -888,12 +893,16 @@ void GM2TabStyle::PaintSeparators(gfx::Canvas* canvas) const { +@@ -888,12 +890,16 @@ void GM2TabStyle::PaintSeparators(gfx::Canvas* canvas) const { SK_AlphaOPAQUE)); }; From 68527dcdf1397edc471532463d1c6010286ac7a3 Mon Sep 17 00:00:00 2001 From: Simon Hong Date: Wed, 21 Oct 2020 08:24:57 +0900 Subject: [PATCH 5/5] Modify copy ctor decls --- chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc index d1eeefef019c..3d1ef173dc01 100644 --- a/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc +++ b/chromium_src/chrome/browser/ui/views/tabs/tab_style_views.cc @@ -16,7 +16,7 @@ namespace { class BraveGM2TabStyle : public GM2TabStyle { public: explicit BraveGM2TabStyle(Tab* tab); - BraveGM2TabStyle(const GM2TabStyle&) = delete; + BraveGM2TabStyle(const BraveGM2TabStyle&) = delete; BraveGM2TabStyle& operator=(const BraveGM2TabStyle&) = delete; protected: