Implement <header> and <footer>. They have nesting restrictions unlike <nav>, <section>, <article> and <aside>.
Created attachment 45553 [details] Proposed patch
style-queue ran check-webkit-style on attachment 45553 [details] without any errors.
Comment on attachment 45553 [details] Proposed patch Nesting <footer> inside <header> or <footer> (or vive versa) is an authoring conformance error, but I don't believe it is supposed to result in any parser error fixup. I do not see any such requirement in the HTML5 parsing algorithm anyway. r- for now but please resubmit if you can cite a source for that requirement.
(In reply to comment #3) > (From update of attachment 45553 [details]) > Nesting <footer> inside <header> or <footer> (or vive versa) is an authoring > conformance error, but I don't believe it is supposed to result in any parser > error fixup. I do not see any such requirement in the HTML5 parsing algorithm > anyway. r- for now but please resubmit if you can cite a source for that > requirement. http://www.whatwg.org/specs/web-apps/current-work/multipage/sections.html#the-header-element 4.4.8 The header element ... Content model: Flow content, but with no header or footer element descendants. http://www.whatwg.org/specs/web-apps/current-work/multipage/sections.html#the-footer-element 4.4.9 The footer element ... Content model: Flow content, but with no header or footer element descendants.
Comment on attachment 45553 [details] Proposed patch (In reply to comment #4) > (In reply to comment #3) > > (From update of attachment 45553 [details] [details]) > > Nesting <footer> inside <header> or <footer> (or vive versa) is an authoring > > conformance error, but I don't believe it is supposed to result in any parser > > error fixup. I do not see any such requirement in the HTML5 parsing algorithm > > anyway. r- for now but please resubmit if you can cite a source for that > > requirement. > > http://www.whatwg.org/specs/web-apps/current-work/multipage/sections.html#the-header-element > 4.4.8 The header element > ... > Content model: > Flow content, but with no header or footer element descendants. > > http://www.whatwg.org/specs/web-apps/current-work/multipage/sections.html#the-footer-element > 4.4.9 The footer element > ... > Content model: > Flow content, but with no header or footer element descendants. Those are authoring requirements (i.e. a document violating these is nonconforming), not implementation requirements (there is no requirement to enforce these conditions at parse time or otherwise. The only error corrections that should be done are those required by section 9.2: http://dev.w3.org/html5/spec/Overview.html#parsing Please keep this in mind when implementing additional new HTML elements. The content models in general do *not* result in parser behavior changes. Please resubmit with the parser changes removed and with correct tests.
Oh, I have just understood "Content model" is an author requirement. Thank you! I'll update the patch.
Created attachment 45610 [details] Proposed patch This updated patch is almost equivalent the patches for <section>, <article> and <aside>. The tests check we *can* nest <header> and <footer> elements.
style-queue ran check-webkit-style on attachment 45610 [details] without any errors.
Comment on attachment 45610 [details] Proposed patch r=me
Comment on attachment 45610 [details] Proposed patch Clearing flags on attachment: 45610 Committed r52846: <http://trac.webkit.org/changeset/52846>
All reviewed patches have been landed. Closing bug.