Temporary fix for Infolist font issue.

chrome/browser/chromeos/input_method/candidate_window_view.h

Index: chrome/browser/chromeos/input_method/candidate_window_view.h
diff --git a/chrome/browser/chromeos/input_method/candidate_window_view.h b/chrome/browser/chromeos/input_method/candidate_window_view.h
index 3c9b5ccf920f5e2fa3d6874c1204e24b67ae5d1f..77f36b9b6eb2ed35e9e63968f76629a2083c2293 100644
--- a/chrome/browser/chromeos/input_method/candidate_window_view.h
+++ b/chrome/browser/chromeos/input_method/candidate_window_view.h
@@ -230,8 +230,9 @@ class InfolistWindowView : public views::View {
   void Hide();
   void DelayHide(unsigned int milliseconds);
   void UpdateCandidates(const InputMethodLookupTable& lookup_table);
-
   void ResizeAndMoveParentFrame();
+  gfx::Font GetTitleFont();

[Yusuke Sato, 2012/06/08 02:22:42]

const; ?

[Yusuke Sato, 2012/06/08 02:22:42]

I briefly checked ui/gfx/platform_font_pango.h. Looks like copying gfx::Font by
value is not that cheap. How about returning a const reference to the object?

[horo, 2012/06/08 03:01:12]

Done.

[horo, 2012/06/08 03:01:12]

Copying gfx::Font by value does not causes copying PlatformFont by value.
It causes only AddRef().

ui/gfx/font.h
// Font provides a wrapper around an underlying font. Copy and assignment
// operators are explicitly allowed, and cheap.

coped_refptr<PlatformFont> platform_font_;

+  gfx::Font GetDescriptionFont();

[Yusuke Sato, 2012/06/08 02:22:42]

ditto.

[horo, 2012/06/08 03:01:12]

Done.

 
  protected:
   // Override View::VisibilityChanged()
@@ -269,6 +270,11 @@ class InfolistWindowView : public views::View {
     const mozc::commands::InformationList* old_usages,
     const mozc::commands::InformationList* new_usages);
 
+  // Information title font
+  scoped_ptr<gfx::Font> title_font_;
+  // Information description font
+  scoped_ptr<gfx::Font> description_font_;
+
   DISALLOW_COPY_AND_ASSIGN(InfolistWindowView);
 };