Remove background painting logic in BraveTabStrip (uplift to 1.64.x) (#…

…22371) Uplift of #22337 (squashed) to beta
brave · Mar 2, 2024 · 1423107 · 1423107
1 parent 183a574
commit 1423107
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/browser/ui/views/tabs/brave_tab_strip.cc b/browser/ui/views/tabs/brave_tab_strip.cc
@@ -404,17 +404,5 @@ void BraveTabStrip::Layout() {
   TabStrip::Layout();
 }
 
-void BraveTabStrip::OnPaintBackground(gfx::Canvas* canvas) {
-  // Unlike upstream, we are painting this view to an opaque layer in order to
-  // support layer-based shadows under the active tab. Paint a background so
-  // that all pixels are painted appropriately.
-  SkColor background_color =
-      ShouldShowVerticalTabs()
-          ? GetColorProvider()->GetColor(kColorToolbar)
-          : controller_->GetFrameColor(BrowserFrameActiveState::kUseCurrent);
-
-  canvas->DrawColor(background_color);
-}
-
 BEGIN_METADATA(BraveTabStrip, TabStrip)
 END_METADATA
diff --git a/browser/ui/views/tabs/brave_tab_strip.h b/browser/ui/views/tabs/brave_tab_strip.h
@@ -46,7 +46,6 @@ class BraveTabStrip : public TabStrip {
   SkColor GetTabSeparatorColor() const override;
   bool ShouldDrawStrokes() const override;
   void Layout() override;
-  void OnPaintBackground(gfx::Canvas* canvas) override;
 
   // Exposed for testing.
   static constexpr float kBraveMinimumContrastRatioForOutlines = 1.0816f;