[Chrome][M55][Android7.0] Photos provides only "Info" menu when opening downloaded image with Photos
Reported by
seiyon.p...@gmail.com,
Jan 23 2017
|
||||
Issue descriptionSteps to reproduce the problem: 1. Launch Chrome 2. Go to any site with images 3. Download image and open via Chrome 4. Touch overflow menu on Chrome > Touch "Open with" > Select Photos What is the expected behavior? "Share", "Edit" and "Overflow menu" should be provided. What went wrong? Only "Info" is provided since Chrome sends file URI to Photos. If Chrome sends content URI to Photos, Photos will provide "Share", "Edit" and "Overflow menu". Did this work before? N/A Chrome version: 55.0.2883.87 Channel: stable OS Version: 7.0 Flash Version: Shockwave Flash 24.0 r0
,
Jan 26 2017
Shakti can you take a look? dfalcantara@ is this related to content URI vs. file path?
,
Jan 26 2017
Probably. Min would know better about the difference and when we could use one versus the other.
,
Jan 27 2017
I vaguely remember it is either because we need to playback the media in CCT or it is because of sharing the downloaded files.
,
Jan 27 2017
Theresa? I think it might have involved sharing.
,
Jan 27 2017
This is intentional - we pass both a file URI and share URI when we create the CCT -- see DownloadUtils#getMediaViewerIntentForDownloadItem(). Tapping the share icon sends an intent with the content URI and selecting "Open with..." uses the file URI. This may not always be the right choice, so I think it's okay to investigate changing either for photos specifically or for all content types. https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java?q=downloads+share+file:.java&sq=package:chromium&dr=CSs&l=375
,
Feb 7 2017
For user experience, it would be better to use content URI in "Open with" menu since content URI can make Photos app provide "Share", "Edit" and "Overflow menu.
,
Feb 7 2017
We recently switched to using a content URI because Android N forced us to. It should show up in M57+ soonish: https://chromiumcodereview.appspot.com/2671573002/ Please let us know if that fixes the issue for you, seiyon.park@
,
Feb 8 2017
Thanks for prompt feedback and I'll test in once M57 is rolled-out. |
||||
►
Sign in to add a comment |
||||
Comment 1 by rsgav...@chromium.org
, Jan 23 2017Components: UI>Browser>Downloads