Implement HTML5 "in scope" algorithm and attempt to use it

[Eric Seidel (no email)]

Implement HTML5 "in scope" algorithm and attempt to use it

[Eric Seidel (no email)]

Created attachment 60107 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 60107 [details]
Patch

Nevermind.  This causes a test failure.  I'll look at this more tomorrow.

[Eric Seidel (no email)]

Created attachment 60109 [details]
Patch

[Adam Barth]

Comment on attachment 60109 [details]
Patch

WebCore/html/HTMLElementStack.cpp:36
 +  using namespace HTMLNames;
Ah, i see you have it now :)

WebCore/html/HTMLElementStack.cpp:186
 +          if (isMarker(element))
We might want to store these bits in the ElementRecord instead of testing each time, depending on how expensive these branches end up being.

[WebKit Commit Bot]

Comment on attachment 60109 [details]
Patch

Clearing flags on attachment: 60109

Committed r62196: <http://trac.webkit.org/changeset/62196>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[WebKit Review Bot]

http://trac.webkit.org/changeset/62196 might have broken Qt Linux Release minimal

[Eric Seidel (no email)]

(In reply to comment #4)
> (From update of attachment 60109 [details])
> WebCore/html/HTMLElementStack.cpp:36
>  +  using namespace HTMLNames;
> Ah, i see you have it now :)
> 
> WebCore/html/HTMLElementStack.cpp:186
>  +          if (isMarker(element))
> We might want to store these bits in the ElementRecord instead of testing each time, depending on how expensive these branches end up being.

I think that's an excellent idea!