Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2009, 2010 Apple Inc. All rights reserved. | 2 * Copyright (C) 2009, 2010 Apple Inc. All rights reserved. |
| 3 * Copyright (C) Research In Motion Limited 2010. All rights reserved. | 3 * Copyright (C) Research In Motion Limited 2010. All rights reserved. |
| 4 * | 4 * |
| 5 * Redistribution and use in source and binary forms, with or without | 5 * Redistribution and use in source and binary forms, with or without |
| 6 * modification, are permitted provided that the following conditions | 6 * modification, are permitted provided that the following conditions |
| 7 * are met: | 7 * are met: |
| 8 * 1. Redistributions of source code must retain the above copyright | 8 * 1. Redistributions of source code must retain the above copyright |
| 9 * notice, this list of conditions and the following disclaimer. | 9 * notice, this list of conditions and the following disclaimer. |
| 10 * 2. Redistributions in binary form must reproduce the above copyright | 10 * 2. Redistributions in binary form must reproduce the above copyright |
| (...skipping 10 matching lines...) Expand all Loading... | |
| 21 * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY | 21 * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY |
| 22 * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | 22 * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
| 23 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | 23 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| 24 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 24 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 25 */ | 25 */ |
| 26 | 26 |
| 27 #include "config.h" | 27 #include "config.h" |
| 28 #include "core/layout/LayoutObjectChildList.h" | 28 #include "core/layout/LayoutObjectChildList.h" |
| 29 | 29 |
| 30 #include "core/dom/AXObjectCache.h" | 30 #include "core/dom/AXObjectCache.h" |
| 31 #include "core/dom/MarkerPseudoElement.h" | |
| 31 #include "core/layout/LayoutCounter.h" | 32 #include "core/layout/LayoutCounter.h" |
| 32 #include "core/layout/LayoutObject.h" | 33 #include "core/layout/LayoutObject.h" |
| 33 #include "core/layout/LayoutView.h" | 34 #include "core/layout/LayoutView.h" |
| 34 #include "core/style/ComputedStyle.h" | 35 #include "core/style/ComputedStyle.h" |
| 35 #include "core/paint/DeprecatedPaintLayer.h" | 36 #include "core/paint/DeprecatedPaintLayer.h" |
| 36 | 37 |
| 37 namespace blink { | 38 namespace blink { |
| 38 | 39 |
| 39 void LayoutObjectChildList::destroyLeftoverChildren() | 40 void LayoutObjectChildList::destroyLeftoverChildren() |
| 40 { | 41 { |
| 41 while (firstChild()) { | 42 while (firstChild()) { |
| 43 // RuntimeEnabledFeatures::listMarkerPseudoElementEnabled | |
|
esprehn
2015/04/22 07:45:46
??
dsinclair
2015/04/22 20:00:39
I need to update this comment when we remove the o
| |
| 44 // List markers are owned by the list marker pseudo element adn so don't get destroyed by this container. | |
|
esprehn
2015/04/22 07:45:46
typo.
dsinclair
2015/04/22 20:00:39
Done.
| |
| 45 // else | |
|
esprehn
2015/04/22 07:45:46
huh?
dsinclair
2015/04/22 20:00:39
Ditto.
| |
| 42 // List markers are owned by their enclosing list and so don't get destr oyed by this container. | 46 // List markers are owned by their enclosing list and so don't get destr oyed by this container. |
| 43 if (firstChild()->isListMarker()) { | 47 if (firstChild()->isListMarker()) { |
| 44 firstChild()->remove(); | 48 firstChild()->remove(); |
| 45 continue; | 49 continue; |
| 46 } | 50 } |
| 47 | 51 |
| 48 // Destroy any anonymous children remaining in the render tree, as well as implicit (shadow) DOM elements like those used in the engine-based text field s. | 52 // Destroy any anonymous children remaining in the render tree, as well as implicit (shadow) DOM elements like those used in the engine-based text field s. |
| 49 if (firstChild()->node()) | 53 if (firstChild()->node()) |
| 50 firstChild()->node()->setLayoutObject(nullptr); | 54 firstChild()->node()->setLayoutObject(nullptr); |
| 51 firstChild()->destroy(); | 55 firstChild()->destroy(); |
| (...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 177 DisableCompositingQueryAsserts disabler; | 181 DisableCompositingQueryAsserts disabler; |
| 178 // FIXME: We should not allow paint invalidation out of paint invalidation s tate. crbug.com/457415 | 182 // FIXME: We should not allow paint invalidation out of paint invalidation s tate. crbug.com/457415 |
| 179 DisablePaintInvalidationStateAsserts paintInvalidationAssertDisabler; | 183 DisablePaintInvalidationStateAsserts paintInvalidationAssertDisabler; |
| 180 const LayoutBoxModelObject* paintInvalidationContainer = oldChild.containerF orPaintInvalidation(); | 184 const LayoutBoxModelObject* paintInvalidationContainer = oldChild.containerF orPaintInvalidation(); |
| 181 oldChild.invalidatePaintUsingContainer(paintInvalidationContainer, oldChild. previousPaintInvalidationRect(), PaintInvalidationRendererRemoval); | 185 oldChild.invalidatePaintUsingContainer(paintInvalidationContainer, oldChild. previousPaintInvalidationRect(), PaintInvalidationRendererRemoval); |
| 182 if (RuntimeEnabledFeatures::slimmingPaintEnabled()) | 186 if (RuntimeEnabledFeatures::slimmingPaintEnabled()) |
| 183 oldChild.invalidateDisplayItemClients(*paintInvalidationContainer); | 187 oldChild.invalidateDisplayItemClients(*paintInvalidationContainer); |
| 184 } | 188 } |
| 185 | 189 |
| 186 } // namespace blink | 190 } // namespace blink |
| OLD | NEW |