Commit a5420aae authored by Avi Drissman's avatar Avi Drissman Committed by Chromium LUCI CQ

Remove unused Mac theme colors

With theme control refresh and other projects, many of the
Mac theme colors are no longer used. Since getting some of
those colors involved using old and deprecated APIs, remove
all colors that aren't currently being used.

Fixed: 1138531
Change-Id: Ie0fdd628f92104c68536efcac8a5a4c298aae982
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2572238
Commit-Queue: Avi Drissman <avi@chromium.org>
Commit-Queue: Robert Sesek <rsesek@chromium.org>
Auto-Submit: Avi Drissman <avi@chromium.org>
Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Reviewed-by: default avatarRobert Sesek <rsesek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833435}
parent 3ef5db76
...@@ -126,36 +126,6 @@ SkColor NSColorToSkColor(NSColor* color) { ...@@ -126,36 +126,6 @@ SkColor NSColorToSkColor(NSColor* color) {
alpha:1.]); alpha:1.]);
} }
SkColor MenuBackgroundColor() {
base::scoped_nsobject<NSBitmapImageRep> offscreen_rep(
[[NSBitmapImageRep alloc] initWithBitmapDataPlanes:nil
pixelsWide:1
pixelsHigh:1
bitsPerSample:8
samplesPerPixel:4
hasAlpha:YES
isPlanar:NO
colorSpaceName:NSDeviceRGBColorSpace
bytesPerRow:4
bitsPerPixel:32]);
CGContextRef context = static_cast<CGContextRef>([[NSGraphicsContext
graphicsContextWithBitmapImageRep:offscreen_rep] graphicsPort]);
CGRect rect = CGRectMake(0, 0, 1, 1);
HIThemeMenuDrawInfo draw_info;
draw_info.version = 0;
draw_info.menuType = kThemeMenuTypePopUp;
HIThemeDrawMenuBackground(&rect, &draw_info, context,
kHIThemeOrientationInverted);
NSUInteger pixel[4];
[offscreen_rep getPixel:pixel atX:0 y:0];
return NSColorToSkColor([NSColor colorWithDeviceRed:pixel[0] / 255.
green:pixel[1] / 255.
blue:pixel[2] / 255.
alpha:1.]);
}
} // namespace } // namespace
@interface SystemThemeObserver : NSObject { @interface SystemThemeObserver : NSObject {
...@@ -328,9 +298,6 @@ void ThemeHelperMac::LoadSystemColorsForCurrentAppearance( ...@@ -328,9 +298,6 @@ void ThemeHelperMac::LoadSystemColorsForCurrentAppearance(
for (size_t i = 0; i < blink::kMacSystemColorIDCount; ++i) { for (size_t i = 0; i < blink::kMacSystemColorIDCount; ++i) {
blink::MacSystemColorID color_id = static_cast<blink::MacSystemColorID>(i); blink::MacSystemColorID color_id = static_cast<blink::MacSystemColorID>(i);
switch (color_id) { switch (color_id) {
case blink::MacSystemColorID::kAlternateSelectedControl:
values[i] = NSColorToSkColor([NSColor alternateSelectedControlColor]);
break;
case blink::MacSystemColorID::kControlAccentBlueColor: { case blink::MacSystemColorID::kControlAccentBlueColor: {
NSColor* color = NSColor* color =
[NSColor colorWithCatalogName:@"System" [NSColor colorWithCatalogName:@"System"
...@@ -353,69 +320,15 @@ void ThemeHelperMac::LoadSystemColorsForCurrentAppearance( ...@@ -353,69 +320,15 @@ void ThemeHelperMac::LoadSystemColorsForCurrentAppearance(
values[i] = NSColorToSkColor([NSColor keyboardFocusIndicatorColor]); values[i] = NSColorToSkColor([NSColor keyboardFocusIndicatorColor]);
} }
break; break;
case blink::MacSystemColorID::kControlBackground:
values[i] = NSColorToSkColor([NSColor controlBackgroundColor]);
break;
case blink::MacSystemColorID::kControlDarkShadow:
values[i] = NSColorToSkColor([NSColor controlDarkShadowColor]);
break;
case blink::MacSystemColorID::kControlHighlight:
values[i] = NSColorToSkColor([NSColor controlHighlightColor]);
break;
case blink::MacSystemColorID::kControlLightHighlight:
values[i] = NSColorToSkColor([NSColor controlLightHighlightColor]);
break;
case blink::MacSystemColorID::kControlShadow:
values[i] = NSColorToSkColor([NSColor controlShadowColor]);
break;
case blink::MacSystemColorID::kControlText:
values[i] = NSColorToSkColor([NSColor controlTextColor]);
break;
case blink::MacSystemColorID::kDisabledControlText:
values[i] = NSColorToSkColor([NSColor disabledControlTextColor]);
break;
case blink::MacSystemColorID::kHeader:
values[i] = NSColorToSkColor([NSColor headerColor]);
break;
case blink::MacSystemColorID::kHighlight:
values[i] = NSColorToSkColor([NSColor highlightColor]);
break;
case blink::MacSystemColorID::kKeyboardFocusIndicator: case blink::MacSystemColorID::kKeyboardFocusIndicator:
values[i] = NSColorToSkColor([NSColor keyboardFocusIndicatorColor]); values[i] = NSColorToSkColor([NSColor keyboardFocusIndicatorColor]);
break; break;
case blink::MacSystemColorID::kMenuBackground:
values[i] = MenuBackgroundColor();
break;
case blink::MacSystemColorID::kScrollBar:
values[i] = NSColorToSkColor([NSColor scrollBarColor]);
break;
case blink::MacSystemColorID::kSecondarySelectedControl: case blink::MacSystemColorID::kSecondarySelectedControl:
values[i] = NSColorToSkColor([NSColor secondarySelectedControlColor]); values[i] = NSColorToSkColor([NSColor secondarySelectedControlColor]);
break; break;
case blink::MacSystemColorID::kSelectedMenuItemText:
values[i] = NSColorToSkColor([NSColor selectedMenuItemTextColor]);
break;
case blink::MacSystemColorID::kSelectedText:
values[i] = NSColorToSkColor([NSColor selectedTextColor]);
break;
case blink::MacSystemColorID::kSelectedTextBackground: case blink::MacSystemColorID::kSelectedTextBackground:
values[i] = NSColorToSkColor([NSColor selectedTextBackgroundColor]); values[i] = NSColorToSkColor([NSColor selectedTextBackgroundColor]);
break; break;
case blink::MacSystemColorID::kShadow:
values[i] = NSColorToSkColor([NSColor shadowColor]);
break;
case blink::MacSystemColorID::kText:
values[i] = NSColorToSkColor([NSColor textColor]);
break;
case blink::MacSystemColorID::kWindowBackground:
values[i] = NSColorToSkColor([NSColor windowBackgroundColor]);
break;
case blink::MacSystemColorID::kWindowFrame:
values[i] = NSColorToSkColor([NSColor windowFrameColor]);
break;
case blink::MacSystemColorID::kWindowFrameText:
values[i] = NSColorToSkColor([NSColor windowFrameTextColor]);
break;
case blink::MacSystemColorID::kCount: case blink::MacSystemColorID::kCount:
NOTREACHED(); NOTREACHED();
break; break;
......
...@@ -7,33 +7,13 @@ ...@@ -7,33 +7,13 @@
namespace blink { namespace blink {
// Named Mac system colors. Each of these corresponds to a selector on // Named Mac system colors.
// NSColor.
enum class MacSystemColorID { enum class MacSystemColorID {
kAlternateSelectedControl,
kControlAccentBlueColor, kControlAccentBlueColor,
kControlAccentColor, kControlAccentColor,
kControlBackground,
kControlDarkShadow,
kControlHighlight,
kControlLightHighlight,
kControlShadow,
kControlText,
kDisabledControlText,
kHeader,
kHighlight,
kKeyboardFocusIndicator, kKeyboardFocusIndicator,
kMenuBackground,
kScrollBar,
kSecondarySelectedControl, kSecondarySelectedControl,
kSelectedMenuItemText,
kSelectedText,
kSelectedTextBackground, kSelectedTextBackground,
kShadow,
kText,
kWindowBackground,
kWindowFrame,
kWindowFrameText,
kCount, kCount,
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment