Bug 61912 – [Refactoring] Node::nextRenderer() and previousRenderer() should be part of NodeRenderingContext

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Hajime Morrita]

This is a preparation for upcoming fix.

[Hajime Morrita]

Created attachment 95735 [details]
Patch

[Kent Tamura]

Comment on attachment 95735 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=95735&action=review

> Source/WebCore/dom/NodeRenderingContext.cpp:118
> +    for (Node* n = m_node->nextSibling(); n; n = n->nextSibling()) {

nit: one-letter variable name

> Source/WebCore/dom/NodeRenderingContext.cpp:143
> +    for (Node* n = m_node->previousSibling(); n; n = n->previousSibling()) {

ditto.

[Hajime Morrita]

Hi Kent-san, thank you for reviewing!

(In reply to comment #2)
> (From update of attachment 95735 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=95735&action=review
> 
> > Source/WebCore/dom/NodeRenderingContext.cpp:118
> > +    for (Node* n = m_node->nextSibling(); n; n = n->nextSibling()) {
> 
> nit: one-letter variable name
Woa, will rename before landing...

[Hajime Morrita]

Committed r87885: <http://trac.webkit.org/changeset/87885>