third_party/WebKit/LayoutTests/compositing/flex-composited-animated-table-row-background.html - Issue 2784673004: Consider compositing state in row background display item client accessor.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/compositing/flex-composited-animated-table-row-background.html

Issue 2784673004: Consider compositing state in row background display item client accessor. (Closed)

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

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/compositing/flex-composited-animated-table-row-background-expected.html » ('j') | third_party/WebKit/LayoutTests/compositing/flex-composited-animated-table-row-background-expected.html » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE html>
	Xianzhu 2017/03/29 00:43:53 I think the bug case is that the row display item I think the bug case is that the row display item client created when a cell was composited is used when the cell is not composited. Wdyt to make a simpler test case by first compositing and then uncompositing a table cell? wkorman 2017/03/29 18:48:58 I took an initial stab but couldn't repro, added h Show quoted text On 2017/03/29 00:43:53, Xianzhu wrote: > I think the bug case is that the row display item client created when a cell was > composited is used when the cell is not composited. Wdyt to make a simpler test > case by first compositing and then uncompositing a table cell? I took an initial stab but couldn't repro, added hack attempt and note to the bug.
	2 <style>

	3 .flex-item {

	4 flex: 1 1 ;

	5 position: relative;

	6 }

	7

	8 .moving-item {

	9 animation-name: moving;

	10 animation-duration: 1s;

	11 }

	12

	13 @keyframes moving {

	14 0% {

	15 flex: 1 1;

	16 opacity: 0.6;

	17 }

	18 100% {

	19 flex: 0.0001 10;

	20 }

	21 }

	22 </style>

	23 <div style="display: flex">

	24 <div id="target" class="flex-item moving-item">y</div>

	25 <div class="flex-item">

	26 <table>

	27 <tr style="background-color: red">

	28 <td style="position: relative">

	29 x

	30 </td>

	31 </tr>

	32 </table>

	33 </div>

	34 </div>

	35 <script>

	36 if (window.testRunner)

	37 testRunner.waitUntilDone();

	38 target.addEventListener('animationend', function(event) {

	39 target.classList.remove('moving-item');

	40 if (window.testRunner)

	41 testRunner.notifyDone();

	42 });

	43 </script>

OLD	NEW