Added padding below text in notifications.

ui/message_center/views/notification_view.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/message_center/views/notification_view.h"
 
 #include "base/command_line.h"
 #include "base/utf_string_conversions.h"
 #include "grit/ui_resources.h"
 #include "ui/base/accessibility/accessible_view_state.h"
@@ skipping 14 matching lines @@
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 
 namespace {
 
 // Dimensions.
 const int kIconColumnWidth = message_center::kNotificationIconSize;
 const int kLegacyIconSize = 40;
 const int kTextLeftPadding = kIconColumnWidth +
-    message_center::kIconToTextPadding;
-const int kTextBottomPadding = 6;
+                             message_center::kIconToTextPadding;
+const int kTextBottomPadding = 12;
 const int kTextRightPadding = 23;
 const int kItemTitleToMessagePadding = 3;
 const int kButtonHeight = 38;
 const int kButtonHorizontalPadding = 16;
 const int kButtonVecticalPadding = 0;
 const int kButtonIconTopPadding = 11;
 const int kButtonIconToTitlePadding = 16;
 const int kButtonTitleTopPadding = 0;
 
 const size_t kTitleCharacterLimit = 100;
@@ skipping 309 matching lines @@
                                    bool expanded)
     : MessageView(notification, observer, expanded) {
   // Create the opaque background that's above the view's shadow.
   background_view_ = new views::View();
   background_view_->set_background(MakeBackground());
 
   // Create the top_view_, which collects into a vertical box all content
   // at the top of the notification (to the right of the icon) except for the
   // close button.
   top_view_ = new ContainerView();
+  top_view_->set_border(MakeBorder(kTextTopPadding - 5,
+                                   kTextBottomPadding - 4,
+                                   0, 0));
 
   // Create the title view if appropriate.
   title_view_ = NULL;
   if (!notification.title().empty()) {
     title_view_ = new views::Label(
         MaybeTruncateText( notification.title(), kTitleCharacterLimit));
     title_view_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
     if (is_expanded())
       title_view_->SetMultiLine(true);
     else
       title_view_->SetElideBehavior(views::Label::ELIDE_AT_END);
     title_view_->SetFont(title_view_->font().DeriveFont(2));
     title_view_->SetEnabledColor(message_center::kRegularTextColor);
     title_view_->SetBackgroundColor(kRegularTextBackgroundColor);
-    title_view_->set_border(MakeBorder(kTextTopPadding, 3));
+    title_view_->set_border(MakeBorder(0, 0));

[msw, 2013/04/04 18:44:50]

nit: this seems unnecessary now.

     top_view_->AddChildView(title_view_);
   }
 
   // Create the message view if appropriate.
   message_view_ = NULL;
   if (!notification.message().empty()) {
     message_view_ = new views::Label(
         MaybeTruncateText(notification.message(), kMessageCharacterLimit));
     message_view_->SetVisible(!is_expanded() || !notification.items().size());
     message_view_->set_collapse_when_hidden(true);
     message_view_->SetHorizontalAlignment(gfx::ALIGN_LEFT);
     if (is_expanded())
       message_view_->SetMultiLine(true);
     else
       message_view_->SetElideBehavior(views::Label::ELIDE_AT_END);
     message_view_->SetEnabledColor(message_center::kRegularTextColor);
     message_view_->SetBackgroundColor(kRegularTextBackgroundColor);
-    message_view_->set_border(MakeBorder(0, 3));
+    message_view_->set_border(MakeBorder(1, 0));
     top_view_->AddChildView(message_view_);
   }
 
   // Create the list item views (up to a maximum).
   std::vector<NotificationItem> items = notification.items();
   for (size_t i = 0; i < items.size() && i < kNotificationMaximumItems; ++i) {
     ItemView* item_view = new ItemView(items[i]);
     item_view->SetVisible(is_expanded());
-    item_view->set_border(MakeBorder(0, 4));
+    item_view->set_border(MakeBorder(1, 0));
     item_views_.push_back(item_view);
     top_view_->AddChildView(item_view);
   }
 
   // Create the notification icon view.
   if (notification.type() == NOTIFICATION_TYPE_SIMPLE) {
     views::ImageView* icon_view = new views::ImageView();
     icon_view->SetImage(notification.icon().AsImageSkia());
     icon_view->SetImageSize(gfx::Size(kLegacyIconSize, kLegacyIconSize));
     icon_view->SetHorizontalAlignment(views::ImageView::CENTER);
@@ skipping 124 matching lines @@
                                              size_t limit) {
   // Currently just truncate the text by the total number of characters.
   // TODO(mukai): add better assumption like number of lines.
   if (!is_expanded())
     return text;
 
   return ui::TruncateString(text, limit);
 }
 
 }  // namespace message_center