Add missing context check in ChromeTypographyProvider::GetColor()

fix brave/brave-browser#7245
brave · Dec 21, 2019 · 1a97f22 · 1a97f22
1 parent ac9391b
commit 1a97f22
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/chromium_src/chrome/browser/ui/views/chrome_typography_provider.cc b/chromium_src/chrome/browser/ui/views/chrome_typography_provider.cc
@@ -9,10 +9,6 @@
 // Comes after the above includes.
 #include "chrome/browser/ui/views/chrome_typography_provider.h"
 
-namespace {
-  const SkColor kBraveGrey800 = SkColorSetRGB(0x3b, 0x3e, 0x4f);
-}
-
 #define ChromeTypographyProvider ChromeTypographyProvider_ChromiumImpl
 #include "../../../../../../chrome/browser/ui/views/chrome_typography_provider.cc"
 #undef ChromeTypographyProvider
@@ -24,8 +20,9 @@ SkColor ChromeTypographyProvider::GetColor(const views::View& view,
   const ui::NativeTheme* native_theme = view.GetNativeTheme();
   DCHECK(native_theme);
   if (ShouldIgnoreHarmonySpec(*native_theme)) {
-    return GetHarmonyTextColorForNonStandardNativeTheme(context, style,
-                                                        *native_theme);
+    return ChromeTypographyProvider_ChromiumImpl::GetColor(view,
+                                                           context,
+                                                           style);
   }
   // Override button text colors
   if (context == views::style::CONTEXT_BUTTON_MD) {
@@ -37,7 +34,7 @@ SkColor ChromeTypographyProvider::GetColor(const views::View& view,
         break;
       default:
         return native_theme->ShouldUseDarkColors() ? SK_ColorWHITE
-                                                   : kBraveGrey800;
+                                                   : gfx::kBraveGrey800;
     }
   }
   return ChromeTypographyProvider_ChromiumImpl::GetColor(view, context, style);

diff --git a/chromium_src/ui/gfx/color_palette.h b/chromium_src/ui/gfx/color_palette.h
@@ -11,6 +11,7 @@
 namespace gfx {
 
 constexpr SkColor kBraveGrey700 = SkColorSetRGB(0x4A, 0x4A, 0x4A);
+constexpr SkColor kBraveGrey800 = SkColorSetRGB(0x3b, 0x3e, 0x4f);
 constexpr SkColor kBraveNeutral300 = SkColorSetRGB(0xDE, 0xE2, 0xE6);
 constexpr SkColor kBraveNeutral800 = SkColorSetRGB(0x34, 0x3A, 0x40);