Move message_center::CustomNotificationView to arc::ArcNotificationView

ui/message_center/notification_delegate.h

 // Copyright 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.
 
 #ifndef UI_MESSAGE_CENTER_NOTIFICATION_DELEGATE_H_
 #define UI_MESSAGE_CENTER_NOTIFICATION_DELEGATE_H_
 
 #include <memory>
 #include <string>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/strings/string16.h"
 #include "ui/message_center/message_center_export.h"
 
-#if defined(TOOLKIT_VIEWS) && !defined(OS_MACOSX)
-#include "ui/message_center/views/custom_notification_content_view_delegate.h"
-#endif
+namespace message_center {
 
-namespace message_center {
+class MessageCenterController;
+class MessageView;
+class Notification;
 
 // Delegate for a notification. This class has two roles: to implement callback
 // methods for notification, and to provide an identity of the associated
 // notification.
 class MESSAGE_CENTER_EXPORT NotificationDelegate
     : public base::RefCountedThreadSafe<NotificationDelegate> {
  public:
   // To be called when the desktop notification is actually shown.
   virtual void Display();
 
@@ skipping 16 matching lines @@
   virtual void ButtonClickWithReply(int button_index,
                                     const base::string16& reply);
 
   // To be called when the user clicks the settings button in a notification.
   // Returns whether the settings click was handled by the delegate.
   virtual bool SettingsClick();
 
   // To be called in order to detect if a settings button should be displayed.
   virtual bool ShouldDisplaySettingsButton();
 
 #if defined(TOOLKIT_VIEWS) && !defined(OS_MACOSX)

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

(Optional) Is this guard still necessary? Ditto for caller. If so, probably it
is good to be documented here.

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

I think this condition is obvious since this returns a view and mac chrome
doesn't support views notification.

-  // To be called to construct the contents view of a popup for notifications
-  // whose type is NOTIFICATION_TYPE_CUSTOM.
-  virtual std::unique_ptr<CustomContent> CreateCustomContent();
+  // To be called to construct the message view for notifications whose type is
+  // NOTIFICATION_TYPE_CUSTOM.
+  virtual std::unique_ptr<MessageView> CreateCustomMessageView(
+      MessageCenterController* controller,
+      const Notification& notification);
 #endif
 
   // Indicates whether this notification should be displayed when there is
   // fullscreen content being displayed.
   virtual bool ShouldDisplayOverFullscreen() const;
 
  protected:
   virtual ~NotificationDelegate() {}
 
  private:
@@ skipping 38 matching lines @@
 
  private:
   ButtonClickCallback button_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(HandleNotificationButtonClickDelegate);
 };
 
 }  //  namespace message_center
 
 #endif  // UI_MESSAGE_CENTER_NOTIFICATION_DELEGATE_H_