Remove sundry IsModeMaterial checks.

chrome/browser/ui/extensions/icon_with_badge_image_source.cc

Index: chrome/browser/ui/extensions/icon_with_badge_image_source.cc
diff --git a/chrome/browser/ui/extensions/icon_with_badge_image_source.cc b/chrome/browser/ui/extensions/icon_with_badge_image_source.cc
index bdef144b76bbac38184d6c7b168df9db1403fd60..f39049f31d3c3ae9af32f7ab641b0325fc0b7ffd 100644
--- a/chrome/browser/ui/extensions/icon_with_badge_image_source.cc
+++ b/chrome/browser/ui/extensions/icon_with_badge_image_source.cc
@@ -16,7 +16,6 @@
 #include "third_party/skia/include/core/SkPaint.h"
 #include "third_party/skia/include/core/SkRefCnt.h"
 #include "third_party/skia/include/core/SkTypeface.h"

[Peter Kasting, 2016/10/11 22:49:18]

Looks like this #include can disappear too?  Please look for any other dead
#includes in this and other files.

[Evan Stade, 2016/10/12 00:13:07]

Done.

-#include "ui/base/material_design/material_design_controller.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_palette.h"
@@ -28,24 +27,6 @@
 
 namespace {
 
-// Different platforms need slightly different constants to look good.
-// TODO(devlin): Comb through these and see if they are all still needed/
-// appropriate.
-#if defined(OS_WIN)
-const float kTextSize = 10;
-// The padding between the top of the badge and the top of the text.
-const int kTopTextPadding = -1;
-#elif defined(OS_MACOSX)
-const float kTextSize = 9.0;
-const int kTopTextPadding = 0;
-#elif defined(OS_CHROMEOS)
-const float kTextSize = 8.0;
-const int kTopTextPadding = 1;
-#elif defined(OS_POSIX)
-const float kTextSize = 9.0;
-const int kTopTextPadding = 0;
-#endif
-
 const int kPadding = 2;
 const int kBadgeHeight = 11;
 const int kMaxTextWidth = 23;
@@ -53,45 +34,6 @@ const int kMaxTextWidth = 23;
 // The minimum width for center-aligning the badge.
 const int kCenterAlignThreshold = 20;
 
-// Helper routine that returns a singleton SkPaint object configured for
-// rendering badge overlay text (correct font, typeface, etc).
-SkPaint* GetBadgeTextPaintSingleton() {
-#if defined(OS_MACOSX)
-  const char kPreferredTypeface[] = "Helvetica Bold";
-#else
-  const char kPreferredTypeface[] = "Arial";
-#endif
-
-  static SkPaint* text_paint = NULL;
-  if (!text_paint) {
-    text_paint = new SkPaint;
-    text_paint->setAntiAlias(true);
-    text_paint->setTextAlign(SkPaint::kLeft_Align);
-
-    sk_sp<SkTypeface> typeface(
-        SkTypeface::MakeFromName(kPreferredTypeface,
-                                 SkFontStyle::FromOldStyle(SkTypeface::kBold)));
-    // Skia doesn't do any font fallback---if the user is missing the font then
-    // typeface will be NULL. If we don't do manual fallback then we'll crash.
-    if (typeface) {
-      text_paint->setFakeBoldText(true);
-    } else {
-      // Fall back to the system font. We don't bold it because we aren't sure
-      // how it will look.
-      // For the most part this code path will only be hit on Linux systems
-      // that don't have Arial.
-      ResourceBundle& rb = ResourceBundle::GetSharedInstance();
-      const gfx::Font& base_font = rb.GetFont(ResourceBundle::BaseFont);
-      typeface = SkTypeface::MakeFromName(base_font.GetFontName().c_str(),
-                                          SkFontStyle());
-      DCHECK(typeface);
-    }
-
-    text_paint->setTypeface(std::move(typeface));
-  }
-  return text_paint;
-}
-
 gfx::ImageSkiaRep ScaleImageSkiaRep(const gfx::ImageSkiaRep& rep,
                                     int target_width_dp,
                                     float target_scale) {
@@ -166,19 +108,13 @@ void IconWithBadgeImageSource::PaintBadge(gfx::Canvas* canvas) {
                            ? SK_ColorWHITE
                            : badge_->text_color;
 
-  SkColor background_color = ui::MaterialDesignController::IsModeMaterial()
-                                 ? gfx::kGoogleBlue500
-                                 : SkColorSetRGB(218, 0, 24);
-  if (SkColorGetA(badge_->background_color) != SK_AlphaTRANSPARENT)
-    background_color = badge_->background_color;
-  // Make sure the background color is opaque. See http://crbug.com/619499
-  if (ui::MaterialDesignController::IsModeMaterial())
-    background_color = SkColorSetA(background_color, SK_AlphaOPAQUE);
-
-  canvas->Save();
+  SkColor background_color =
+      SkColorGetA(badge_->background_color) == SK_AlphaTRANSPARENT
+          ? gfx::kGoogleBlue500
+          // Make sure the background color is opaque. See
+          // http://crbug.com/619499

[Peter Kasting, 2016/10/11 22:49:18]

Nit: I'd move this comment above the whole statement; it's effectively what the
whole statement is accomplishing, and I think it's easier to read code when it
doesn't have comments mid-statement.

[Evan Stade, 2016/10/12 00:13:07]

Done.

+          : SkColorSetA(badge_->background_color, SK_AlphaOPAQUE);
 
-  SkPaint* text_paint = nullptr;
-  int text_width = 0;
   ResourceBundle* rb = &ResourceBundle::GetSharedInstance();
   gfx::FontList base_font = rb->GetFontList(ResourceBundle::BaseFont)
                                 .DeriveWithHeightUpperBound(kBadgeHeight);
@@ -199,30 +135,8 @@ void IconWithBadgeImageSource::PaintBadge(gfx::Canvas* canvas) {
     base_font = bigger_font;
   }
 
-  if (ui::MaterialDesignController::IsModeMaterial()) {
-    text_width =
+  const int text_width =
         std::min(kMaxTextWidth, canvas->GetStringWidth(utf16_text, base_font));

[Peter Kasting, 2016/10/11 22:49:18]

Nit: Should this still have a blank line after?

[Evan Stade, 2016/10/12 00:13:07]

I dunno, seems fine this way? In my view of the world the blank line was only
there because of the closing curly brace (I for one like blank lines after
scoped blocks like that).

-  } else {
-    text_paint = GetBadgeTextPaintSingleton();
-    text_paint->setColor(text_color);
-    float scale = canvas->image_scale();
-
-    // Calculate text width. Font width may not be linear with respect to the
-    // scale factor (e.g. when hinting is applied), so we need to use the font
-    // size that canvas actually uses when drawing a text.
-    text_paint->setTextSize(SkFloatToScalar(kTextSize) * scale);
-    SkScalar sk_text_width_in_pixel =
-        text_paint->measureText(badge_->text.c_str(), badge_->text.size());
-    text_paint->setTextSize(SkFloatToScalar(kTextSize));
-
-    // We clamp the width to a max size. SkPaint::measureText returns the width
-    // in pixel (as a result of scale multiplier), so convert
-    // sk_text_width_in_pixel back to DIP (density independent pixel) first.
-    text_width = std::min(
-        kMaxTextWidth, static_cast<int>(std::ceil(
-                           SkScalarToFloat(sk_text_width_in_pixel) / scale)));
-  }
-
   // Calculate badge size. It is clamped to a min width just because it looks
   // silly if it is too skinny.
   int badge_width = text_width + kPadding * 2;
@@ -243,53 +157,20 @@ void IconWithBadgeImageSource::PaintBadge(gfx::Canvas* canvas) {
   rect_paint.setAntiAlias(true);
   rect_paint.setColor(background_color);
 
-  if (ui::MaterialDesignController::IsModeMaterial()) {
-    // Clear part of the background icon.
-    gfx::Rect cutout_rect(rect);
-    cutout_rect.Inset(-1, -1);
-    SkPaint cutout_paint = rect_paint;
-    cutout_paint.setXfermodeMode(SkXfermode::kClear_Mode);
-    canvas->DrawRoundRect(cutout_rect, 2, cutout_paint);
-
-    // Paint the backdrop.
-    canvas->DrawRoundRect(rect, 1, rect_paint);
-
-    // Paint the text.
-    rect.Inset(std::max(kPadding, (rect.width() - text_width) / 2),
-               kBadgeHeight - base_font.GetHeight(), kPadding, 0);
-    canvas->DrawStringRect(utf16_text, base_font, text_color, rect);
-  } else {
-    // Paint the backdrop.
-    canvas->DrawRoundRect(rect, 2, rect_paint);
-
-    // Overlay the gradient. It is stretchy, so we do this in three parts.
-    ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
-    gfx::ImageSkia* gradient_left =
-        rb.GetImageSkiaNamed(IDR_BROWSER_ACTION_BADGE_LEFT);
-    gfx::ImageSkia* gradient_right =
-        rb.GetImageSkiaNamed(IDR_BROWSER_ACTION_BADGE_RIGHT);
-    gfx::ImageSkia* gradient_center =
-        rb.GetImageSkiaNamed(IDR_BROWSER_ACTION_BADGE_CENTER);
-
-    canvas->DrawImageInt(*gradient_left, rect.x(), rect.y());
-    canvas->TileImageInt(
-        *gradient_center, rect.x() + gradient_left->width(), rect.y(),
-        rect.width() - gradient_left->width() - gradient_right->width(),
-        rect.height());
-    canvas->DrawImageInt(*gradient_right,
-                         rect.right() - gradient_right->width(), rect.y());
-
-    // Finally, draw the text centered within the badge. We set a clip in case
-    // the text was too large.
-    rect.Inset(kPadding, 0);
-    canvas->ClipRect(rect);
-    canvas->sk_canvas()->drawText(
-        badge_->text.c_str(), badge_->text.size(),
-        SkFloatToScalar(rect.x() +
-                        static_cast<float>(rect.width() - text_width) / 2),
-        SkFloatToScalar(rect.y() + kTextSize + kTopTextPadding), *text_paint);
-  }
-  canvas->Restore();
+  // Clear part of the background icon.
+  gfx::Rect cutout_rect(rect);
+  cutout_rect.Inset(-1, -1);
+  SkPaint cutout_paint = rect_paint;
+  cutout_paint.setXfermodeMode(SkXfermode::kClear_Mode);
+  canvas->DrawRoundRect(cutout_rect, 2, cutout_paint);
+
+  // Paint the backdrop.
+  canvas->DrawRoundRect(rect, 1, rect_paint);
+
+  // Paint the text.
+  rect.Inset(std::max(kPadding, (rect.width() - text_width) / 2),
+             kBadgeHeight - base_font.GetHeight(), kPadding, 0);
+  canvas->DrawStringRect(utf16_text, base_font, text_color, rect);
 }
 
 void IconWithBadgeImageSource::PaintPageActionDecoration(gfx::Canvas* canvas) {