Issue metadata
Sign in to add a comment
|
content/ should destroy ImageDownloaderImpl() before shutting down Blink |
||||||||||||||||||||||
Issue descriptionBranched from crbug.com/588566 ImageDownloaderImpl() is using an AssociatedURLLoader(). So, before Blink is shutdown, ImageDownLoaderImpl() should cancel active loaders.
,
Apr 9 2016
,
Apr 11 2016
Short-term fix has been landed: https://codereview.chromium.org/1862073002/
,
Apr 11 2016
Long-term fix for ImageDownloaderImpl is ready at https://codereview.chromium.org/1873003002/. I'd like to continue merging the short-term fix at bug 588566 so that even if there's any other class doing similar they're also short-term-fixed.
,
Apr 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d0173cb77f8a5771bc9f27b570149436707626f7 commit d0173cb77f8a5771bc9f27b570149436707626f7 Author: leon.han <leon.han@intel.com> Date: Mon Apr 11 05:41:32 2016 Ensure all AssociatedURLLoader be cancelled before shutting down Blink. ImageDownloaderImpl is using AssociatedURLLoader, all active AssociatedURLLoader must be cancelled before shutting down Blink. BUG= 601737 Review URL: https://codereview.chromium.org/1873003002 Cr-Commit-Position: refs/heads/master@{#386338} [modify] https://crrev.com/d0173cb77f8a5771bc9f27b570149436707626f7/content/renderer/image_downloader/image_downloader_impl.cc [modify] https://crrev.com/d0173cb77f8a5771bc9f27b570149436707626f7/content/renderer/image_downloader/image_downloader_impl.h
,
Apr 11 2016
We might want to also merge r386338 for less crash.
,
Apr 11 2016
,
Apr 11 2016
,
Jul 18 2016
This bug has been closed for more than 14 weeks. Removing security view restrictions. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 20 2016
,
Oct 1 2016
This bug has been closed for more than 14 weeks. Removing security view restrictions. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 2 2016
This bug has been closed for more than 14 weeks. Removing security view restrictions. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Oct 2 2016
,
Jul 28
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by kenrb@chromium.org
, Apr 8 2016