Move message_center::CustomNotificationView to arc::ArcNotificationView

ui/arc/notification/arc_notification_view_unittest.cc

 // Copyright 2016 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 <memory>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/strings/utf_string_conversions.h"
 #include "third_party/skia/include/core/SkColor.h"
+#include "ui/arc/notification/arc_notification_content_view_delegate.h"
+#include "ui/arc/notification/arc_notification_view.h"
 #include "ui/base/ime/dummy_text_input_client.h"
 #include "ui/base/ime/input_method.h"
 #include "ui/base/ime/text_input_client.h"
 #include "ui/events/event.h"
 #include "ui/events/event_utils.h"
 #include "ui/message_center/notification.h"
 #include "ui/message_center/notification_delegate.h"
-#include "ui/message_center/views/custom_notification_view.h"
 #include "ui/message_center/views/message_center_controller.h"
 #include "ui/message_center/views/message_view_factory.h"
 #include "ui/views/background.h"
 #include "ui/views/controls/button/image_button.h"
 #include "ui/views/test/views_test_base.h"
 
-namespace message_center {
+namespace arc {
 
 namespace {
 
 const SkColor kBackgroundColor = SK_ColorGREEN;
 
-class TestCustomView : public views::View {
+class TestNotificationContentsView : public views::View {
  public:
-  TestCustomView() {
+  TestNotificationContentsView() {
     SetFocusBehavior(FocusBehavior::ALWAYS);
     set_background(views::Background::CreateSolidBackground(kBackgroundColor));
   }
-  ~TestCustomView() override {}
+  ~TestNotificationContentsView() override {}

[hidehiko, 2017/05/11 14:22:47]

Similarly, s/{}/= default;/

Ditto for below.

[yoshiki, 2017/05/12 08:42:38]

Done.

 
   void Reset() {
     mouse_event_count_ = 0;
     keyboard_event_count_ = 0;
   }
 
   void set_preferred_size(gfx::Size size) { preferred_size_ = size; }
 
   // views::View
   gfx::Size GetPreferredSize() const override { return preferred_size_; }
@@ skipping 13 matching lines @@
   }
 
   int mouse_event_count() const { return mouse_event_count_; }
   int keyboard_event_count() const { return keyboard_event_count_; }
 
  private:
   int mouse_event_count_ = 0;
   int keyboard_event_count_ = 0;
   gfx::Size preferred_size_ = gfx::Size(100, 100);
 
-  DISALLOW_COPY_AND_ASSIGN(TestCustomView);
+  DISALLOW_COPY_AND_ASSIGN(TestNotificationContentsView);
 };
 
-class TestContentViewDelegate : public CustomNotificationContentViewDelegate {
+class TestContentViewDelegate : public ArcNotificationContentViewDelegate {
  public:
   bool IsCloseButtonFocused() const override { return false; }
   void RequestFocusOnCloseButton() override {}
   void UpdateControlButtonsVisibility() override {}
 };
 
-class TestNotificationDelegate : public NotificationDelegate {
+class TestNotificationDelegate : public message_center::NotificationDelegate {
  public:
   TestNotificationDelegate() {}
 
   // NotificateDelegate
-  std::unique_ptr<CustomContent> CreateCustomContent() override {
-    return base::MakeUnique<CustomContent>(
-        base::MakeUnique<TestCustomView>(),
-        base::MakeUnique<TestContentViewDelegate>());
+  std::unique_ptr<message_center::MessageView> CreateCustomMessageView(
+      message_center::MessageCenterController* controller,
+      const message_center::Notification& notification) override {
+    return base::MakeUnique<ArcNotificationView>(
+        base::MakeUnique<TestNotificationContentsView>(),
+        base::MakeUnique<TestContentViewDelegate>(), controller, notification);
   }
 
  private:
   ~TestNotificationDelegate() override {}
 
   DISALLOW_COPY_AND_ASSIGN(TestNotificationDelegate);
 };
 
-class TestMessageCenterController : public MessageCenterController {
+class TestMessageCenterController
+    : public message_center::MessageCenterController {
  public:
   TestMessageCenterController() {}
 
   // MessageCenterController
   void ClickOnNotification(const std::string& notification_id) override {
     // For this test, this method should not be invoked.
     NOTREACHED();
   }
 
   void RemoveNotification(const std::string& notification_id,
                           bool by_user) override {
     removed_ids_.insert(notification_id);
   }
 
   std::unique_ptr<ui::MenuModel> CreateMenuModel(
-      const NotifierId& notifier_id,
+      const message_center::NotifierId& notifier_id,
       const base::string16& display_source) override {
     // For this test, this method should not be invoked.
     NOTREACHED();
     return nullptr;
   }
 
   bool HasClickedListener(const std::string& notification_id) override {
     return false;
   }
 
@@ skipping 32 matching lines @@
   void set_text_input_type(ui::TextInputType type) { type_ = type; }
 
  private:
   ui::TextInputType type_ = ui::TEXT_INPUT_TYPE_NONE;
 
   DISALLOW_COPY_AND_ASSIGN(TestTextInputClient);
 };
 
 }  // namespace
 
-class CustomNotificationViewTest : public views::ViewsTestBase {
+class ArcNotificationViewTest : public views::ViewsTestBase {
  public:
-  CustomNotificationViewTest() {}
-  ~CustomNotificationViewTest() override {}
+  ArcNotificationViewTest() {}
+  ~ArcNotificationViewTest() override {}
 
   // views::ViewsTestBase
   void SetUp() override {
     views::ViewsTestBase::SetUp();
 
     notification_delegate_ = new TestNotificationDelegate;
 
-    notification_.reset(new Notification(
-        NOTIFICATION_TYPE_CUSTOM, std::string("notification id"),
-        base::UTF8ToUTF16("title"), base::UTF8ToUTF16("message"), gfx::Image(),
+    notification_.reset(new message_center::Notification(

[hidehiko, 2017/05/11 14:22:47]

nit/style: Could you use base::MakeUnique<> instead?

[yoshiki, 2017/05/12 08:42:38]

Done.

+        message_center::NOTIFICATION_TYPE_CUSTOM,
+        std::string("notification id"), base::UTF8ToUTF16("title"),
+        base::UTF8ToUTF16("message"), gfx::Image(),
         base::UTF8ToUTF16("display source"), GURL(),
-        NotifierId(NotifierId::APPLICATION, "extension_id"),
+        message_center::NotifierId(message_center::NotifierId::APPLICATION,
+                                   "extension_id"),
         message_center::RichNotificationData(), notification_delegate_.get()));
 
-    notification_view_.reset(static_cast<CustomNotificationView*>(
-        MessageViewFactory::Create(controller(), *notification_, true)));
+    notification_view_.reset(static_cast<ArcNotificationView*>(

[hidehiko, 2017/05/11 14:22:47]

Ditto.

[yoshiki, 2017/05/12 08:42:38]

This line doesn't construct an object so we can't use MakeUnique here.

+        message_center::MessageViewFactory::Create(controller(), *notification_,
+                                                   true)));
     notification_view_->set_owned_by_client();
 
     views::Widget::InitParams init_params(
         CreateParams(views::Widget::InitParams::TYPE_POPUP));
     views::Widget* widget = new views::Widget();
     widget->Init(init_params);
     widget->SetContentsView(notification_view_.get());
     widget->SetSize(notification_view_->GetPreferredSize());
   }
 
@@ skipping 28 matching lines @@
   void KeyPress(ui::KeyboardCode key_code) {
     ui::KeyEvent event(ui::ET_KEY_PRESSED, key_code, ui::EF_NONE);
     widget()->OnKeyEvent(&event);
   }
 
   void UpdateNotificationViews() {
     notification_view()->UpdateWithNotification(*notification());
   }
 
   TestMessageCenterController* controller() { return &controller_; }
-  Notification* notification() { return notification_.get(); }
-  TestCustomView* custom_view() {
-    return static_cast<TestCustomView*>(notification_view_->contents_view_);
+  message_center::Notification* notification() { return notification_.get(); }
+  TestNotificationContentsView* contents_view() {
+    return static_cast<TestNotificationContentsView*>(
+        notification_view_->contents_view_);
   }
   views::Widget* widget() { return notification_view_->GetWidget(); }
-  CustomNotificationView* notification_view() {
-    return notification_view_.get();
-  }
+  ArcNotificationView* notification_view() { return notification_view_.get(); }
 
  private:
   TestMessageCenterController controller_;
   scoped_refptr<TestNotificationDelegate> notification_delegate_;
-  std::unique_ptr<Notification> notification_;
-  std::unique_ptr<CustomNotificationView> notification_view_;
+  std::unique_ptr<message_center::Notification> notification_;
+  std::unique_ptr<ArcNotificationView> notification_view_;
 
-  DISALLOW_COPY_AND_ASSIGN(CustomNotificationViewTest);
+  DISALLOW_COPY_AND_ASSIGN(ArcNotificationViewTest);
 };
 
-TEST_F(CustomNotificationViewTest, Background) {
+TEST_F(ArcNotificationViewTest, Background) {
   EXPECT_EQ(kBackgroundColor, GetBackgroundColor());
 }
 
-TEST_F(CustomNotificationViewTest, Events) {
+TEST_F(ArcNotificationViewTest, Events) {
   widget()->Show();
-  custom_view()->RequestFocus();
+  contents_view()->RequestFocus();
 
-  EXPECT_EQ(0, custom_view()->mouse_event_count());
+  EXPECT_EQ(0, contents_view()->mouse_event_count());
   gfx::Point cursor_location(1, 1);
-  views::View::ConvertPointToWidget(custom_view(), &cursor_location);
+  views::View::ConvertPointToWidget(contents_view(), &cursor_location);
   PerformClick(cursor_location);
-  EXPECT_EQ(2, custom_view()->mouse_event_count());
+  EXPECT_EQ(2, contents_view()->mouse_event_count());
 
   ui::MouseEvent move(ui::ET_MOUSE_MOVED, cursor_location, cursor_location,
                       ui::EventTimeForNow(), ui::EF_NONE, ui::EF_NONE);
   widget()->OnMouseEvent(&move);
-  EXPECT_EQ(3, custom_view()->mouse_event_count());
+  EXPECT_EQ(3, contents_view()->mouse_event_count());
 
-  EXPECT_EQ(0, custom_view()->keyboard_event_count());
+  EXPECT_EQ(0, contents_view()->keyboard_event_count());
   KeyPress(ui::VKEY_A);
-  EXPECT_EQ(1, custom_view()->keyboard_event_count());
+  EXPECT_EQ(1, contents_view()->keyboard_event_count());
 }
 
-TEST_F(CustomNotificationViewTest, PressBackspaceKey) {
+TEST_F(ArcNotificationViewTest, PressBackspaceKey) {
   std::string notification_id = notification()->id();
-  custom_view()->RequestFocus();
+  contents_view()->RequestFocus();
 
-  ui::InputMethod* input_method = custom_view()->GetInputMethod();
+  ui::InputMethod* input_method = contents_view()->GetInputMethod();
   ASSERT_TRUE(input_method);
   TestTextInputClient text_input_client;
   input_method->SetFocusedTextInputClient(&text_input_client);
   ASSERT_EQ(&text_input_client, input_method->GetTextInputClient());
 
   EXPECT_FALSE(controller()->IsRemoved(notification_id));
   PerformKeyEvents(ui::VKEY_BACK);
   EXPECT_TRUE(controller()->IsRemoved(notification_id));
 
   input_method->SetFocusedTextInputClient(nullptr);
 }
 
-TEST_F(CustomNotificationViewTest, PressBackspaceKeyOnEditBox) {
+TEST_F(ArcNotificationViewTest, PressBackspaceKeyOnEditBox) {
   std::string notification_id = notification()->id();
-  custom_view()->RequestFocus();
+  contents_view()->RequestFocus();
 
-  ui::InputMethod* input_method = custom_view()->GetInputMethod();
+  ui::InputMethod* input_method = contents_view()->GetInputMethod();
   ASSERT_TRUE(input_method);
   TestTextInputClient text_input_client;
   input_method->SetFocusedTextInputClient(&text_input_client);
   ASSERT_EQ(&text_input_client, input_method->GetTextInputClient());
 
   text_input_client.set_text_input_type(ui::TEXT_INPUT_TYPE_TEXT);
 
   EXPECT_FALSE(controller()->IsRemoved(notification_id));
   PerformKeyEvents(ui::VKEY_BACK);
   EXPECT_FALSE(controller()->IsRemoved(notification_id));
 
   input_method->SetFocusedTextInputClient(nullptr);
 }
 
-TEST_F(CustomNotificationViewTest, ChangeContentHeight) {
+TEST_F(ArcNotificationViewTest, ChangeContentHeight) {
   // Default size.
   gfx::Size size = notification_view()->GetPreferredSize();
   size.Enlarge(0, -notification_view()->GetInsets().height());
   EXPECT_EQ("360x100", size.ToString());
 
   // Allow small notifications.
-  custom_view()->set_preferred_size(gfx::Size(10, 10));
+  contents_view()->set_preferred_size(gfx::Size(10, 10));
   size = notification_view()->GetPreferredSize();
   size.Enlarge(0, -notification_view()->GetInsets().height());
   EXPECT_EQ("360x10", size.ToString());
 
   // The long notification.
-  custom_view()->set_preferred_size(gfx::Size(1000, 1000));
+  contents_view()->set_preferred_size(gfx::Size(1000, 1000));
   size = notification_view()->GetPreferredSize();
   size.Enlarge(0, -notification_view()->GetInsets().height());
   EXPECT_EQ("360x1000", size.ToString());
 }
 
-}  // namespace message_center
+}  // namespace arc