pageCapture.saveAsMHTML doesn't account for srcset attribute
Reported by
dbodi...@gmail.com,
Jul 12 2017
|
||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36 Steps to reproduce the problem: 1. Load webpage: https://www.fastcodesign.com/90132364/nest-founder-i-wake-up-in-cold-sweats-thinking-what-did-we-bring-to-the-world 2. Use (extension) pageCapture.saveAsMHTML api get mhtml 3. Load the mhtml file in chrome What is the expected behavior? The srcset attribute should be persisted during mhtml save process and the image used from the srcset attribute should be included in the mhtml What went wrong? The loaded source code shows the srset attributes are stripped from the original elements during mhtml creation, and the associated image used from srcset is not added to the mhtml file Did this work before? N/A Does this work in other browsers? N/A Chrome version: 59.0.3071.115 Channel: stable OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: I'm an entrepreneur/developer and rely on this for a product I'm building. Please keep me posted. dbodin11@gmail.com
,
Jul 13 2017
What about shadow DOM: for example, the tweets on this page: https://www.buzzfeed.com/buzzfeednews/live-updates-washington-dc-week-of-071017?utm_term=.djV3enWnXW#.kp0pVwDwRD
,
Jul 19 2017
dbodin11@ - Thanks for filing the issue...!! Could you please provide a sample extension url or an html file to test the issue from TE-end. This will help us in triaging the issue further. Thanks...!!
,
Jul 19 2017
I ran into the issue on this page (https://www.fastcodesign.com/90132364/nest-founder-i-wake-up-in-cold-sweats-thinking-what-did-we-bring-to-the-world) working with a local extension I built that literally calls that one function to save the page as mhtml. Side note, lazily loaded images aren't accounted for either. Could probably load page and do simple full page scroll to trigger network calls and have the currentSrc of imgs be updated in dom object before mhtml conversion.
,
Jul 19 2017
Thank you for providing more feedback. Adding requester "krajshree@chromium.org" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 21 2017
@dbodin11 -- Thanks for the reply. Could you also please provide the sample extension to reproduce the issue. That would help us in test and triage the issue further. Thanks in advance.
,
Jul 26 2017
,
Nov 7 2017
Marking the issue as wontfix, as there is no update from the reporter for more than 1 month. Please feel free to raise a new issue if able to reproduce the issue using latest chrome builds. Please use the following link to download latest chrome builds https://www.chromium.org/getting-involved/dev-channel Thanks...!! |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by dbodi...@gmail.com
, Jul 12 2017