Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
1751993004:
Don't anti-alias clips in the Blink-replay path. (Closed)
|
Created:
4 years, 9 months ago by chrishtr Modified:
4 years, 9 months ago Reviewers:
f(malita) CC:
blink-reviews, blink-reviews-platform-graphics_chromium.org, Rik, chromium-reviews, danakj+watch_chromium.org, dshwang, drott+blinkwatch_chromium.org, krit, jbroman, Justin Novosad, kinuko+watch, pdr+graphicswatchlist_chromium.org, rwlbuis, Stephen Chennney, vmpstr+blinkwatch_chromium.org Base URL:
https://chromium.googlesource.com/chromium/src.git@master Target Ref:
refs/pending/heads/master Project:
chromium Visibility:
Public. |
DescriptionDon't anti-alias clips in the Blink-replay path.
This is a workaround to unblock M49.
BUG=590971
R=fmalita@chromium.org
Committed: https://chromium.googlesource.com/chromium/src/+/8679e8bbd2de5b40e7f9b1c9cfc7585446caa0e6
Patch Set 1 #Patch Set 2 : #Patch Set 3 : #Patch Set 4 : #
Messages
Total messages: 25 (13 generated)
Description was changed from ========== none BUG= ========== to ========== Don't anti-alias clips in the Blink-replay path. This is a workaround to unblock M49. BUG=590971 ==========
chrishtr@chromium.org changed reviewers: + fmalita@chromium.org
It should be noted that this code path only runs for printing or other image-capture code paths. It does not affect the regular paint/raster pipeline.
On 2016/03/01 20:59:45, chrishtr wrote: > It should be noted that this code path only runs for printing or other > image-capture code paths. > It does not affect the regular paint/raster pipeline. Ah, thanks for the clarification, it wasn't immediately obvious. LGTM
The CQ bit was checked by chrishtr@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1751993004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1751993004/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by chrishtr@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from fmalita@chromium.org Link to the patchset: https://codereview.chromium.org/1751993004/#ps40001 (title: " ")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1751993004/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1751993004/40001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...) ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
The CQ bit was checked by chrishtr@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from fmalita@chromium.org Link to the patchset: https://codereview.chromium.org/1751993004/#ps60001 (title: " ")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1751993004/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1751993004/60001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by chrishtr@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1751993004/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1751993004/60001
Message was sent while issue was closed.
Description was changed from ========== Don't anti-alias clips in the Blink-replay path. This is a workaround to unblock M49. BUG=590971 ========== to ========== Don't anti-alias clips in the Blink-replay path. This is a workaround to unblock M49. BUG=590971 R=fmalita@chromium.org Committed: https://crrev.com/8679e8bbd2de5b40e7f9b1c9cfc7585446caa0e6 Cr-Commit-Position: refs/heads/master@{#378635} ==========
Message was sent while issue was closed.
Patchset 4 (id:??) landed as https://crrev.com/8679e8bbd2de5b40e7f9b1c9cfc7585446caa0e6 Cr-Commit-Position: refs/heads/master@{#378635}
Message was sent while issue was closed.
Description was changed from ========== Don't anti-alias clips in the Blink-replay path. This is a workaround to unblock M49. BUG=590971 R=fmalita@chromium.org Committed: https://crrev.com/8679e8bbd2de5b40e7f9b1c9cfc7585446caa0e6 Cr-Commit-Position: refs/heads/master@{#378635} ========== to ========== Don't anti-alias clips in the Blink-replay path. This is a workaround to unblock M49. BUG=590971 R=fmalita@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/8679e8bbd2de5b40e7f9b1c9cfc7... ==========
Message was sent while issue was closed.
Committed patchset #4 (id:60001) manually as 8679e8bbd2de5b40e7f9b1c9cfc7585446caa0e6 (presubmit successful). |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
