Downloading image from three-dots menu does not work as expedcted |
||||
Issue descriptionApplication Version (from "Chrome Settings > About Chrome"): 56.0.22924.18 Android Build Number (from "Android Settings > About Phone/Tablet"): N2F62 Device: Pixel Steps to reproduce: 1) Visit a "bare" JPEG image directly using a URL, e.g., https://blogintomystery.files.wordpress.com/2010/07/atlas.jpg 2) Tap three dots menu 3) Tap Download button on first row of the menu 4) See "blogintomystery.files.wordpress.com downloaded" 5) Tap "Open" in the snackbar Observed behavior: A few problems here: 1) When opening the downloaded image, the snack bar shows up "Viewing an offline copy of this page". This suggests that Chrome is treating the bare JPEG that I downloaded as an offline page, rather than just an image. 2) The downloaded image does not appear in the Android "Downloads" app. Meaning, the image cannot be read by other apps (e.g., Facebook, WhatsApp, ShareIt) for sharing *from within those apps*. 3) Now that long-pressing on an image and selecting "Download image" will save it to, e.g., file:///storage/emulated/0/Download/atlas.jpg which means that the image *is* visible to other apps for sharing. Expected behavior: I believe tapping the download button in the three-dots menu and long-pressing an image and selecting "Download image" should do the same thing.
,
Feb 17 2017
,
Mar 24 2017
We need to check the webcontent's mime type and instead of downloading a page request a regular file download (via DownloadManager API).
,
Apr 26 2017
,
Apr 28 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/030745885a307ced7964ab2288cbd36ecd9ec969 commit 030745885a307ced7964ab2288cbd36ecd9ec969 Author: jianli <jianli@chromium.org> Date: Fri Apr 28 08:35:18 2017 Use DownloadManager to download non-html res when download icon pressed We check the content's MIME type. If it is not meant for HTML page,we route to DownloadManager. BUG= 673635 Review-Url: https://codereview.chromium.org/2841823004 Cr-Commit-Position: refs/heads/master@{#467931} [modify] https://crrev.com/030745885a307ced7964ab2288cbd36ecd9ec969/chrome/browser/android/offline_pages/downloads/offline_page_download_bridge.cc [modify] https://crrev.com/030745885a307ced7964ab2288cbd36ecd9ec969/chrome/browser/android/offline_pages/downloads/resource_throttle.cc [modify] https://crrev.com/030745885a307ced7964ab2288cbd36ecd9ec969/chrome/browser/android/offline_pages/offline_page_utils.cc [modify] https://crrev.com/030745885a307ced7964ab2288cbd36ecd9ec969/chrome/browser/android/offline_pages/offline_page_utils.h
,
Apr 28 2017
|
||||
►
Sign in to add a comment |
||||
Comment 1 by dim...@chromium.org
, Dec 14 2016Status: Assigned (was: Untriaged)