RenderCounter shouldn't need a pre-destructor hook.
[WebKit-https.git] / Source / WebCore / rendering / RenderCounter.cpp
1 /**
2  * Copyright (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
3  * Copyright (C) 2006, 2007 Apple Inc. All rights reserved.
4  *
5  * This library is free software; you can redistribute it and/or
6  * modify it under the terms of the GNU Library General Public
7  * License as published by the Free Software Foundation; either
8  * version 2 of the License, or (at your option) any later version.
9  *
10  * This library is distributed in the hope that it will be useful,
11  * but WITHOUT ANY WARRANTY; without even the implied warranty of
12  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
13  * Library General Public License for more details.
14  *
15  * You should have received a copy of the GNU Library General Public License
16  * along with this library; see the file COPYING.LIB.  If not, write to
17  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
18  * Boston, MA 02110-1301, USA.
19  *
20  */
21
22 #include "config.h"
23 #include "RenderCounter.h"
24
25 #include "CounterNode.h"
26 #include "Document.h"
27 #include "Element.h"
28 #include "ElementTraversal.h"
29 #include "HTMLNames.h"
30 #include "HTMLOListElement.h"
31 #include "PseudoElement.h"
32 #include "RenderListItem.h"
33 #include "RenderListMarker.h"
34 #include "RenderStyle.h"
35 #include "RenderView.h"
36 #include <wtf/StdLibExtras.h>
37
38 #ifndef NDEBUG
39 #include <stdio.h>
40 #endif
41
42 namespace WebCore {
43
44 using namespace HTMLNames;
45
46 typedef HashMap<AtomicString, RefPtr<CounterNode>> CounterMap;
47 typedef HashMap<const RenderElement*, std::unique_ptr<CounterMap>> CounterMaps;
48
49 static CounterNode* makeCounterNode(RenderElement&, const AtomicString& identifier, bool alwaysCreateCounter);
50
51 static CounterMaps& counterMaps()
52 {
53     DEPRECATED_DEFINE_STATIC_LOCAL(CounterMaps, staticCounterMaps, ());
54     return staticCounterMaps;
55 }
56
57 // This function processes the renderer tree in the order of the DOM tree
58 // including pseudo elements as defined in CSS 2.1.
59 static RenderElement* previousInPreOrder(const RenderElement& renderer)
60 {
61     Element* previous = ElementTraversal::previousIncludingPseudo(renderer.element());
62     while (previous && !previous->renderer())
63         previous = ElementTraversal::previousIncludingPseudo(previous);
64     return previous ? previous->renderer() : 0;
65 }
66
67 static inline Element* parentOrPseudoHostElement(const RenderElement& renderer)
68 {
69     if (renderer.isPseudoElement())
70         return renderer.generatingElement();
71     return renderer.element() ? renderer.element()->parentElement() : nullptr;
72 }
73
74 // This function processes the renderer tree in the order of the DOM tree
75 // including pseudo elements as defined in CSS 2.1.
76 static RenderElement* previousSiblingOrParent(const RenderElement& renderer)
77 {
78     Element* previous = ElementTraversal::pseudoAwarePreviousSibling(renderer.element());
79     while (previous && !previous->renderer())
80         previous = ElementTraversal::pseudoAwarePreviousSibling(previous);
81     if (previous)
82         return previous->renderer();
83     previous = parentOrPseudoHostElement(renderer);
84     return previous ? previous->renderer() : nullptr;
85 }
86
87 static inline bool areRenderersElementsSiblings(const RenderElement& first, const RenderElement& second)
88 {
89     return parentOrPseudoHostElement(first) == parentOrPseudoHostElement(second);
90 }
91
92 // This function processes the renderer tree in the order of the DOM tree
93 // including pseudo elements as defined in CSS 2.1.
94 static RenderElement* nextInPreOrder(const RenderElement& renderer, const Element* stayWithin, bool skipDescendants = false)
95 {
96     Element* self = renderer.element();
97     Element* next = skipDescendants ? ElementTraversal::nextIncludingPseudoSkippingChildren(self, stayWithin) : ElementTraversal::nextIncludingPseudo(self, stayWithin);
98     while (next && !next->renderer())
99         next = skipDescendants ? ElementTraversal::nextIncludingPseudoSkippingChildren(next, stayWithin) : ElementTraversal::nextIncludingPseudo(next, stayWithin);
100     return next ? next->renderer() : 0;
101 }
102
103 static bool planCounter(RenderElement& renderer, const AtomicString& identifier, bool& isReset, int& value)
104 {
105     // We must have a generating node or else we cannot have a counter.
106     Element* generatingElement = renderer.generatingElement();
107     if (!generatingElement)
108         return false;
109
110     const RenderStyle& style = renderer.style();
111
112     switch (style.styleType()) {
113     case NOPSEUDO:
114         // Sometimes elements have more then one renderer. Only the first one gets the counter
115         // LayoutTests/http/tests/css/counter-crash.html
116         if (generatingElement->renderer() != &renderer)
117             return false;
118         break;
119     case BEFORE:
120     case AFTER:
121         break;
122     default:
123         return false; // Counters are forbidden from all other pseudo elements.
124     }
125
126     const CounterDirectives directives = style.getCounterDirectives(identifier);
127     if (directives.isDefined()) {
128         value = directives.combinedValue();
129         isReset = directives.isReset();
130         return true;
131     }
132
133     if (identifier == "list-item") {
134         if (is<RenderListItem>(renderer)) {
135             if (downcast<RenderListItem>(renderer).hasExplicitValue()) {
136                 value = downcast<RenderListItem>(renderer).explicitValue();
137                 isReset = true;
138                 return true;
139             }
140             value = 1;
141             isReset = false;
142             return true;
143         }
144         if (Element* element = renderer.element()) {
145             if (is<HTMLOListElement>(*element)) {
146                 value = downcast<HTMLOListElement>(*element).start();
147                 isReset = true;
148                 return true;
149             }
150             if (element->hasTagName(ulTag) || element->hasTagName(menuTag) || element->hasTagName(dirTag)) {
151                 value = 0;
152                 isReset = true;
153                 return true;
154             }
155         }
156     }
157
158     return false;
159 }
160
161 // - Finds the insertion point for the counter described by counterOwner, isReset and 
162 // identifier in the CounterNode tree for identifier and sets parent and
163 // previousSibling accordingly.
164 // - The function returns true if the counter whose insertion point is searched is NOT
165 // the root of the tree.
166 // - The root of the tree is a counter reference that is not in the scope of any other
167 // counter with the same identifier.
168 // - All the counter references with the same identifier as this one that are in
169 // children or subsequent siblings of the renderer that owns the root of the tree
170 // form the rest of of the nodes of the tree.
171 // - The root of the tree is always a reset type reference.
172 // - A subtree rooted at any reset node in the tree is equivalent to all counter 
173 // references that are in the scope of the counter or nested counter defined by that
174 // reset node.
175 // - Non-reset CounterNodes cannot have descendants.
176
177 static bool findPlaceForCounter(RenderElement& counterOwner, const AtomicString& identifier, bool isReset, RefPtr<CounterNode>& parent, RefPtr<CounterNode>& previousSibling)
178 {
179     // We cannot stop searching for counters with the same identifier before we also
180     // check this renderer, because it may affect the positioning in the tree of our counter.
181     RenderElement* searchEndRenderer = previousSiblingOrParent(counterOwner);
182     // We check renderers in preOrder from the renderer that our counter is attached to
183     // towards the begining of the document for counters with the same identifier as the one
184     // we are trying to find a place for. This is the next renderer to be checked.
185     RenderElement* currentRenderer = previousInPreOrder(counterOwner);
186     previousSibling = 0;
187     RefPtr<CounterNode> previousSiblingProtector = 0;
188
189     while (currentRenderer) {
190         CounterNode* currentCounter = makeCounterNode(*currentRenderer, identifier, false);
191         if (searchEndRenderer == currentRenderer) {
192             // We may be at the end of our search.
193             if (currentCounter) {
194                 // We have a suitable counter on the EndSearchRenderer.
195                 if (previousSiblingProtector) { // But we already found another counter that we come after.
196                     if (currentCounter->actsAsReset()) {
197                         // We found a reset counter that is on a renderer that is a sibling of ours or a parent.
198                         if (isReset && areRenderersElementsSiblings(*currentRenderer, counterOwner)) {
199                             // We are also a reset counter and the previous reset was on a sibling renderer
200                             // hence we are the next sibling of that counter if that reset is not a root or
201                             // we are a root node if that reset is a root.
202                             parent = currentCounter->parent();
203                             previousSibling = parent ? currentCounter : 0;
204                             return parent;
205                         }
206                         // We are not a reset node or the previous reset must be on an ancestor of our owner renderer
207                         // hence we must be a child of that reset counter.
208                         parent = currentCounter;
209                         // In some cases renders can be reparented (ex. nodes inside a table but not in a column or row).
210                         // In these cases the identified previousSibling will be invalid as its parent is different from
211                         // our identified parent.
212                         if (previousSiblingProtector->parent() != currentCounter)
213                             previousSiblingProtector = 0;
214
215                         previousSibling = previousSiblingProtector.get();
216                         return true;
217                     }
218                     // CurrentCounter, the counter at the EndSearchRenderer, is not reset.
219                     if (!isReset || !areRenderersElementsSiblings(*currentRenderer, counterOwner)) {
220                         // If the node we are placing is not reset or we have found a counter that is attached
221                         // to an ancestor of the placed counter's owner renderer we know we are a sibling of that node.
222                         if (currentCounter->parent() != previousSiblingProtector->parent())
223                             return false;
224
225                         parent = currentCounter->parent();
226                         previousSibling = previousSiblingProtector.get();
227                         return true;
228                     }
229                 } else { 
230                     // We are at the potential end of the search, but we had no previous sibling candidate
231                     // In this case we follow pretty much the same logic as above but no ASSERTs about 
232                     // previousSibling, and when we are a sibling of the end counter we must set previousSibling
233                     // to currentCounter.
234                     if (currentCounter->actsAsReset()) {
235                         if (isReset && areRenderersElementsSiblings(*currentRenderer, counterOwner)) {
236                             parent = currentCounter->parent();
237                             previousSibling = currentCounter;
238                             return parent;
239                         }
240                         parent = currentCounter;
241                         previousSibling = previousSiblingProtector.get();
242                         return true;
243                     }
244                     if (!isReset || !areRenderersElementsSiblings(*currentRenderer, counterOwner)) {
245                         parent = currentCounter->parent();
246                         previousSibling = currentCounter;
247                         return true;
248                     }
249                     previousSiblingProtector = currentCounter;
250                 }
251             }
252             // We come here if the previous sibling or parent of our owner renderer had no
253             // good counter, or we are a reset node and the counter on the previous sibling
254             // of our owner renderer was not a reset counter.
255             // Set a new goal for the end of the search.
256             searchEndRenderer = previousSiblingOrParent(*currentRenderer);
257         } else {
258             // We are searching descendants of a previous sibling of the renderer that the
259             // counter being placed is attached to.
260             if (currentCounter) {
261                 // We found a suitable counter.
262                 if (previousSiblingProtector) {
263                     // Since we had a suitable previous counter before, we should only consider this one as our 
264                     // previousSibling if it is a reset counter and hence the current previousSibling is its child.
265                     if (currentCounter->actsAsReset()) {
266                         previousSiblingProtector = currentCounter;
267                         // We are no longer interested in previous siblings of the currentRenderer or their children
268                         // as counters they may have attached cannot be the previous sibling of the counter we are placing.
269                         currentRenderer = parentOrPseudoHostElement(*currentRenderer)->renderer();
270                         continue;
271                     }
272                 } else
273                     previousSiblingProtector = currentCounter;
274                 currentRenderer = previousSiblingOrParent(*currentRenderer);
275                 continue;
276             }
277         }
278         // This function is designed so that the same test is not done twice in an iteration, except for this one
279         // which may be done twice in some cases. Rearranging the decision points though, to accommodate this 
280         // performance improvement would create more code duplication than is worthwhile in my oppinion and may further
281         // impede the readability of this already complex algorithm.
282         if (previousSiblingProtector)
283             currentRenderer = previousSiblingOrParent(*currentRenderer);
284         else
285             currentRenderer = previousInPreOrder(*currentRenderer);
286     }
287     return false;
288 }
289
290 static CounterNode* makeCounterNode(RenderElement& renderer, const AtomicString& identifier, bool alwaysCreateCounter)
291 {
292     if (renderer.hasCounterNodeMap()) {
293         if (CounterMap* nodeMap = counterMaps().get(&renderer)) {
294             if (CounterNode* node = nodeMap->get(identifier))
295                 return node;
296         }
297     }
298
299     bool isReset = false;
300     int value = 0;
301     if (!planCounter(renderer, identifier, isReset, value) && !alwaysCreateCounter)
302         return nullptr;
303
304     RefPtr<CounterNode> newParent = 0;
305     RefPtr<CounterNode> newPreviousSibling = 0;
306     RefPtr<CounterNode> newNode = CounterNode::create(renderer, isReset, value);
307     if (findPlaceForCounter(renderer, identifier, isReset, newParent, newPreviousSibling))
308         newParent->insertAfter(newNode.get(), newPreviousSibling.get(), identifier);
309     CounterMap* nodeMap;
310     if (renderer.hasCounterNodeMap())
311         nodeMap = counterMaps().get(&renderer);
312     else {
313         nodeMap = new CounterMap;
314         counterMaps().set(&renderer, std::unique_ptr<CounterMap>(nodeMap));
315         renderer.setHasCounterNodeMap(true);
316     }
317     nodeMap->set(identifier, newNode);
318     if (newNode->parent())
319         return newNode.get();
320     // Checking if some nodes that were previously counter tree root nodes
321     // should become children of this node now.
322     CounterMaps& maps = counterMaps();
323     Element* stayWithin = parentOrPseudoHostElement(renderer);
324     bool skipDescendants;
325     for (RenderElement* currentRenderer = nextInPreOrder(renderer, stayWithin); currentRenderer; currentRenderer = nextInPreOrder(*currentRenderer, stayWithin, skipDescendants)) {
326         skipDescendants = false;
327         if (!currentRenderer->hasCounterNodeMap())
328             continue;
329         CounterNode* currentCounter = maps.get(currentRenderer)->get(identifier);
330         if (!currentCounter)
331             continue;
332         skipDescendants = true;
333         if (currentCounter->parent())
334             continue;
335         if (stayWithin == parentOrPseudoHostElement(*currentRenderer) && currentCounter->hasResetType())
336             break;
337         newNode->insertAfter(currentCounter, newNode->lastChild(), identifier);
338     }
339     return newNode.get();
340 }
341
342 RenderCounter::RenderCounter(Document& document, const CounterContent& counter)
343     : RenderText(document, emptyString())
344     , m_counter(counter)
345     , m_counterNode(nullptr)
346     , m_nextForSameCounter(0)
347 {
348     view().addRenderCounter();
349 }
350
351 RenderCounter::~RenderCounter()
352 {
353     view().removeRenderCounter();
354
355     if (m_counterNode) {
356         m_counterNode->removeRenderer(this);
357         ASSERT(!m_counterNode);
358     }
359 }
360
361 const char* RenderCounter::renderName() const
362 {
363     return "RenderCounter";
364 }
365
366 bool RenderCounter::isCounter() const
367 {
368     return true;
369 }
370
371 String RenderCounter::originalText() const
372 {
373     if (!m_counterNode) {
374         RenderElement* beforeAfterContainer = parent();
375         while (true) {
376             if (!beforeAfterContainer)
377                 return String();
378             if (!beforeAfterContainer->isAnonymous() && !beforeAfterContainer->isPseudoElement())
379                 return String(); // RenderCounters are restricted to before and after pseudo elements
380             PseudoId containerStyle = beforeAfterContainer->style().styleType();
381             if ((containerStyle == BEFORE) || (containerStyle == AFTER))
382                 break;
383             beforeAfterContainer = beforeAfterContainer->parent();
384         }
385         makeCounterNode(*beforeAfterContainer, m_counter.identifier(), true)->addRenderer(const_cast<RenderCounter*>(this));
386         ASSERT(m_counterNode);
387     }
388     CounterNode* child = m_counterNode;
389     int value = child->actsAsReset() ? child->value() : child->countInParent();
390
391     String text = listMarkerText(m_counter.listStyle(), value);
392
393     if (!m_counter.separator().isNull()) {
394         if (!child->actsAsReset())
395             child = child->parent();
396         while (CounterNode* parent = child->parent()) {
397             text = listMarkerText(m_counter.listStyle(), child->countInParent())
398                 + m_counter.separator() + text;
399             child = parent;
400         }
401     }
402
403     return text;
404 }
405
406 void RenderCounter::updateCounter()
407 {
408     computePreferredLogicalWidths(0);
409 }
410
411 void RenderCounter::computePreferredLogicalWidths(float lead)
412 {
413 #ifndef NDEBUG
414     // FIXME: We shouldn't be modifying the tree in computePreferredLogicalWidths.
415     // Instead, we should properly hook the appropriate changes in the DOM and modify
416     // the render tree then. When that's done, we also won't need to override
417     // computePreferredLogicalWidths at all.
418     // https://bugs.webkit.org/show_bug.cgi?id=104829
419     SetLayoutNeededForbiddenScope layoutForbiddenScope(this, false);
420 #endif
421
422     setRenderedText(originalText());
423
424     RenderText::computePreferredLogicalWidths(lead);
425 }
426
427 void RenderCounter::invalidate()
428 {
429     m_counterNode->removeRenderer(this);
430     ASSERT(!m_counterNode);
431     if (documentBeingDestroyed())
432         return;
433     setNeedsLayoutAndPrefWidthsRecalc();
434 }
435
436 static void destroyCounterNodeWithoutMapRemoval(const AtomicString& identifier, CounterNode* node)
437 {
438     CounterNode* previous;
439     for (RefPtr<CounterNode> child = node->lastDescendant(); child && child != node; child = previous) {
440         previous = child->previousInPreOrder();
441         child->parent()->removeChild(child.get());
442         ASSERT(counterMaps().get(&child->owner())->get(identifier) == child);
443         counterMaps().get(&child->owner())->remove(identifier);
444     }
445     if (CounterNode* parent = node->parent())
446         parent->removeChild(node);
447 }
448
449 void RenderCounter::destroyCounterNodes(RenderElement& owner)
450 {
451     CounterMaps& maps = counterMaps();
452     CounterMaps::iterator mapsIterator = maps.find(&owner);
453     if (mapsIterator == maps.end())
454         return;
455     CounterMap* map = mapsIterator->value.get();
456     CounterMap::const_iterator end = map->end();
457     for (CounterMap::const_iterator it = map->begin(); it != end; ++it) {
458         destroyCounterNodeWithoutMapRemoval(it->key, it->value.get());
459     }
460     maps.remove(mapsIterator);
461     owner.setHasCounterNodeMap(false);
462 }
463
464 void RenderCounter::destroyCounterNode(RenderElement& owner, const AtomicString& identifier)
465 {
466     CounterMap* map = counterMaps().get(&owner);
467     if (!map)
468         return;
469     CounterMap::iterator mapIterator = map->find(identifier);
470     if (mapIterator == map->end())
471         return;
472     destroyCounterNodeWithoutMapRemoval(identifier, mapIterator->value.get());
473     map->remove(mapIterator);
474     // We do not delete "map" here even if empty because we expect to reuse
475     // it soon. In order for a renderer to lose all its counters permanently,
476     // a style change for the renderer involving removal of all counter
477     // directives must occur, in which case, RenderCounter::destroyCounterNodes()
478     // must be called.
479     // The destruction of the Renderer (possibly caused by the removal of its 
480     // associated DOM node) is the other case that leads to the permanent
481     // destruction of all counters attached to a Renderer. In this case
482     // RenderCounter::destroyCounterNodes() must be and is now called, too.
483     // RenderCounter::destroyCounterNodes() handles destruction of the counter
484     // map associated with a renderer, so there is no risk in leaking the map.
485 }
486
487 void RenderCounter::rendererRemovedFromTree(RenderElement& renderer)
488 {
489     if (!renderer.view().hasRenderCounters())
490         return;
491     RenderObject* currentRenderer = renderer.lastLeafChild();
492     if (!currentRenderer)
493         currentRenderer = &renderer;
494     while (true) {
495         if (is<RenderElement>(*currentRenderer))
496             destroyCounterNodes(downcast<RenderElement>(*currentRenderer));
497         if (currentRenderer == &renderer)
498             break;
499         currentRenderer = currentRenderer->previousInPreOrder();
500     }
501 }
502
503 static void updateCounters(RenderElement& renderer)
504 {
505     const CounterDirectiveMap* directiveMap = renderer.style().counterDirectives();
506     if (!directiveMap)
507         return;
508     CounterDirectiveMap::const_iterator end = directiveMap->end();
509     if (!renderer.hasCounterNodeMap()) {
510         for (CounterDirectiveMap::const_iterator it = directiveMap->begin(); it != end; ++it)
511             makeCounterNode(renderer, it->key, false);
512         return;
513     }
514     CounterMap* counterMap = counterMaps().get(&renderer);
515     ASSERT(counterMap);
516     for (CounterDirectiveMap::const_iterator it = directiveMap->begin(); it != end; ++it) {
517         RefPtr<CounterNode> node = counterMap->get(it->key);
518         if (!node) {
519             makeCounterNode(renderer, it->key, false);
520             continue;
521         }
522         RefPtr<CounterNode> newParent = 0;
523         RefPtr<CounterNode> newPreviousSibling = 0;
524         
525         findPlaceForCounter(renderer, it->key, node->hasResetType(), newParent, newPreviousSibling);
526         if (node != counterMap->get(it->key))
527             continue;
528         CounterNode* parent = node->parent();
529         if (newParent == parent && newPreviousSibling == node->previousSibling())
530             continue;
531         if (parent)
532             parent->removeChild(node.get());
533         if (newParent)
534             newParent->insertAfter(node.get(), newPreviousSibling.get(), it->key);
535     }
536 }
537
538 void RenderCounter::rendererSubtreeAttached(RenderElement& renderer)
539 {
540     if (!renderer.view().hasRenderCounters())
541         return;
542     Element* element = renderer.element();
543     if (element && !element->isPseudoElement())
544         element = element->parentElement();
545     else
546         element = renderer.generatingElement();
547     if (element && !element->renderer())
548         return; // No need to update if the parent is not attached yet
549     for (RenderObject* descendant = &renderer; descendant; descendant = descendant->nextInPreOrder(&renderer)) {
550         if (is<RenderElement>(*descendant))
551             updateCounters(downcast<RenderElement>(*descendant));
552     }
553 }
554
555 void RenderCounter::rendererStyleChanged(RenderElement& renderer, const RenderStyle* oldStyle, const RenderStyle* newStyle)
556 {
557     Element* element = renderer.generatingElement();
558     if (!element || !element->renderer())
559         return; // cannot have generated content or if it can have, it will be handled during attaching
560     const CounterDirectiveMap* newCounterDirectives;
561     const CounterDirectiveMap* oldCounterDirectives;
562     if (oldStyle && (oldCounterDirectives = oldStyle->counterDirectives())) {
563         if (newStyle && (newCounterDirectives = newStyle->counterDirectives())) {
564             CounterDirectiveMap::const_iterator newMapEnd = newCounterDirectives->end();
565             CounterDirectiveMap::const_iterator oldMapEnd = oldCounterDirectives->end();
566             for (CounterDirectiveMap::const_iterator it = newCounterDirectives->begin(); it != newMapEnd; ++it) {
567                 CounterDirectiveMap::const_iterator oldMapIt = oldCounterDirectives->find(it->key);
568                 if (oldMapIt != oldMapEnd) {
569                     if (oldMapIt->value == it->value)
570                         continue;
571                     RenderCounter::destroyCounterNode(renderer, it->key);
572                 }
573                 // We must create this node here, because the changed node may be a node with no display such as
574                 // as those created by the increment or reset directives and the re-layout that will happen will
575                 // not catch the change if the node had no children.
576                 makeCounterNode(renderer, it->key, false);
577             }
578             // Destroying old counters that do not exist in the new counterDirective map.
579             for (CounterDirectiveMap::const_iterator it = oldCounterDirectives->begin(); it !=oldMapEnd; ++it) {
580                 if (!newCounterDirectives->contains(it->key))
581                     RenderCounter::destroyCounterNode(renderer, it->key);
582             }
583         } else {
584             if (renderer.hasCounterNodeMap())
585                 RenderCounter::destroyCounterNodes(renderer);
586         }
587     } else if (newStyle && (newCounterDirectives = newStyle->counterDirectives())) {
588         CounterDirectiveMap::const_iterator newMapEnd = newCounterDirectives->end();
589         for (CounterDirectiveMap::const_iterator it = newCounterDirectives->begin(); it != newMapEnd; ++it) {
590             // We must create this node here, because the added node may be a node with no display such as
591             // as those created by the increment or reset directives and the re-layout that will happen will
592             // not catch the change if the node had no children.
593             makeCounterNode(renderer, it->key, false);
594         }
595     }
596 }
597
598 } // namespace WebCore
599
600 #ifndef NDEBUG
601
602 void showCounterRendererTree(const WebCore::RenderObject* renderer, const char* counterName)
603 {
604     if (!renderer)
605         return;
606     const WebCore::RenderObject* root = renderer;
607     while (root->parent())
608         root = root->parent();
609
610     AtomicString identifier(counterName);
611     for (const WebCore::RenderObject* current = root; current; current = current->nextInPreOrder()) {
612         if (!is<WebCore::RenderElement>(*current))
613             continue;
614         fprintf(stderr, "%c", (current == renderer) ? '*' : ' ');
615         for (const WebCore::RenderObject* parent = current; parent && parent != root; parent = parent->parent())
616             fprintf(stderr, "    ");
617         fprintf(stderr, "%p N:%p P:%p PS:%p NS:%p C:%p\n",
618             current, current->node(), current->parent(), current->previousSibling(),
619             current->nextSibling(), downcast<WebCore::RenderElement>(*current).hasCounterNodeMap() ?
620             counterName ? WebCore::counterMaps().get(downcast<WebCore::RenderElement>(current))->get(identifier) : (WebCore::CounterNode*)1 : (WebCore::CounterNode*)0);
621     }
622     fflush(stderr);
623 }
624
625 #endif // NDEBUG