Refine the scoring of MediaImage |
|||
Issue descriptionShould we give a score penalty for non-square images? What's the proper image size upper bound? Should it be density-dependent or independent?
,
Oct 31 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/76252384438261f0c575e22296d5776a7e7f4f47 commit 76252384438261f0c575e22296d5776a7e7f4f47 Author: zqzhang <zqzhang@chromium.org> Date: Mon Oct 31 12:37:11 2016 Improving non-square large icon in MediaNotification This CL improves non-square large icon in MediaNotification by: * Giving penalty score to non-square image. * Rescalling non-square image while keeping the aspect ratio (paddings added). BUG= 656028 Review-Url: https://codereview.chromium.org/2462703002 Cr-Commit-Position: refs/heads/master@{#428691} [modify] https://crrev.com/76252384438261f0c575e22296d5776a7e7f4f47/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaImageManager.java [modify] https://crrev.com/76252384438261f0c575e22296d5776a7e7f4f47/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaNotificationManager.java [modify] https://crrev.com/76252384438261f0c575e22296d5776a7e7f4f47/chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaSessionTabHelper.java
,
Jan 19 2017
,
Mar 7 2017
I think this bug can be marked as fixed unless someone complain on the current situation. |
|||
►
Sign in to add a comment |
|||
Comment 1 by zqzh...@chromium.org
, Oct 27 2016