Closed Bug 475621 Opened 11 years ago Closed 11 years ago
when saving an iframe whose source is an image, image is not saved
Hi, when saving an iframe whose src attribute is just an image (not a html document), saved iframe source is set to a local relative uri, but that uri is not fetched. steps to reproduce: - save testcase as a complete page. - load saved testcase what you see: - an iframe whose source links to a non existent image what you would like to get: - an iframe whose source links to an image.
Comment on attachment 359145 [details] testcase ><HTML><HEAD> ><TITLE>testcase for bug 475621</TITLE> ></HEAD><BODY> ><P>When saving current page, mozilla logo is not saved.</P> ><IFRAME src="http://upload.wikimedia.org/wikipedia/en/9/94/Mozilla_logo.png"/> ></BODY></HTML>
Oups, forget my previous message. I think the root of the bug is: When an iframe is encountered in the tree walker, MakeAndStoreLocalFilenameInURIMap is called with aNeedsPersisting set to false. But if that uri does not reference a document (ie: it's an image for example), MakeAndStoreLocalFilenameInURIMap is called again with aNeedsPersisting set to true. Currently, previous data is returned as is. So, the mNeedsPersisting is not updated as needed. Then, uri will not be fetched.
sets mNeedsPersisting an retrieved URIData when calling with aNeedsPersisting to true. That would be useful when MakeAndStoreLocalFilenameInURIMap was already called with aURI but aNeedsPersisting set to false.
Comment on attachment 359162 [details] [diff] [review] patch v1 sorry for that attachment. I must have done something wrong and didn't check.
Comment on attachment 360364 [details] [diff] [review] patch v1.0 Looks ok to me.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.