Since r192953, HTMLSourceElement is built even if video is disabled, since it is used by the picture element. But this breaks build with -no-video, since HTMLMediaElement usage should be guarded by VIDEO guards, and its JS bindings were still under a video conditional.
Created attachment 267637 [details] Patch
Comment on attachment 267637 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=267637&action=review > Source/WebCore/html/HTMLSourceElement.cpp:67 > + else > +#endif > + if (is<HTMLPictureElement>(*parent)) This is strange, but it's clear what is going on and has the correct behavior, and I can't make anything that does this that doesn't also look strange.
(In reply to comment #2) > Comment on attachment 267637 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=267637&action=review > > > Source/WebCore/html/HTMLSourceElement.cpp:67 > > + else > > +#endif > > + if (is<HTMLPictureElement>(*parent)) > > This is strange, but it's clear what is going on and has the correct > behavior, and I can't make anything that does this that doesn't also look > strange. This is indeed a bit weird, but I didn't find a better way, and saw that this pattern is already used in a few places, like HTMLOptionElement.cpp Thanks for the review!
Comment on attachment 267637 [details] Patch Clearing flags on attachment: 267637 Committed r194278: <http://trac.webkit.org/changeset/194278>
All reviewed patches have been landed. Closing bug.