• Ella Ge's avatar
    Revert "[Media Notification] Derive background color from artwork" · 1d3a4939
    Ella Ge authored
    This reverts commit e60e1e6e.
    
    Reason for revert: 
    MediaNotificationBackgroundTest.BoundsSanityCheck
    MediaNotificationViewTest.UpdateArtworkFromItem
    failure on  Linux ChromiumOS MSan Tests bot:
    https://ci.chromium.org/p/chromium/builders/ci/Linux%20ChromiumOS%20MSan%20Tests/12036
    
    Original change's description:
    > [Media Notification] Derive background color from artwork
    > 
    > The media notification should derive its background
    > color from prominent colors in the artwork. This
    > uses as similar algorithm from Android.
    > 
    > Also breaks out some of the logic of CalculateProminentColors
    > so we can use it without the reset of the function. This
    > is because we want uninteresting colors and do not want
    > the color profile matching.
    > 
    > BUG=944598
    > 
    > Change-Id: I60df7f600f6fc881dfb38d2057c903a5679845df
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1534923
    > Commit-Queue: Becca Hughes <beccahughes@chromium.org>
    > Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
    > Reviewed-by: Evan Stade <estade@chromium.org>
    > Reviewed-by: Tommy Steimel <steimel@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#644985}
    
    TBR=sadrul@chromium.org,estade@chromium.org,beccahughes@chromium.org,steimel@chromium.org
    
    Change-Id: Ie18b564a68ffdf2560492eb1e7cf53c59447ea54
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 944598
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1543464Reviewed-by: default avatarElla Ge <eirage@chromium.org>
    Commit-Queue: Ella Ge <eirage@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#645338}
    1d3a4939
media_notification_background_unittest.cc 2.32 KB