Clean-up: Replaces Font with FontList in ui/message_center.

ui/message_center/views/bounded_label.h

Index: ui/message_center/views/bounded_label.h
diff --git a/ui/message_center/views/bounded_label.h b/ui/message_center/views/bounded_label.h
index 6d97c8850ccc7f3e449369767554b4012213646f..bb5456bf169061862f1f3b3c31d7ef35da21f2f6 100644
--- a/ui/message_center/views/bounded_label.h
+++ b/ui/message_center/views/bounded_label.h
@@ -14,8 +14,8 @@
 #include "ui/views/view.h"
 
 namespace gfx {
-class Font;
-}
+class FontList;
+}  // namespace gfx

[Jun Mukai, 2013/09/10 08:59:41]

you can omit "// namespace ~" comment in this size of section in the header.

[Yuki, 2013/09/10 09:16:34]

I know some of people omit it, but I was not able to find if it's okay or not in
the style guide, or in what cases we should omit it if it's okay.

As I read the style guide, it's at least okay to always put an ending comment. 
Considering there is no description how to omit it, probably it's recommended to
always write it.

[Jun Mukai, 2013/09/10 17:29:46]

If the coding style doesn't say specifically, usually follow the convention on
that module. As I quickly checked, ui/message_center tends to omit this thing,
see message_center.h or notification_list.h or views/message_popup_collection.h
(except for views/message_center_view.h)

It's still up to you though, you can go ahead with this, but I'd prefer to omit.

[Yuki, 2013/09/11 12:53:18]

Done.

 
 namespace message_center {
 
@@ -23,7 +23,7 @@ class InnerBoundedLabel;
 
 namespace test {
 class BoundedLabelTest;
-}
+}  // namespace test

[Jun Mukai, 2013/09/10 08:59:41]

ditto

[Yuki, 2013/09/10 09:16:34]

Done.

 
 // BoundedLabels display left aligned text up to a maximum number of lines, with
 // ellipsis at the end of the last line for any omitted text. BoundedLabel is a
@@ -33,7 +33,7 @@ class BoundedLabelTest;
 // bounded_label.cc file for details.
 class MESSAGE_CENTER_EXPORT BoundedLabel : public views::View {
  public:
-  BoundedLabel(const string16& text, gfx::Font font);
+  BoundedLabel(const string16& text, const gfx::FontList& font_list);
   BoundedLabel(const string16& text);
   virtual ~BoundedLabel();