• Tommy Steimel's avatar
    GMC: Make MediaNotificationItem independent of Ash · 875de1d4
    Tommy Steimel authored
    This CL updates the relationship between MediaNotificationItem and
    MediaNotificationController (now MediaNotificationControllerImpl) such
    that MediaNotificationItem is only responsible for deciding when to
    show or hide a media notification, and isn't responsible for how that
    media notification is displayed. This is done by creating a
    MediaNotificationController interface that
    MediaNotificationControllerImpl implements.
    
    This CL also updates the relationship of MediaNotificationItem and
    MediaNotificationView so that MediaNotificationView calls methods on
    MediaNotificationItem directly instead of calling them through the
    MediaNotificationController or the message_center::MessageCenter.
    
    This is part of a refactoring of the media notification to make it
    reusable outside of ash/media.
    
    Bug: 959478
    Change-Id: I77d3f0329f0e8ee32d4cbb79829a0fdf50ac8113
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1615497Reviewed-by: default avatarScott Violet <sky@chromium.org>
    Reviewed-by: default avatarBecca Hughes <beccahughes@chromium.org>
    Commit-Queue: Tommy Steimel <steimel@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#660873}
    875de1d4
media_notification_view.cc 19.2 KB