Chromium Code Reviews Chromium Code Reviews
Issue 2019583002:
Don't include scroll offset in offsetFromLayoutObject for scrolling contents layers. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| OLD | NEW |
|---|---|
| 1 { | 1 { |
| 2 "bounds": [800, 600], | 2 "bounds": [800, 600], |
| 3 "children": [ | 3 "children": [ |
| 4 { | 4 { |
| 5 "bounds": [800, 600], | 5 "bounds": [800, 600], |
| 6 "contentsOpaque": true, | 6 "contentsOpaque": true, |
| 7 "drawsContent": true, | 7 "drawsContent": true, |
| 8 "children": [ | 8 "children": [ |
| 9 { | 9 { |
| 10 "position": [8, 8], | 10 "position": [8, 8], |
| 11 "bounds": [200, 200], | 11 "bounds": [200, 200], |
| 12 "contentsOpaque": true, | |
| 12 "shouldFlattenTransform": false, | 13 "shouldFlattenTransform": false, |
| 13 "drawsContent": true, | 14 "drawsContent": true, |
| 14 "paintInvalidations": [ | 15 "backgroundColor": "#D3D3D3", |
| 15 { | |
| 16 "object": "LayoutBlockFlow DIV id='indicator'", | |
| 17 "rect": [0, 0, 185, 200], | |
| 18 "reason": "style change" | |
| 19 } | |
| 20 ], | |
| 21 "children": [ | 16 "children": [ |
| 22 { | 17 { |
| 23 "bounds": [185, 185], | 18 "bounds": [185, 185], |
| 24 "shouldFlattenTransform": false, | 19 "shouldFlattenTransform": false, |
| 25 "children": [ | 20 "children": [ |
| 26 { | 21 { |
| 27 "bounds": [185, 1200], | 22 "bounds": [300, 300], |
| 28 "drawsContent": true, | 23 "drawsContent": true, |
| 29 "paintInvalidations": [ | 24 "paintInvalidations": [ |
| 30 { | 25 { |
| 31 "object": "LayoutBlockFlow DIV id='indicator'", | 26 "object": "LayoutBlockFlow DIV id='target'", |
| 32 "rect": [0, 0, 185, 200], | 27 "rect": [100, 100, 200, 200], |
|
chrishtr
2016/05/31 18:14:47
This is the scrolling contents layer.
| |
| 33 "reason": "style change" | 28 "reason": "style change" |
| 34 } | 29 } |
| 30 ], | |
| 31 "children": [ | |
| 32 { | |
| 33 "bounds": [200, 200], | |
| 34 "contentsOpaque": true, | |
| 35 "drawsContent": true, | |
| 36 "backgroundColor": "#ADD8E6" | |
| 37 }, | |
| 38 { | |
| 39 "bounds": [300, 300], | |
| 40 "drawsContent": true, | |
| 41 "paintInvalidations": [ | |
| 42 { | |
| 43 "object": "LayoutBlockFlow DIV id='target'", | |
|
chrishtr
2016/05/31 18:14:47
This is the foreground layer.
| |
| 44 "rect": [100, 100, 200, 200], | |
| 45 "reason": "style change" | |
| 46 } | |
| 47 ] | |
| 48 } | |
| 35 ] | 49 ] |
| 36 } | 50 } |
| 37 ] | 51 ] |
| 38 }, | 52 }, |
| 39 { | 53 { |
| 40 "bounds": [200, 200], | 54 "bounds": [200, 200], |
| 41 "children": [ | 55 "children": [ |
| 42 { | 56 { |
| 43 "position": [0, 185], | 57 "position": [0, 185], |
| 44 "bounds": [185, 15] | 58 "bounds": [185, 15] |
| 45 }, | 59 }, |
| 46 { | 60 { |
| 47 "position": [185, 0], | 61 "position": [185, 0], |
| 48 "bounds": [15, 185], | 62 "bounds": [15, 185] |
| 49 "paintInvalidations": [ | |
| 50 { | |
| 51 "object": "Vertical Scrollbar Layer", | |
| 52 "rect": [0, 0, 15, 185], | |
| 53 "reason": "full" | |
| 54 } | |
| 55 ] | |
| 56 }, | 63 }, |
| 57 { | 64 { |
| 58 "position": [185, 185], | 65 "position": [185, 185], |
| 59 "bounds": [15, 15], | 66 "bounds": [15, 15], |
| 60 "drawsContent": true | 67 "drawsContent": true |
| 61 } | 68 } |
| 62 ] | 69 ] |
| 63 } | 70 } |
| 64 ] | 71 ] |
| 65 } | 72 } |
| 66 ] | 73 ] |
| 67 } | 74 } |
| 68 ] | 75 ] |
| 69 } | 76 } |
| 70 | 77 |
| OLD | NEW |
