Skip to content

GitLab

T
tangled
Commit f01d8fd8 authored by Philip Rogers's avatar Philip Rogers Committed by Commit Bot
Browse files
Options

Switch 3 tests in perf/page_sets to use will-change transform 

In crbug.com/1012775 we are unifying how trivial 3d transforms are
handled across platforms by making them not a compositing hint. This
caused performance regressions in tough_animation_cases,
tough_canvas_cases, and tough_scrolling_cases which were using trivial
3d transforms as a compositing hint.

This patch updates these tests to be more explicit about layer triggers
by using "will-change: transform".

Bug: 1012775, 1016320
Change-Id: Ia2978b867e439bbffa173da747cda7ecd5a9f2bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1872743
Auto-Submit: Philip Rogers <pdr@chromium.org>
Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: default avatarAnnie Sullivan <sullivan@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#707936}
parent 2303042a
Expand all Hide whitespace changes
Inline Side-by-side
Showing with 3 additions and 3 deletions
tools/perf/page_sets/tough_animation_cases/overlay_background_color_css_transitions.html
View file @ f01d8fd8
......@@ -53,7 +53,7 @@ body {
position: absolute;
width: 600px;
height: 600px;
transform: translateZ(0);
will-change: transform;
transition: background-color 1s linear;
}
</style>
......
tools/perf/page_sets/tough_canvas_cases/compositing_huge_div/index.html
View file @ f01d8fd8
......@@ -3,7 +3,7 @@
<title>Compositing - Huge &lt;div&gt;</title>
</head>
<body>
<div id="big" style="-webkit-transform: translateZ(0);">
<div id="big" style="will-change: transform;">
<div style="left: 0px; top: 0px; width: 100px; height: 100px; position: absolute; background-color: green;"></div>
<div style="left: 3000000px; top: 1000000px; width: 100px; height: 100px; position: absolute; background-color: red;" id="bottomRight"></div>
</div>
......
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