Supporting shrinking/enlarging for notifications

ui/message_center/views/message_popup_collection_unittest.cc

 // Copyright (c) 2013 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/message_popup_collection.h"
 
 #include <stddef.h>
 
 #include <list>
+#include <memory>

[Jun Mukai, 2016/05/04 17:00:09]

Is this necessary?

[dyaroshev, 2016/05/05 14:13:19]

Done.

 #include <utility>
 
 #include "base/message_loop/message_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/events/event.h"
 #include "ui/events/event_constants.h"
 #include "ui/events/event_utils.h"
 #include "ui/gfx/display.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/message_center/fake_message_center.h"
 #include "ui/message_center/views/desktop_popup_alignment_delegate.h"
 #include "ui/message_center/views/toast_contents_view.h"
 #include "ui/views/test/views_test_base.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_delegate.h"
 
+namespace {
+
+std::unique_ptr<message_center::Notification> CreateTestNotification(
+    std::string id,
+    std::string text) {
+  return base::WrapUnique(new message_center::Notification(
+      message_center::NOTIFICATION_TYPE_BASE_FORMAT, id,
+      base::UTF8ToUTF16("test title"), base::ASCIIToUTF16(text), gfx::Image(),
+      base::string16() /* display_source */, GURL(),
+      message_center::NotifierId(message_center::NotifierId::APPLICATION, id),
+      message_center::RichNotificationData(),
+      new message_center::NotificationDelegate()));
+}
+
+}  // namespace
+
 namespace message_center {
 namespace test {
 
 class MessagePopupCollectionTest : public views::ViewsTestBase {
  public:
   void SetUp() override {
     views::ViewsTestBase::SetUp();
     MessageCenter::Initialize();
     MessageCenter::Get()->DisableTimersForTest();
     alignment_delegate_.reset(new DesktopPopupAlignmentDelegate);
@@ skipping 474 matching lines @@
   WaitForTransitionsDone();
 
   // Confirms that there is no toast.
   EXPECT_EQ(0u, GetToastCounts());
   // But the notification still exists.
   EXPECT_EQ(1u, MessageCenter::Get()->NotificationCount());
 }
 
 #endif  // defined(OS_CHROMEOS)
 
+// When notifications were displayed on top, change of notification
+// size didn't affect corresponding widget.
+TEST_F(MessagePopupCollectionTest, ChangingNotificationSize) {
+  // Simulate a taskbar at the top.
+  SetDisplayInfo(gfx::Rect(0, 10, 600, 390),  // Work-area.
+                 gfx::Rect(0, 0, 600, 400));  // Display-bounds.
+
+  std::string id = AddNotification();

[Jun Mukai, 2016/05/04 17:00:09]

const std::string&

[dyaroshev, 2016/05/05 14:13:19]

not apliable anymore

+  WaitForTransitionsDone();
+
+  ToastContentsView* toast = GetToast(id);
+  ASSERT_TRUE(toast);
+
+  struct {
+    std::string name;
+    std::string text;
+  } test_cases[] = {
+      {"shrinking", ""}, {"enlarging", "abc\ndef\nghk\n"},
+  };

[stevenjb, 2016/05/04 23:39:02]

Should we also test a smaller notification after the larger one? I assume that
should have no effect?

[dyaroshev, 2016/05/05 14:13:19]

Done.

+
+  for (const auto& test_case : test_cases) {
+    EXPECT_EQ(toast->bounds(), GetWidget(id)->GetWindowBoundsInScreen())
+        << test_case.name << " before update";
+
+    MessageCenter::Get()->UpdateNotification(
+        id, CreateTestNotification(id, test_case.text));
+    EXPECT_EQ(toast->bounds(), GetWidget(id)->GetWindowBoundsInScreen())
+        << test_case.name << " before transition";
+
+    WaitForTransitionsDone();
+    EXPECT_EQ(toast->bounds(), GetWidget(id)->GetWindowBoundsInScreen())
+        << test_case.name << " after transition";
+  }
+
+  CloseAllToasts();
+  WaitForTransitionsDone();
+
+  // Restore simulated taskbar position to bottom.
+  SetDisplayInfo(gfx::Rect(0, 0, 600, 390),   // Work-area.
+                 gfx::Rect(0, 0, 600, 400));  // Display-bounds.

[Jun Mukai, 2016/05/04 17:00:09]

I believe you don't have to reset it.
If necessary, put into TearDown()

[dyaroshev, 2016/05/05 14:13:19]

I removed it in other tests too.

+}
+
 }  // namespace test
 }  // namespace message_center