Commit fa8f73f7 authored by Koji Ishii's avatar Koji Ishii Committed by Commit Bot

[FragmentItem] Rebaseline LayoutBlockFlow changes in paint/invalidation

This patch rebaselines paint/invalidation failures that are
caused when empty |LayoutBlowFlow| becomes non-empty, or
vice-versa.

This is |NGPaintFragment|-specific behavior. When
|LayoutBlockFlow| got or lost |NGPaintFragment|, its
|DisplayItemClient| changes between |LayoutBlockFlow| and
corresponding |NGPaintFragment|.

|FragmentItem| does not have this behavior; i.e., uses
|LayoutBlockFlow| as |DisplayItemClient|, and that the
invalidation results are more similar to legacy.

TBR=wangxianzhu@chromium.org, yosin@chromium.org

Bug: 982194
Change-Id: Id433962da7a32e51ca6046135ccc3e300b3cb7b6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096796Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748899}
parent fdebb6fe
......@@ -220,15 +220,9 @@ crbug.com/982194 media/picture-in-picture/v2/detached-iframe.html [ Failure ]
crbug.com/982194 media/picture-in-picture/v2/request-picture-in-picture-twice.html [ Failure ]
crbug.com/982194 media/picture-in-picture/v2/request-picture-in-picture.html [ Failure ]
crbug.com/874695 media/remoteplayback/prompt-twice-throws.html [ Pass ]
crbug.com/982194 paint/invalidation/background/background-misaligned.html [ Failure ]
crbug.com/982194 paint/invalidation/block-layout-inline-children-replaced.html [ Failure ]
crbug.com/982194 paint/invalidation/float-new-in-block.html [ Failure ]
crbug.com/982194 paint/invalidation/outline/focus-ring-on-child-move.html [ Failure ]
crbug.com/835484 paint/invalidation/outline/inline-focus.html [ Failure ]
crbug.com/982194 paint/invalidation/position/block-layout-inline-children-float-positioned.html [ Failure ]
crbug.com/982194 paint/invalidation/svg/svg-background-partial-redraw.html [ Failure ]
crbug.com/982194 paint/invalidation/svg/transform-focus-ring-repaint.html [ Failure ]
crbug.com/982194 paint/invalidation/table/cached-change-cell-sl-border-color.html [ Failure ]
crbug.com/676229 plugins/mouse-click-plugin-clears-selection.html [ Failure ]
crbug.com/718155 virtual/android/fullscreen/full-screen-iframe-not-allowed.html [ Failure ]
crbug.com/982194 virtual/audio-service/http/tests/media/video-frame-size-change.html [ Failure Pass ]
......@@ -1013,7 +1007,6 @@ crbug.com/982194 jquery/manipulation.html [ Pass Timeout ]
crbug.com/982194 jquery/offset.html [ Pass ]
crbug.com/982194 jquery/traversing.html [ Pass ]
crbug.com/982194 media/video-canvas-draw.html [ Failure Pass ]
crbug.com/982194 paint/invalidation/position/absolute-position-moved.html [ Failure ]
crbug.com/982194 paint/invalidation/svg/scroll-hit-test.xhtml [ Failure Pass ]
crbug.com/982194 virtual/android/url-bar/bottom-and-top-fixed-sticks-to-top.html [ Failure ]
crbug.com/908347 virtual/audio-service/media/autoplay/webaudio-audio-context-resume.html [ Pass ]
......
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [800, 600],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "LayoutNGBlockFlow (positioned) DIV id='target'",
"rect": [238, 241, 40, 40],
"reason": "disappeared"
}
]
}
]
}
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [800, 600],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "LayoutImage IMG",
"rect": [151, 117, 100, 100],
"reason": "appeared"
}
]
}
]
}
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [800, 600],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "LayoutNGBlockFlow (floating) DIV id='target'",
"rect": [8, 8, 100, 100],
"reason": "appeared"
}
]
}
]
}
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [785, 2016],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "LayoutNGBlockFlow (positioned) DIV id='absoluteDiv' class='absolute green'",
"rect": [100, 100, 100, 100],
"reason": "appeared"
}
]
},
{
"name": "LayoutNGBlockFlow (positioned) DIV class='fixed red'",
"bounds": [100, 100],
"contentsOpaque": true,
"backgroundColor": "#FF0000",
"paintInvalidations": [
{
"object": "LayoutNGBlockFlow (positioned) DIV id='absoluteDiv' class='absolute green'",
"rect": [100, 100, 100, 100],
"reason": "chunk disappeared"
}
],
"transform": 1
},
{
"name": "ContentsLayer for Vertical Scrollbar Layer",
"position": [785, 0],
"bounds": [15, 600],
"contentsOpaque": true
}
],
"transforms": [
{
"id": 1,
"transform": [
[1, 0, 0, 0],
[0, 1, 0, 0],
[0, 0, 1, 0],
[8, 1000, 0, 1]
]
}
]
}
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [800, 600],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "NGPhysicalTextFragment 'the quick brown fox jumped over the lazy dog'",
"rect": [309, 37, 292, 19],
"reason": "appeared"
},
{
"object": "NGPhysicalTextFragment 'the quick brown fox jumped over the lazy dog'",
"rect": [155, 37, 292, 19],
"reason": "appeared"
}
]
}
]
}
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [800, 600],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "LayoutNGBlockFlow (positioned) DIV id='revealer'",
"rect": [9, 153, 200, 100],
"reason": "disappeared"
}
]
}
]
}
{
"layers": [
{
"name": "Scrolling Contents Layer",
"bounds": [800, 600],
"contentsOpaque": true,
"backgroundColor": "#FFFFFF",
"paintInvalidations": [
{
"object": "LayoutTableRow TR",
"rect": [8, 8, 114, 54],
"reason": "style change"
},
{
"object": "NGPhysicalTextFragment 'A'",
"rect": [90, 30, 12, 19],
"reason": "disappeared"
}
]
}
]
}
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment