Show number of hidden items in new-style list notification.

ui/message_center/views/notification_view_md.cc

 // Copyright 2017 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_md.h"
 
 #include <stddef.h>
 
+#include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
+#include "base/strings/utf_string_conversions.h"
 #include "ui/base/cursor/cursor.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/image/image_skia_operations.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/gfx/skia_util.h"
 #include "ui/gfx/text_elider.h"
 #include "ui/message_center/message_center.h"
 #include "ui/message_center/message_center_style.h"
 #include "ui/message_center/notification.h"
@@ skipping 69 matching lines @@
 const gfx::ImageSkia GetProductIcon() {
   return gfx::CreateVectorIcon(kProductIcon, kSmallImageColor);
 }
 
 // ItemView ////////////////////////////////////////////////////////////////////
 
 // ItemViews are responsible for drawing each list notification item's title and
 // message next to each other within a single column.
 class ItemView : public views::View {
  public:
-  explicit ItemView(const message_center::NotificationItem& item);
+  explicit ItemView(const message_center::NotificationItem& item,
+                    int num_remaining);
   ~ItemView() override;
 
+  void SetRemainingCountVisible(bool visible);
+
  private:
   DISALLOW_COPY_AND_ASSIGN(ItemView);

[yoshiki, 2017/06/21 07:32:12]

please place this at the last.

[tetsui, 2017/06/22 07:25:23]

Done.

+
+  views::View* container_;
+  views::Label* remaining_count_;
 };
 
-ItemView::ItemView(const message_center::NotificationItem& item) {
-  SetLayoutManager(
+ItemView::ItemView(const message_center::NotificationItem& item,
+                   int num_remaining) {
+  container_ = new views::View;
+  AddChildView(container_);
+  SetLayoutManager(new views::FillLayout());
+
+  container_->SetLayoutManager(
       new views::BoxLayout(views::BoxLayout::kHorizontal, gfx::Insets(),
                            message_center::kItemTitleToMessagePadding));
 
   views::Label* title = new views::Label(item.title);
   title->set_collapse_when_hidden(true);
   title->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   title->SetEnabledColor(message_center::kRegularTextColor);
   title->SetBackgroundColor(message_center::kDimTextBackgroundColor);
-  AddChildView(title);
+  container_->AddChildView(title);
 
   views::Label* message = new views::Label(item.message);
   message->set_collapse_when_hidden(true);
   message->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   message->SetEnabledColor(message_center::kDimTextColor);
   message->SetBackgroundColor(message_center::kDimTextBackgroundColor);
-  AddChildView(message);
+  container_->AddChildView(message);
+
+  remaining_count_ = new views::Label();
+  remaining_count_->SetHorizontalAlignment(gfx::ALIGN_RIGHT);
+  remaining_count_->SetEnabledColor(message_center::kDimTextColor);
+  remaining_count_->SetBackgroundColor(message_center::kDimTextBackgroundColor);
+  AddChildView(remaining_count_);
+  if (num_remaining > 0) {
+    remaining_count_->SetText(base::UTF8ToUTF16("+") +
+                              base::IntToString16(num_remaining));

[yoshiki, 2017/06/21 07:32:12]

Could you use a localized string instead of just "+" and number? This string can
be localized in Android.

[tetsui, 2017/06/22 07:25:23]

Done.

+  }
+  remaining_count_->SetVisible(false);
 }
 
 ItemView::~ItemView() {}
 
+void ItemView::SetRemainingCountVisible(bool visible) {
+  remaining_count_->SetVisible(visible);
+}
+
 }  // anonymous namespace
 
 // ////////////////////////////////////////////////////////////
 // NotificationViewMD
 // ////////////////////////////////////////////////////////////
 
 views::View* NotificationViewMD::TargetForRect(views::View* root,
                                                const gfx::Rect& rect) {
   CHECK_EQ(root, this);
 
@@ skipping 254 matching lines @@
 }
 
 void NotificationViewMD::CreateOrUpdateListItemViews(
     const Notification& notification) {
   for (auto* item_view : item_views_)
     delete item_view;
   item_views_.clear();
 
   const std::vector<NotificationItem>& items = notification.items();
 
-  for (size_t i = 0; i < items.size() && i < kNotificationMaximumItems; ++i) {
-    ItemView* item_view = new ItemView(items[i]);
+  for (size_t i = 0; i < items.size() && i < kMaxLinesForExpandedMessageView;
+       ++i) {
+    ItemView* item_view = new ItemView(items[i], items.size() - i - 1);
     item_views_.push_back(item_view);
     left_content_->AddChildView(item_view);
   }
 
   // Needed when CreateOrUpdateViews is called for update.
   if (!item_views_.empty())
     left_content_->InvalidateLayout();
 }
 
 void NotificationViewMD::CreateOrUpdateIconView(
@@ skipping 148 matching lines @@
 
 void NotificationViewMD::UpdateViewForExpandedState(bool expanded) {
   header_row_->SetExpanded(expanded);
   if (message_view_) {
     message_view_->SetLineLimit(expanded ? kMaxLinesForExpandedMessageView
                                          : kMaxLinesForMessageView);
   }
   if (image_container_)
     image_container_->SetVisible(expanded);
   actions_row_->SetVisible(expanded && actions_row_->has_children());
-  for (size_t i = 1; i < item_views_.size(); ++i) {
+  for (size_t i = kMaxLinesForMessageView; i < item_views_.size(); ++i) {
     item_views_[i]->SetVisible(expanded);
   }
+  if (!item_views_.empty()) {
+    item_views_.front()->SetRemainingCountVisible(!expanded);
+    item_views_.back()->SetRemainingCountVisible(expanded);
+  }
 }
 
 void NotificationViewMD::UpdateControlButtonsVisibility() {
   const bool target_visibility = IsMouseHovered() || HasFocus() ||
                                  (header_row_->IsExpandButtonEnabled() &&
                                   header_row_->expand_button()->HasFocus()) ||
                                  (header_row_->IsCloseButtonEnabled() &&
                                   header_row_->close_button()->HasFocus()) ||
                                  (header_row_->IsSettingsButtonEnabled() &&
                                   header_row_->settings_button()->HasFocus());
 
   header_row_->SetControlButtonsVisible(target_visibility);
 }
 
 }  // namespace message_center