Chromium Code Reviews| Index: chrome/browser/ui/cocoa/location_bar/zoom_decoration.mm |
| diff --git a/chrome/browser/ui/cocoa/location_bar/zoom_decoration.mm b/chrome/browser/ui/cocoa/location_bar/zoom_decoration.mm |
| index 8b5b27be631f9f5502c140957b970087e715fc73..7cdd8c5a74c3595fb67853d9c4a471350aefa620 100644 |
| --- a/chrome/browser/ui/cocoa/location_bar/zoom_decoration.mm |
| +++ b/chrome/browser/ui/cocoa/location_bar/zoom_decoration.mm |
| @@ -98,23 +98,26 @@ void ZoomDecoration::ShowAndUpdateUI(ui_zoom::ZoomController* zoom_controller, |
| NSString* tooltip_string, |
| bool location_bar_is_dark) { |
| if (ui::MaterialDesignController::IsModeMaterial()) { |
| - gfx::VectorIconId iconId = gfx::VectorIconId::VECTOR_ICON_NONE; |
| + gfx::VectorIconId icon_id = gfx::VectorIconId::VECTOR_ICON_NONE; |
| ui_zoom::ZoomController::RelativeZoom relative_zoom = |
| zoom_controller->GetZoomRelativeToDefault(); |
| if (relative_zoom == ui_zoom::ZoomController::ZOOM_BELOW_DEFAULT_ZOOM) { |
| - iconId = gfx::VectorIconId::ZOOM_MINUS; |
| + icon_id = gfx::VectorIconId::ZOOM_MINUS; |
| } else if (relative_zoom == |
| ui_zoom::ZoomController::ZOOM_ABOVE_DEFAULT_ZOOM) { |
| - iconId = gfx::VectorIconId::ZOOM_PLUS; |
| + icon_id = gfx::VectorIconId::ZOOM_PLUS; |
| } |
| - NSImage* theImage = nil; |
| - if (iconId != gfx::VectorIconId::VECTOR_ICON_NONE) { |
| - SkColor vectorIconColor = location_bar_is_dark ? SK_ColorWHITE |
| - : gfx::kChromeIconGrey; |
| - theImage = NSImageFromImageSkia(gfx::CreateVectorIcon(iconId, |
| - 16, |
| - vectorIconColor)); |
| + NSImage* the_image = nil; |
| + const int kIconSize = 16; |
| + if (icon_id != gfx::VectorIconId::VECTOR_ICON_NONE) { |
| + SkColor vector_icon_color = |
| + location_bar_is_dark ? SkColorSetA(SK_ColorWHITE, 0xCC) |
| + : gfx::kChromeIconGrey; |
| + the_image = |
| + NSImageFromImageSkia(gfx::CreateVectorIcon(icon_id, |
| + kIconSize, |
| + vector_icon_color)); |
| } else { |
| // Under Material Design there is no icon for ZOOM_NORMAL. This means |
| // it should be OK to set a nil image. However if the user is actively |
| @@ -124,10 +127,11 @@ void ZoomDecoration::ShowAndUpdateUI(ui_zoom::ZoomController* zoom_controller, |
| // position the bubble, but when the decoration's image is nil it has |
| // no frame. The result is the bubble positioned incorrectly. So, we have |
| // to set an empty image. |
| - theImage = |
| - [[[NSImage alloc] initWithSize:NSMakeSize(16, 16)] autorelease]; |
| + NSSize icon_size = NSMakeSize(kIconSize, kIconSize); |
| + the_image = |
| + [[[NSImage alloc] initWithSize:icon_size] autorelease]; |
|
tapted
2016/04/28 14:29:44
this might fit on the line above now
shrike
2016/04/28 21:05:35
Almost - the semi-colon end up being over the limi
|
| } |
| - SetImage(theImage); |
| + SetImage(the_image); |
| } else { |
| int image_id = IDR_ZOOM_NORMAL; |
| ui_zoom::ZoomController::RelativeZoom relative_zoom = |