The CQ bit was checked by timloh@chromium.org
Created Revert of Skip paint property update and visual rect update if no geometry change
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 1, "attempt_start_ts": 1491183741457080, "parent_rev": "542dc4c9f70f7c9e1a3ec096c6b62bceceb5e47f", "commit_rev": "6d954094c746673d37970ab1321e810622c8fd49"}
Description was changed from ========== Revert of Skip paint property update and visual rect update if no geometry change (patchset #25 id:480001 of https://codereview.chromium.org/2732573003/ ) Reason for revert: Looks like this is making tests fail on most builders, e.g. https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty... Original issue's description: > Skip paint property update and visual rect update if no geometry change > > This avoids unnecessary visual rect computation and update of paint > offset related paint properties. > > BUG=697081,685179 > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 > > Review-Url: https://codereview.chromium.org/2732573003 > Cr-Commit-Position: refs/heads/master@{#461348} > Committed: https://chromium.googlesource.com/chromium/src/+/81f28ed19f771fe93406496d1d23... TBR=pdr@chromium.org,chrishtr@chromium.org,wangxianzhu@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=697081,685179 ========== to ========== Revert of Skip paint property update and visual rect update if no geometry change (patchset #25 id:480001 of https://codereview.chromium.org/2732573003/ ) Reason for revert: Looks like this is making tests fail on most builders, e.g. https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty... Original issue's description: > Skip paint property update and visual rect update if no geometry change > > This avoids unnecessary visual rect computation and update of paint > offset related paint properties. > > BUG=697081,685179 > CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 > > Review-Url: https://codereview.chromium.org/2732573003 > Cr-Commit-Position: refs/heads/master@{#461348} > Committed: https://chromium.googlesource.com/chromium/src/+/81f28ed19f771fe93406496d1d23... TBR=pdr@chromium.org,chrishtr@chromium.org,wangxianzhu@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=697081,685179 Review-Url: https://codereview.chromium.org/2792863002 Cr-Commit-Position: refs/heads/master@{#461357} Committed: https://chromium.googlesource.com/chromium/src/+/6d954094c746673d37970ab1321e... ==========
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/src/+/6d954094c746673d37970ab1321e...
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2792063002/ by wangxianzhu@chromium.org. The reason for reverting is: (Not for landing yet. Just create a reland CL.).
Chromium Code Reviews
Issue
2792863002:
Revert of Skip paint property update and visual rect update if no geometry change (Closed)
