summaryrefslogtreecommitdiffstats
path: root/courgette
diff options
context:
space:
mode:
authorfalken <falken@chromium.org>2015-12-08 00:36:56 -0800
committerCommit bot <commit-bot@chromium.org>2015-12-08 08:37:35 +0000
commit7696afc5c0ce2f160ff27c7c86821441ddfc4b75 (patch)
treea32b603f19d3ec0c2229bdd7eee9fe093b8c013d /courgette
parent67d96fd3ee108e5b94bdb6e09200108d36ad152d (diff)
downloadchromium_src-7696afc5c0ce2f160ff27c7c86821441ddfc4b75.zip
chromium_src-7696afc5c0ce2f160ff27c7c86821441ddfc4b75.tar.gz
chromium_src-7696afc5c0ce2f160ff27c7c86821441ddfc4b75.tar.bz2
Revert of Don't set layer needsRepaint in LayoutObject::markContainerChainForPaintInvalidation() (patchset #3 id:40001 of https://codereview.chromium.org/1499263002/ )
Reason for revert: This makes the following layout test flaky: fast/reflections/reflection-masks-opacity.html The reflection is missing. I repro'd locally on Linux and verified that reverting fixed the bug, using: run-webkit-tests --no-retry-failures --iterations=10 fast/reflections/reflection-masks-opacity.html Original issue's description: > Don't set layer needsRepaint in LayoutObject::markContainerChainForPaintInvalidation() > > This was added for integration of invalidation and paint, but caused unnecessary > paint invalidation and paint when layout is executed but nothing changed and needs > invalidation. > > We need to rethink how to integrate invalidation and paint for spv2 when some object > may need paint invalidation. > > BUG=563496 > > Committed: https://crrev.com/1a7d76e9ee86ba96c412cc6937433aacb6a82909 > Cr-Commit-Position: refs/heads/master@{#363732} TBR=chrishtr@chromium.org,wangxianzhu@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=563496 Review URL: https://codereview.chromium.org/1510793002 Cr-Commit-Position: refs/heads/master@{#363755}
Diffstat (limited to 'courgette')
0 files changed, 0 insertions, 0 deletions