881000 - Consider reflecting img.lowsrc as a URL

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Spec has it that way: https://www.w3.org/Bugs/Public/show_bug.cgi?id=10341

Comment 1

[Boris Zbarsky [:bzbarsky]]

What do UAs do?

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

It seems from the patch in the webkit bug that they'd reflect as a URL; I haven't tested anything, though.

(Not sure why I assigned this to myself, either.)

Comment 3

[Mounir Lamouri (:mounir)]

Attachment: Testcase

Comment 4

[Mounir Lamouri (:mounir)]

The specs describe the behaviour of:
IE8+, Safari 4+ (didn't try before) and Chrome

Firefox doesn't have an implementation yet in release and Presto (Opera) doesn't reflect the attribute as an URL.

Basically, given that Opera switched to Blink, we would be the only browser not to reflect the attribute as an URL.

Comment 5

[Mounir Lamouri (:mounir)]

(In reply to :Ms2ger from comment #2)
> (Not sure why I assigned this to myself, either.)

Because you wanted to fix it maybe? :)

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Test for the test suite: https://github.com/w3c/web-platform-tests/pull/317

Comment 7

[Aryeh Gregor (:ayg) (no longer with Mozilla)]

Attachment: 0001-Bug-881000-Reflect-img.lowsrc-as-a-URL-not-string.patch

https://treeherder.mozilla.org/#/jobs?repo=try&revision=791a5eb96edf&exclusion_profile=false

Comment 8

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8749206 [details] [diff] [review]
0001-Bug-881000-Reflect-img.lowsrc-as-a-URL-not-string.patch

r=me, though it's a little weird given we don't actually support the attr at all last I checked, so this could mess with support detection... watch out for that.

Comment 9

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/5b4e248577b3

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5b4e248577b3