[webkit-reviews] review denied: [Bug 99677] When using SVG as an image, we should allow datauri images : [Attachment 233895] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jun 26 02:24:53 PDT 2014
Dirk Schulze <krit at webkit.org> has denied Allan Sandfeld Jensen
<allan.jensen at digia.com>'s request for review:
Bug 99677: When using SVG as an image, we should allow datauri images
https://bugs.webkit.org/show_bug.cgi?id=99677
Attachment 233895: Patch
https://bugs.webkit.org/attachment.cgi?id=233895&action=review
------- Additional Comments from Dirk Schulze <krit at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=233895&action=review
r- to make sure that this does not just get landed as is.
> Source/WebCore/ChangeLog:7
> + in a SVG images was not loaded in most cases.
On purpose!
> Source/WebCore/ChangeLog:11
> + This patch enables auto-loading of images. This only affects data
> + URLs since the dummy chrome and page of SVG images still do not
> + have a NetworkingContext.
Again, on purpose! No network requests at all from content of SVG images...
dataURLs should be interpreted but the same restrictions apply here as well of
course.
> Source/WebCore/svg/graphics/SVGImage.cpp:367
> + m_page->settings()->setLoadsImagesAutomatically(true);
Hm, in theory this allows loading further resources. Is that blocked at a
different place? Could you point out where? Please also add a comment that we
need to be very careful that this does not do any network requests.
> LayoutTests/svg/in-html/resources/embedded.svg:3
> + <image x="20" y="20" width="30px" height="30px"
xlink:href="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAAUAAAAFCAYAAACNbyblA
AAAHElEQVQI12P4//8/w38GIAXDIBKE0DHxgljNBAAO9TXL0Y4OHwAAAABJRU5ErkJggg=="/>
Please add examples with an SVG that loads another image decoded with dataURL.
More information about the webkit-reviews
mailing list