Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(11)

Side by Side Diff: third_party/WebKit/LayoutTests/compositing/squashing/add-remove-squashed-layers-expected.txt

Issue 1951043002: Don't let PaintInvalidationLocationChange shadow other reasons (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 Test that layers can be nicely added or removed from a squashed layer, without u nnecessary repaints on any layer. Click anywhere to test interactively; keep cli cking to proceed through the test. 1 Test that layers can be nicely added or removed from a squashed layer, without u nnecessary repaints on any layer. Click anywhere to test interactively; keep cli cking to proceed through the test.
2 2
3 CASE 1, original layer tree with overlap1 and overlap2: 3 CASE 1, original layer tree with overlap1 and overlap2:
4 { 4 {
5 "bounds": [800, 600], 5 "bounds": [800, 600],
6 "children": [ 6 "children": [
7 { 7 {
8 "bounds": [800, 600], 8 "bounds": [800, 600],
9 "contentsOpaque": true, 9 "contentsOpaque": true,
10 "drawsContent": true, 10 "drawsContent": true,
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
56 "backgroundColor": "#808080" 56 "backgroundColor": "#808080"
57 }, 57 },
58 { 58 {
59 "position": [140, 140], 59 "position": [140, 140],
60 "bounds": [260, 260], 60 "bounds": [260, 260],
61 "drawsContent": true, 61 "drawsContent": true,
62 "paintInvalidations": [ 62 "paintInvalidations": [
63 { 63 {
64 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'", 64 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'",
65 "rect": [160, 160, 100, 100], 65 "rect": [160, 160, 100, 100],
66 "reason": "location change" 66 "reason": "layoutObject insertion"
67 } 67 }
68 ] 68 ]
69 } 69 }
70 ] 70 ]
71 } 71 }
72 ] 72 ]
73 } 73 }
74 ] 74 ]
75 } 75 }
76 CASE 3, overlap2 gets removed. Since this does not resize the layer, there shoul d only be a repaint of overlap2: 76 CASE 3, overlap2 gets removed. Since this does not resize the layer, there shoul d only be a repaint of overlap2:
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 "reason": "layer" 140 "reason": "layer"
141 }, 141 },
142 { 142 {
143 "object": "LayoutBlockFlow (positioned) DIV id='A' class='over lap1'", 143 "object": "LayoutBlockFlow (positioned) DIV id='A' class='over lap1'",
144 "rect": [0, 0, 100, 100], 144 "rect": [0, 0, 100, 100],
145 "reason": "layoutObject removal" 145 "reason": "layoutObject removal"
146 }, 146 },
147 { 147 {
148 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'", 148 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'",
149 "rect": [0, 0, 100, 100], 149 "rect": [0, 0, 100, 100],
150 "reason": "location change" 150 "reason": "compositing update"
151 } 151 }
152 ] 152 ]
153 } 153 }
154 ] 154 ]
155 } 155 }
156 ] 156 ]
157 } 157 }
158 ] 158 ]
159 } 159 }
160 CASE 5, overlap2 gets added back: 160 CASE 5, overlap2 gets added back:
(...skipping 22 matching lines...) Expand all
183 "backgroundColor": "#808080" 183 "backgroundColor": "#808080"
184 }, 184 },
185 { 185 {
186 "position": [220, 220], 186 "position": [220, 220],
187 "bounds": [180, 180], 187 "bounds": [180, 180],
188 "drawsContent": true, 188 "drawsContent": true,
189 "paintInvalidations": [ 189 "paintInvalidations": [
190 { 190 {
191 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'", 191 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'",
192 "rect": [80, 80, 100, 100], 192 "rect": [80, 80, 100, 100],
193 "reason": "location change" 193 "reason": "compositing update"
194 }, 194 },
195 { 195 {
196 "object": "LayoutBlockFlow (positioned) DIV id='B' class='over lap2'", 196 "object": "LayoutBlockFlow (positioned) DIV id='B' class='over lap2'",
197 "rect": [0, 0, 100, 100], 197 "rect": [0, 0, 100, 100],
198 "reason": "layoutObject insertion" 198 "reason": "layoutObject insertion"
199 }, 199 },
200 { 200 {
201 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'", 201 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'",
202 "rect": [0, 0, 100, 100], 202 "rect": [0, 0, 100, 100],
203 "reason": "layer" 203 "reason": "layer"
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 "backgroundColor": "#808080" 236 "backgroundColor": "#808080"
237 }, 237 },
238 { 238 {
239 "position": [140, 140], 239 "position": [140, 140],
240 "bounds": [180, 180], 240 "bounds": [180, 180],
241 "drawsContent": true, 241 "drawsContent": true,
242 "paintInvalidations": [ 242 "paintInvalidations": [
243 { 243 {
244 "object": "LayoutBlockFlow (positioned) DIV id='B' class='over lap2'", 244 "object": "LayoutBlockFlow (positioned) DIV id='B' class='over lap2'",
245 "rect": [80, 80, 100, 100], 245 "rect": [80, 80, 100, 100],
246 "reason": "location change" 246 "reason": "compositing update"
247 }, 247 },
248 { 248 {
249 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'", 249 "object": "LayoutBlockFlow (positioned) DIV id='C' class='over lap3'",
250 "rect": [80, 80, 100, 100], 250 "rect": [80, 80, 100, 100],
251 "reason": "layoutObject removal" 251 "reason": "layoutObject removal"
252 }, 252 },
253 { 253 {
254 "object": "LayoutBlockFlow (positioned) DIV id='A' class='over lap1'", 254 "object": "LayoutBlockFlow (positioned) DIV id='A' class='over lap1'",
255 "rect": [0, 0, 100, 100], 255 "rect": [0, 0, 100, 100],
256 "reason": "layoutObject insertion" 256 "reason": "layoutObject insertion"
257 }, 257 },
258 { 258 {
259 "object": "LayoutBlockFlow (positioned) DIV id='B' class='over lap2'", 259 "object": "LayoutBlockFlow (positioned) DIV id='B' class='over lap2'",
260 "rect": [0, 0, 100, 100], 260 "rect": [0, 0, 100, 100],
261 "reason": "layer" 261 "reason": "layer"
262 } 262 }
263 ] 263 ]
264 } 264 }
265 ] 265 ]
266 } 266 }
267 ] 267 ]
268 } 268 }
269 ] 269 ]
270 } 270 }
271 271
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698