Allow various font weights in gfx.

ui/base/resource/resource_bundle.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/base/resource/resource_bundle.h"
 
 #include <stdint.h>
 
 #include <limits>
 #include <utility>
@@ skipping 140 matching lines @@
     return gfx::ImageSkiaRep(image, scale);
   }
 
  private:
   ResourceBundle* rb_;
   const int resource_id_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourceBundleImageSource);
 };
 
+struct ResourceBundle::FontKey {
+  FontKey(int in_size_delta,
+          gfx::Font::FontStyle in_style,
+          gfx::Font::Weight in_weight)
+      : size_delta(in_size_delta), style(in_style), weight(in_weight) {}
+
+  ~FontKey() {}
+
+  bool operator==(const FontKey& rhs) const {
+    return size_delta == rhs.size_delta && style == rhs.style &&
+           weight == rhs.weight;
+  }
+
+  bool operator<(const FontKey& rhs) const {
+    if (size_delta != rhs.size_delta)
+      return size_delta < rhs.size_delta;
+    if (style != rhs.style)
+      return style < rhs.style;
+    return weight < rhs.weight;
+  }
+
+  int size_delta;
+  gfx::Font::FontStyle style;
+  gfx::Font::Weight weight;
+};
+
 // static
 std::string ResourceBundle::InitSharedInstanceWithLocale(
     const std::string& pref_locale,
     Delegate* delegate,
     LoadResources load_resources) {
   InitSharedInstance(delegate);
   if (load_resources == LOAD_COMMON_RESOURCES)
     g_shared_instance_->LoadCommonResources();
   std::string result = g_shared_instance_->LoadLocaleResources(pref_locale);
   g_shared_instance_->InitDefaultFontList();
@@ skipping 355 matching lines @@
     msg = base::string16(reinterpret_cast<const base::char16*>(data.data()),
                          data.length() / 2);
   } else if (encoding == ResourceHandle::UTF8) {
     msg = base::UTF8ToUTF16(data);
   }
   return msg;
 }
 
 const gfx::FontList& ResourceBundle::GetFontListWithDelta(
     int size_delta,
-    gfx::Font::FontStyle style) {
+    gfx::Font::FontStyle style,
+    gfx::Font::Weight weight) {
   base::AutoLock lock_scope(*images_and_fonts_lock_);
 
-  typedef std::pair<int, gfx::Font::FontStyle> Key;
-  const Key styled_key(size_delta, style);
+  const FontKey styled_key = {size_delta, style, weight};
 
   auto found = font_cache_.find(styled_key);
   if (found != font_cache_.end())
     return found->second;
 
-  const Key base_key(0, gfx::Font::NORMAL);
+  const FontKey base_key = {0, gfx::Font::NORMAL, gfx::Font::Weight::NORMAL};
   gfx::FontList& base = font_cache_[base_key];
   if (styled_key == base_key)
     return base;
 
   // Fonts of a given style are derived from the unstyled font of the same size.
   // Cache the unstyled font by first inserting a default-constructed font list.
   // Then, derive it for the initial insertion, or use the iterator that points
   // to the existing entry that the insertion collided with.
-  const Key sized_key(size_delta, gfx::Font::NORMAL);
+  const FontKey sized_key = {size_delta, gfx::Font::NORMAL,
+                             gfx::Font::Weight::NORMAL};

[tapted, 2016/03/22 23:31:32]

running `git cl format` should update this to a more chromey style

[Mikus, 2016/03/23 17:53:21]

This is what clang actually does

   auto sized = font_cache_.insert(std::make_pair(sized_key, gfx::FontList()));
   if (sized.second)
     sized.first->second = base.DeriveWithSizeDelta(size_delta);
   if (styled_key == sized_key)
     return sized.first->second;
 
   auto styled = font_cache_.insert(std::make_pair(styled_key, gfx::FontList()));
   DCHECK(styled.second);  // Otherwise font_cache_.find(..) would have found it.
-  styled.first->second = sized.first->second.DeriveWithStyle(
-      sized.first->second.GetFontStyle() | style);
+  styled.first->second = sized.first->second.Derive(
+      0, sized.first->second.GetFontStyle() | style, weight);

[msw, 2016/03/22 18:24:10]

Isn't the style of the |sized| font gfx::Font::Normal? If so, you can probably
omit "sized.first->second.GetFontStyle() | ", and just pass |style|.

[tapted, 2016/03/22 23:31:32]

This confused me initially too. There's actually no guarantee that a
default-constructed FontList creates something with a "normal" style. If someone
has configured their default GTK font to be italics, then.... I guess that's
what they want for bold fonts too.

[Mikus, 2016/03/23 17:53:21]

Acknowledged.

+
   return styled.first->second;
 }
 
 const gfx::Font& ResourceBundle::GetFontWithDelta(int size_delta,
                                                   gfx::Font::FontStyle style) {
   return GetFontListWithDelta(size_delta, style).GetPrimaryFont();
 }
 
 const gfx::FontList& ResourceBundle::GetFontList(FontStyle legacy_style) {
-  gfx::Font::FontStyle font_style = gfx::Font::NORMAL;
+  gfx::Font::Weight font_weight = gfx::Font::Weight::NORMAL;
   if (legacy_style == BoldFont || legacy_style == SmallBoldFont ||
       legacy_style == MediumBoldFont || legacy_style == LargeBoldFont)
-    font_style = gfx::Font::BOLD;
+    font_weight = gfx::Font::Weight::BOLD;
 
   int size_delta = 0;
   switch (legacy_style) {
     case SmallFont:
     case SmallBoldFont:
       size_delta = kSmallFontDelta;
       break;
     case MediumFont:
     case MediumBoldFont:
       size_delta = kMediumFontDelta;
       break;
     case LargeFont:
     case LargeBoldFont:
       size_delta = kLargeFontDelta;
       break;
     case BaseFont:
     case BoldFont:
       break;
   }
 
-  return GetFontListWithDelta(size_delta, font_style);
+  return GetFontListWithDelta(size_delta, gfx::Font::NORMAL, font_weight);
 }
 
 const gfx::Font& ResourceBundle::GetFont(FontStyle style) {
   return GetFontList(style).GetPrimaryFont();
 }
 
 void ResourceBundle::ReloadFonts() {
   base::AutoLock lock_scope(*images_and_fonts_lock_);
   InitDefaultFontList();
   font_cache_.clear();
@@ skipping 273 matching lines @@
 // static
 bool ResourceBundle::DecodePNG(const unsigned char* buf,
                                size_t size,
                                SkBitmap* bitmap,
                                bool* fell_back_to_1x) {
   *fell_back_to_1x = PNGContainsFallbackMarker(buf, size);
   return gfx::PNGCodec::Decode(buf, size, bitmap);
 }
 
 }  // namespace ui