third_party/WebKit/LayoutTests/platform/linux/paint/invalidation/invalidate-caret-in-composited-scrolling-container-expected.txt - Issue 2665823002: Invalidate caret during paint invalidation

Side by Side Diff: third_party/WebKit/LayoutTests/platform/linux/paint/invalidation/invalidate-caret-in-composited-scrolling-container-expected.txt

Issue 2665823002: Invalidate caret during paint invalidation (Closed)

Patch Set: Rebaseline Created 3 years, 10 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:

« third_party/WebKit/LayoutTests/platform/linux/paint/invalidation/caret-with-composited-scroll-expected.txt ('K') | « third_party/WebKit/LayoutTests/platform/linux/paint/invalidation/inline-outline-repaint-expected.txt ('k') | third_party/WebKit/LayoutTests/platform/linux/paint/invalidation/invalidate-caret-in-non-composited-scrolling-container-expected.txt » ('j') | third_party/WebKit/Source/core/editing/FrameCaret.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 {	1 {

2 "layers": [	2 "layers": [

3 {	3 {

4 "name": "LayoutView #document",	4 "name": "LayoutView #document",

5 "bounds": [800, 600],	5 "bounds": [800, 600],

6 "contentsOpaque": true,	6 "contentsOpaque": true,

7 "drawsContent": true	7 "drawsContent": true

8 },	8 },

9 {	9 {

10 "name": "LayoutTextControl INPUT id='root'",	10 "name": "LayoutTextControl INPUT id='root'",

(...skipping 12 matching lines...) Expand all Loading...
23 "rect": [0, 0, 64, 22],	23 "rect": [0, 0, 64, 22],

24 "reason": "full"	24 "reason": "full"

25 }	25 }

26 ]	26 ]

27 },	27 },

28 {	28 {

29 "name": "LayoutBlockFlow DIV id='inner-editor'",	29 "name": "LayoutBlockFlow DIV id='inner-editor'",

30 "position": [3, 4],	30 "position": [3, 4],

31 "bounds": [60, 16],	31 "bounds": [60, 16],

32 "shouldFlattenTransform": false,	32 "shouldFlattenTransform": false,

33 "drawsContent": true	33 "drawsContent": true,

	34 "paintInvalidations": [

	35 {

	36 "object": "LayoutBlockFlow DIV id='inner-editor'",

	37 "rect": [57, -1, 3, 18],

	38 "reason": "caret"

	39 }

	40 ]

34 },	41 },

35 {	42 {

36 "name": "Scrolling Layer",	43 "name": "Scrolling Layer",

37 "bounds": [60, 16],	44 "bounds": [60, 16],

38 "shouldFlattenTransform": false	45 "shouldFlattenTransform": false

39 },	46 },

40 {	47 {

41 "name": "Scrolling Contents Layer",	48 "name": "Scrolling Contents Layer",

42 "bounds": [75, 16],	49 "bounds": [75, 16],

43 "drawsContent": true,	50 "drawsContent": true,

44 "paintInvalidations": [	51 "underPaintInvalidations": [
	chrishtr 2017/01/31 22:51:03 Under-invalidations? Under-invalidations? Xianzhu 2017/02/02 23:42:06 Oops. Wondering why I overlooked this. We should a Show quoted text On 2017/01/31 22:51:03, chrishtr wrote: > Under-invalidations? Oops. Wondering why I overlooked this. We should always invalidate caret on the scrolling layer. Fixed.
45 {	52 {

46 "object": "LayoutText #text",	53 "x": 73,

47 "rect": [72, -1, 3, 18],	54 "y": 0,

48 "reason": "invalidate paint rectangle"	55 "oldPixel": "#00000000",

	56 "newPixel": "#000000"

	57 },

	58 {

	59 "x": 73,

	60 "y": 1,

	61 "oldPixel": "#00000000",

	62 "newPixel": "#000000"

	63 },

	64 {

	65 "x": 73,

	66 "y": 2,

	67 "oldPixel": "#00000000",

	68 "newPixel": "#000000"

	69 },

	70 {

	71 "x": 73,

	72 "y": 3,

	73 "oldPixel": "#00000000",

	74 "newPixel": "#000000"

	75 },

	76 {

	77 "x": 73,

	78 "y": 4,

	79 "oldPixel": "#00000000",

	80 "newPixel": "#000000"

	81 },

	82 {

	83 "x": 73,

	84 "y": 5,

	85 "oldPixel": "#00000000",

	86 "newPixel": "#000000"

	87 },

	88 {

	89 "x": 73,

	90 "y": 6,

	91 "oldPixel": "#00000000",

	92 "newPixel": "#000000"

	93 },

	94 {

	95 "x": 73,

	96 "y": 7,

	97 "oldPixel": "#00000000",

	98 "newPixel": "#000000"

	99 },

	100 {

	101 "x": 73,

	102 "y": 8,

	103 "oldPixel": "#00000000",

	104 "newPixel": "#000000"

	105 },

	106 {

	107 "x": 73,

	108 "y": 9,

	109 "oldPixel": "#00000000",

	110 "newPixel": "#000000"

	111 },

	112 {

	113 "x": 73,

	114 "y": 10,

	115 "oldPixel": "#00000000",

	116 "newPixel": "#000000"

	117 },

	118 {

	119 "x": 73,

	120 "y": 11,

	121 "oldPixel": "#00000000",

	122 "newPixel": "#000000"

	123 },

	124 {

	125 "x": 73,

	126 "y": 12,

	127 "oldPixel": "#00000000",

	128 "newPixel": "#000000"

	129 },

	130 {

	131 "x": 73,

	132 "y": 13,

	133 "oldPixel": "#00000000",

	134 "newPixel": "#000000"

	135 },

	136 {

	137 "x": 73,

	138 "y": 14,

	139 "oldPixel": "#00000000",

	140 "newPixel": "#000000"

	141 },

	142 {

	143 "x": 73,

	144 "y": 15,

	145 "oldPixel": "#00000000",

	146 "newPixel": "#000000"

49 }	147 }

50 ]	148 ]

51 },	149 },

52 {	150 {

53 "name": "Overflow Controls Host Layer",	151 "name": "Overflow Controls Host Layer",

54 "bounds": [60, 16]	152 "bounds": [60, 16]

55 },	153 },

56 {	154 {

57 "name": "Horizontal Scrollbar Layer",	155 "name": "Horizontal Scrollbar Layer",

58 "position": [0, 16],	156 "position": [0, 16],

59 "bounds": [60, 0],	157 "bounds": [60, 0],

60 "drawsContent": true	158 "drawsContent": true

61 }	159 }

62 ],	160 ],

63 "objectPaintInvalidations": [	161 "objectPaintInvalidations": [

64 {	162 {

65 "object": "Caret",	163 "object": "LayoutTextControl INPUT id='root'",

66 "reason": "invalidate paint rectangle"	164 "reason": "subtree"

67 },	165 },

68 {	166 {

69 "object": "LayoutTextControl INPUT id='root'",	167 "object": "Caret",

70 "reason": "subtree"	168 "reason": "caret"

	169 },

	170 {

	171 "object": "Caret",

	172 "reason": "caret"

71 }	173 }

72 ]	174 ]

73 }	175 }

74	176

OLD	NEW