Downloads from webui don't work |
||||||
Issue descriptionChrome Version: 69.0.3455.0 OS: Win10 What steps will reproduce the problem? 1. Install and launch Chrome Canary 2. Navigate to chrome://version page 3. Right click on the 'Chrome' image at the top right hand side What is the expected result? Should be able to download the 'Chrome' image What happens instead? Not able to download the image, failed with Network error. Did this work with Enable network service flag disable? Yes Note: Issue observed on this page only (chrome://version). Able to download all the other images from other sites (tried from cnn.com, yahoo.com, google.com etc)
,
Jun 19 2018
,
Jun 20 2018
,
Jul 10
This is trivial to fix, so I'll take a look.
,
Jul 10
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c8b32bc33748ce3ed716da9bfc2b78435d62d515 commit c8b32bc33748ce3ed716da9bfc2b78435d62d515 Author: John Abd-El-Malek <jam@chromium.org> Date: Tue Jul 10 21:20:36 2018 Fix downloads from webui with network service. Bug: 851901 Change-Id: I739438315126443e5557b096bc47c28039c95688 Reviewed-on: https://chromium-review.googlesource.com/1131830 Reviewed-by: Min Qin <qinmin@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#573912} [modify] https://crrev.com/c8b32bc33748ce3ed716da9bfc2b78435d62d515/content/browser/BUILD.gn [modify] https://crrev.com/c8b32bc33748ce3ed716da9bfc2b78435d62d515/content/browser/download/download_browsertest.cc [modify] https://crrev.com/c8b32bc33748ce3ed716da9bfc2b78435d62d515/content/browser/download/download_manager_impl.cc [add] https://crrev.com/c8b32bc33748ce3ed716da9bfc2b78435d62d515/content/browser/download/web_ui_download_url_loader_factory_getter.cc [add] https://crrev.com/c8b32bc33748ce3ed716da9bfc2b78435d62d515/content/browser/download/web_ui_download_url_loader_factory_getter.h
,
Jul 10
,
Jul 16
Update: Rechecked the above issue on latest canary version #69.0.3493.0 on Win 10 and the issue is Fixed. Hence, adding TE-verified labels. Please refer the attached screen-cast. Thank You..!! |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by dxie@chromium.org
, Jun 12 2018Status: Assigned (was: Untriaged)