[Webkit-unassigned] [Bug 42001] Update media element's handling of empty 'src' attribute

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jul 9 17:51:02 PDT 2010


https://bugs.webkit.org/show_bug.cgi?id=42001





--- Comment #3 from Darin Adler <darin at apple.com>  2010-07-09 17:51:02 PST ---
(From update of attachment 61122)
Don’t we want this for all sorts of URL attributes? I seem to recall discussion of that on one of the HTML5 mailing lists. I don’t want to go through all the classes removing the ReflectURL calls one by one if we can just change the rules about reflected URLs. It's not even out of the question to have more than one flavor of ReflectURL. I hate to hand-write the code just because of this issue. I can help you make a version of ReflectURL that treats an empty attribute the same way it treats a missing one.

Long term we might even want to change the behavior of completeURL itself. I don’t think there are very may contexts where we want to treat the empty string as meaning "expand this to be the base URL".

I also noticed that HTMLMediaElement::src and HTMLSourceElement::src don't strip whitespace before completing the URL the way other similar attribute getters do. That may be a bug.

Functions like HTMLVideoElement::setPoster should take const AtomicString& not const String&.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list