Replace X with back button in media playback CCT |
||||
Issue descriptionPer UI review, we should replace the "X" with a back arrow for users to go back to the downloads home form the media playback CCT. This is a blocking bug for M55
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4f5ce82e33afe872b6959dc381925666b14a413f commit 4f5ce82e33afe872b6959dc381925666b14a413f Author: dfalcantara <dfalcantara@chromium.org> Date: Thu Sep 29 21:14:47 2016 [Download Home] Various M55 adjustments * Replace "Open externally" with "Open with..." and force a chooser to appear when there's more than one possibility for an app, even if the user has a default set. * Replace file size display with logical size values that are smaller than GB * Replace X with <- BUG= 650451 , 650447 , 643795 Review-Url: https://codereview.chromium.org/2359763003 Cr-Commit-Position: refs/heads/master@{#421933} [modify] https://crrev.com/4f5ce82e33afe872b6959dc381925666b14a413f/chrome/android/java/src/org/chromium/chrome/browser/customtabs/CustomTabIntentDataProvider.java [modify] https://crrev.com/4f5ce82e33afe872b6959dc381925666b14a413f/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java [modify] https://crrev.com/4f5ce82e33afe872b6959dc381925666b14a413f/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java [modify] https://crrev.com/4f5ce82e33afe872b6959dc381925666b14a413f/chrome/android/java/src/org/chromium/chrome/browser/download/ui/SpaceDisplay.java [modify] https://crrev.com/4f5ce82e33afe872b6959dc381925666b14a413f/chrome/android/java/strings/android_chrome_strings.grd [modify] https://crrev.com/4f5ce82e33afe872b6959dc381925666b14a413f/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadActivityTest.java
,
Sep 29 2016
,
Oct 4 2016
Verified in M55 - 55.0.2880.3 build
,
Oct 28 2016
[Auto-generated comment by a script] We noticed that this issue is targeted for M-55; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-55 label, otherwise remove Merge-TBD label. Thanks.
,
Oct 28 2016
Just when I thought the erroneous Merge-Merged-2840 merge bot was annoying enough, a wild blockerbot with erroneous merge requests appears. |
||||
►
Sign in to add a comment |
||||
Comment 1 by dfalcant...@chromium.org
, Sep 27 2016