We had location and phase in NodeRenderingContext because we had to traverse a shadow tree twice to attach it. Currently we don't traverse it twice, we should be able to make NodeRenderingContext much simpler.
Created attachment 127354 [details] Test
Created attachment 127477 [details] Patch
Comment on attachment 127477 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=127477&action=review basically looks good. Let's take small time to make the intention clear. > Source/WebCore/dom/NodeRenderingContext.h:79 > }; It's time to give good name for these phase. Could you pick some relevant word from the Shadow DOM node composition algorithm?
Created attachment 127485 [details] Patch
Comment on attachment 127485 [details] Patch Clearing flags on attachment: 127485 Committed r108021: <http://trac.webkit.org/changeset/108021>
All reviewed patches have been landed. Closing bug.