Transfer responsibility for providing a close button for a notification to each implementation of M…

ui/message_center/views/custom_notification_content_view_delegate.h

+// 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.
+
+#ifndef UI_MESSAGE_CENTER_CUSTOM_NOTIFICATION_CONTENT_VIEW_DELEGATE_H_
+#define UI_MESSAGE_CENTER_CUSTOM_NOTIFICATION_CONTENT_VIEW_DELEGATE_H_
+
+#include <memory>
+
+#include "base/macros.h"
+#include "ui/message_center/message_center_export.h"
+
+namespace views {
+class View;
+}

[yoshiki, 2016/12/06 05:28:27]

Please add "// namespace views"

[yhanada, 2016/12/06 06:41:26]

Done.

+
+namespace message_center {
+
+// Delegate for a view that is hosted by CustomNotificationView.
+class MESSAGE_CENTER_EXPORT CustomNotificationContentViewDelegate {
+ public:
+  virtual bool IsCloseButtonFocused() const = 0;
+  virtual void RequestFocusOnCloseButton() = 0;
+  virtual bool IsPinned() const = 0;
+};
+
+// The struct to hold a view and CustomNotificationContentViewDelegate
+// associating with the view.
+struct MESSAGE_CENTER_EXPORT CustomContent {
+ public:
+  CustomContent(
+      std::unique_ptr<views::View> view,
+      std::unique_ptr<CustomNotificationContentViewDelegate> delegate);
+  ~CustomContent();
+
+  std::unique_ptr<views::View> view_;

[yoshiki, 2016/12/06 05:28:27]

Remove the underscode at the end since this is public.

[yhanada, 2016/12/06 06:41:26]

Done.

+  std::unique_ptr<CustomNotificationContentViewDelegate> delegate_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(CustomContent);
+};
+
+}  // namespace message_center
+
+#endif  // UI_MESSAGE_CENTER_CUSTOM_NOTIFICATION_CONTENT_VIEW_DELEGATE_H_