Latest spec reads:
"Otherwise the media element has neither a src attribute nor a source element child: set the networkState to NETWORK_EMPTY, and abort these steps; the synchronous section ends."
I also updated the comments to match the steps outlined in the spec.
Created attachment 41246 [details]
Comment on attachment 41246 [details]
Digging around some more I found more bugs... I'll keep working on this patch.
Created attachment 41255 [details]
Few things (as mentioned in the patch):
- Having no source is defined as a null src attribute and no <source> children, and should result in the element going to NETWORK_EMPTY
- Empty string "" is considered a valid src attribute and should be loaded (before it slipped through the cracks and the element would get jammed in NETWORK_LOADING)
Created attachment 41256 [details]
Removed useless findMediaElement() line from video-src-empty.html
CCing the media master.
Comment on attachment 41256 [details]
> + <p>Empty src attribute defined. Valid URL and video should error.</p>
I think you meant to say "Invalid URL"
r=me with this change.
Thanks for catching this!
Created attachment 41330 [details]
Valid -> Invalid
Comment on attachment 41330 [details]
Clearing flags on attachment: 41330
Committed r49702: <http://trac.webkit.org/changeset/49702>
All reviewed patches have been landed. Closing bug.