third_party/WebKit/LayoutTests/compositing/resources/composited-iframe-hidden-2.html - Issue 1616183002: Don't composite layers with an invisible frameview.

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/resources/composited-iframe-hidden-2.html

Issue 1616183002: Don't composite layers with an invisible frameview. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Remove verbose logging. Created 4 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:

View unified diff | Download patch

« third_party/WebKit/LayoutTests/compositing/composited-iframe-hidden-expected.html ('K') | « third_party/WebKit/LayoutTests/compositing/iframes/invisible-nested-iframe-show.html ('k') | third_party/WebKit/LayoutTests/compositing/resources/shine.png » ('j') | third_party/WebKit/Source/core/frame/FrameView.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <!DOCTYPE html>
	chrishtr 2016/01/28 00:32:31 Rename file to composited-iframe-hidden-subframe.h Rename file to composited-iframe-hidden-subframe.html wkorman 2016/01/28 00:55:21 Done. Show quoted text On 2016/01/28 at 00:32:31, chrishtr wrote: > Rename file to composited-iframe-hidden-subframe.html Done.
	2 <style>

	3 .animatedBackground {

	4 width: 70px;

	5 height: 80px;

	6 position: absolute;

	7 background: url("./shine.png") no-repeat 0 0 transparent;
	chrishtr 2016/01/28 00:32:31 Try to use an existing infinitely animated gif. Th Try to use an existing infinitely animated gif. There is a red-green blinking one somewhere IIRC. wkorman 2016/01/28 00:55:21 Done. Took a 100x100 blue from another existing te Show quoted text On 2016/01/28 at 00:32:31, chrishtr wrote: > Try to use an existing infinitely animated gif. There is a red-green blinking one somewhere > IIRC. Done. Took a 100x100 blue from another existing test. Doesn't need to be an animated image, and was able to remove the background-color property as well.
	8 background-color: red;

	9 animation: shineSmall 4s steps(6) infinite;

	10 }

	11

	12 .transformedItem {

	13 transform: translate3d(42px, 43px, 0);

	14 width: 40px;

	15 height: 50px;

	16 background-color: green;

	17 }

	18

	19 @keyframes shineSmall {

	20 100% {

	21 background-position: 9px 10px

	22 }

	23 }

	24 </style>

	25 <div class="animatedBackground"></div>

	26 <div class="transformedItem"></div>

OLD	NEW