OLD | NEW |
---|---|
1 /* | 1 /* |
2 * Copyright (c) 2009, Google Inc. All rights reserved. | 2 * Copyright (c) 2009, Google Inc. All rights reserved. |
3 * | 3 * |
4 * Redistribution and use in source and binary forms, with or without | 4 * Redistribution and use in source and binary forms, with or without |
5 * modification, are permitted provided that the following conditions are | 5 * modification, are permitted provided that the following conditions are |
6 * met: | 6 * met: |
7 * | 7 * |
8 * * Redistributions of source code must retain the above copyright | 8 * * 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 * * Redistributions in binary form must reproduce the above | 10 * * Redistributions in binary form must reproduce the above |
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
118 return false; | 118 return false; |
119 } | 119 } |
120 | 120 |
121 // The SVG addFocusRingRects() method adds rects in local coordinates so the def ault absoluteFocusRingQuads | 121 // The SVG addFocusRingRects() method adds rects in local coordinates so the def ault absoluteFocusRingQuads |
122 // returns incorrect values for SVG objects. Overriding this method provides acc ess to the absolute bounds. | 122 // returns incorrect values for SVG objects. Overriding this method provides acc ess to the absolute bounds. |
123 void RenderSVGModelObject::absoluteFocusRingQuads(Vector<FloatQuad>& quads) | 123 void RenderSVGModelObject::absoluteFocusRingQuads(Vector<FloatQuad>& quads) |
124 { | 124 { |
125 quads.append(localToAbsoluteQuad(FloatQuad(paintInvalidationRectInLocalCoord inates()))); | 125 quads.append(localToAbsoluteQuad(FloatQuad(paintInvalidationRectInLocalCoord inates()))); |
126 } | 126 } |
127 | 127 |
128 InvalidationReason RenderSVGModelObject::invalidatePaintIfNeeded(const PaintInva lidationState& paintInvalidationState, const RenderLayerModelObject& paintInvali dationContainer) | 128 void RenderSVGModelObject::invalidateTreeIfNeeded(const PaintInvalidationState& paintInvalidationState) |
129 { | 129 { |
130 ForceHorriblySlowRectMapping slowRectMapping(&paintInvalidationState); | 130 ASSERT(!needsLayout()); |
131 | 131 |
132 return RenderObject::invalidatePaintIfNeeded(paintInvalidationState, paintIn validationContainer); | 132 // If we didn't need paint invalidation then our children don't need as well . |
pdr.
2014/09/25 05:38:59
Nit: In my experience, children are extremely need
fs
2014/09/25 12:17:53
My diplomatic response to this would be: "No comme
fs
2014/09/25 12:30:26
s/this statement/the statement the comment makes/
dsinclair
2014/09/25 13:14:11
As part of repaint-after-layout, we made this an i
| |
133 // Skip walking down the tree as everything should be fine below us. | |
134 if (!shouldCheckForPaintInvalidation(paintInvalidationState)) | |
135 return; | |
136 | |
137 invalidatePaintIfNeeded(paintInvalidationState, paintInvalidationState.paint InvalidationContainer()); | |
138 clearPaintInvalidationState(paintInvalidationState); | |
139 | |
140 PaintInvalidationState childPaintInvalidationState(paintInvalidationState, * this); | |
141 invalidatePaintOfSubtreesIfNeeded(childPaintInvalidationState); | |
133 } | 142 } |
134 | 143 |
135 } // namespace blink | 144 } // namespace blink |
OLD | NEW |