Consolidate to use MessageView

Consolidate to use MessageView

- Make factory NotificationView::Create to return MessageView* instead
  of NotificationView*;
- Use MessageView in MessageCenterView and MessagePopupCollection;
- Remove MessageViewController and replace it with MessageCenterController;

BUG=610767
Committed: https://crrev.com/f1c72f97ea348dcf3ffd92966975d4105795fd3d
Cr-Commit-Position: refs/heads/master@{#393663}

[xiyuan, 2016-05-13 19:45:09]

xiyuan@chromium.org changed reviewers:
+ dewittj@chromium.org

[xiyuan, 2016-05-13 19:45:09]

First part of the custom notification support CL. Split so that it is more
readable. There are two more to come:

https://codereview.chromium.org/1980753002/
https://codereview.chromium.org/1979583003/

Thanks.

[dewittj, 2016-05-13 20:24:32]

lgtm with a nit

https://codereview.chromium.org/1979553003/diff/1/ui/message_center/views/not...
File ui/message_center/views/notification_view.h (right):

https://codereview.chromium.org/1979553003/diff/1/ui/message_center/views/not...
ui/message_center/views/notification_view.h:47: static MessageView*
Create(MessageCenterController* controller,
Is there a reason to return a MessageView here?

[xiyuan, 2016-05-13 21:07:39]

https://codereview.chromium.org/1979553003/diff/1/ui/message_center/views/not...
File ui/message_center/views/notification_view.h (right):

https://codereview.chromium.org/1979553003/diff/1/ui/message_center/views/not...
ui/message_center/views/notification_view.h:47: static MessageView*
Create(MessageCenterController* controller,
On 2016/05/13 20:24:32, dewittj wrote:
> Is there a reason to return a MessageView here?

Yes. The incoming CustomNotificationView will be created here and it is derived
from MessageView. MessageView is what this function returned before we fold
everything into NotificationView. This cl brings that back.

[xiyuan, 2016-05-13 22:19:30]

The CQ bit was checked by xiyuan@chromium.org

[commit-bot: I haz the power, 2016-05-13 22:20:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1979553003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1979553003/1

[commit-bot: I haz the power, 2016-05-13 22:24:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-13 22:26:32]

Description was changed from

==========
Consolidate to use MessageView

- Make factory NotificationView::Create to return MessageView* instead
  of NotificationView*;
- Use MessageView in MessageCenterView and MessagePopupCollection;
- Remove MessageViewController and replace it with MessageCenterController;

BUG=610767
==========

to

==========
Consolidate to use MessageView

- Make factory NotificationView::Create to return MessageView* instead
  of NotificationView*;
- Use MessageView in MessageCenterView and MessagePopupCollection;
- Remove MessageViewController and replace it with MessageCenterController;

BUG=610767

Committed: https://crrev.com/f1c72f97ea348dcf3ffd92966975d4105795fd3d
Cr-Commit-Position: refs/heads/master@{#393663}
==========

[commit-bot: I haz the power, 2016-05-13 22:26:33]

Patchset 1 (id:??) landed as
https://crrev.com/f1c72f97ea348dcf3ffd92966975d4105795fd3d
Cr-Commit-Position: refs/heads/master@{#393663}