|
|
Chromium Code Reviews
DescriptionUse more accurate sRGB, Adobe, matrices
TBR=fmalita@chromium.org
BUG=657839
Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c
Committed: https://crrev.com/7bf60df60dea0a5d295549957e23381898e0af33
Cr-Original-Commit-Position: refs/heads/master@{#439209}
Cr-Commit-Position: refs/heads/master@{#440253}
Patch Set 1 #Patch Set 2 : Add rebaselines #Patch Set 3 : Remove conflict #Patch Set 4 : More rebaselines #Patch Set 5 : More rebaselines #Messages
Total messages: 49 (33 generated)
Description was changed from ========== Use more accurate sRGB, Adobe, matrices BUG= ========== to ========== Use more accurate sRGB, Adobe, matrices BUG=657839 ==========
The CQ bit was checked by msarett@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
msarett@chromium.org changed reviewers: + brianosman@google.com, fmalita@chromium.org
Just doing a dry run so I can find all the tests that I need to rebaseline :).
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
Ping. Looks like I don't need to rebaseline :).
lgtm Nice.
The CQ bit was checked by msarett@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full committers or once the patch has received an L-G-T-M from a full committer. Even if an L-G-T-M may have been provided, it was from a non-committer, _not_ a full super star committer. See http://www.chromium.org/getting-involved/become-a-committer Note that this has nothing to do with OWNERS files.
Description was changed from ========== Use more accurate sRGB, Adobe, matrices BUG=657839 ========== to ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 ==========
The CQ bit was checked by msarett@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full committers or once the patch has received an L-G-T-M from a full committer. Even if an L-G-T-M may have been provided, it was from a non-committer, _not_ a full super star committer. See http://www.chromium.org/getting-involved/become-a-committer Note that this has nothing to do with OWNERS files.
On 2016/12/16 21:52:21, commit-bot: I haz the power wrote: > No L-G-T-M from a valid reviewer yet. > CQ run can only be started by full committers or once the patch has > received an L-G-T-M from a full committer. > Even if an L-G-T-M may have been provided, it was from a non-committer, > _not_ a full super star committer. > See http://www.chromium.org/getting-involved/become-a-committer > Note that this has nothing to do with OWNERS files. lgtm
The CQ bit was checked by msarett@chromium.org
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": 1481925194977880, "parent_rev":
"0d03ad6eabf782fa71f2932adf66a670e736ef05", "commit_rev":
"e831fff1aa84b6d23b48083c5d8c8ef65801dbf4"}
Message was sent while issue was closed.
Description was changed from ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 ========== to ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Review-Url: https://codereview.chromium.org/2583543003 ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Description was changed from ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Review-Url: https://codereview.chromium.org/2583543003 ========== to ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209} ==========
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209}
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2580153002/ by rjkroege@chromium.org. The reason for reverting is: Best guess at fixing many colour related layout test failures (mostly image but also text) after this CL landed. Test failures (across multiple WebKit bots): virtual/gpu-rasterization/images/embed-image.html css2.1/20110323/inline-replaced-height-002.htm ietestcenter/css3/bordersbackgrounds/background-color-border-box.htm css3/filters/effect-invert-hw.html virtual/gpu-rasterization/images/color-profile-image-profile-match.html virtual/display_list_2d_canvas/fast/canvas/canvas-toBlob-jpeg-medium-quality.html css3/blending/background-blend-mode-tiled-gradient.html css2.1/20110323/absolute-replaced-height-017.htm virtual/layout_ng/css2.1/20110323/inline-replaced-width-006.htm css2.1/20110323/absolute-replaced-height-022.htm css2.1/20110323/float-replaced-width-006.htm virtual/spinvalidation/paint/invalidation/compositing/page-scale-repaint.html virtual/gpu-rasterization/images/paletted-png-with-color-profile.html css3/blending/mix-blend-mode-isolated-group-1.html images/paletted-png-with-color-profile.html virtual/layout_ng/css2.1/20110323/block-replaced-height-001.htm fast/reflections/reflection-masks-opacity.html css3/blending/background-blend-mode-single-layer-no-blending.html css2.1/20110323/inline-block-replaced-height-001.htm css2.1/20110323/absolute-replaced-width-069.htm css3/blending/background-blend-mode-multiple-background-layers.html css2.1/20110323/absolute-replaced-height-023.htm css3/filters/effect-invert.html css2.1/20110323/block-replaced-height-003.htm css3/filters/effect-contrast.html css3/filters/effect-brightness.html images/color-profile-border-image-source.html css2.1/20110323/absolute-replaced-height-029.htm mhtml/invalid-bad-boundary2.mht css3/filters/effect-brightness-clamping-hw.html virtual/layout_ng/css2.1/20110323/inline-block-replaced-width-006.htm virtual/layout_ng/css2.1/20110323/inline-block-replaced-width-001.htm css2.1/20110323/absolute-replaced-height-031.htm css2.1/20110323/float-replaced-width-005.htm virtual/spinvalidation/paint/invalidation/background-image-paint-invalidation-small-document.html css2.1/20110323/float-replaced-height-001.htm css2.1/20110323/float-replaced-height-002.htm fast/canvas/canvas-toDataURL-webp.html css3/blending/effect-background-blend-mode.html css2.1/20110323/inline-replaced-height-003.htm virtual/layout_ng/css2.1/20110323/inline-block-replaced-height-001.htm css2.1/20110323/absolute-replaced-width-041.htm ietestcenter/css3/bordersbackgrounds/background_color_padding_box.htm fast/reflections/reflection-masks.html css2.1/20110323/block-replaced-height-002.htm virtual/gpu-rasterization/images/color-profile-border-image-source.html virtual/gpu/fast/canvas/canvas-toBlob-jpeg-medium-quality.html css3/filters/effect-blur-hw.html css3/filters/effect-combined.html css2.1/20110323/absolute-replaced-width-050.htm css3/filters/effect-brightness-clamping.html ietestcenter/css3/bordersbackgrounds/background_repeat_space_content_box.htm ietestcenter/css3/bordersbackgrounds/background-attachment-local-scrolling.htm fast/canvas/canvas-toBlob-jpeg-maximum-quality.html svg/as-border-image/svg-as-border-image-2.html css3/blending/background-blend-mode-image-image.html css2.1/20110323/absolute-replaced-width-029.htm css3/blending/background-blend-mode-separate-layer-declaration.html css2.1/20110323/float-replaced-width-001.htm css2.1/20110323/absolute-replaced-height-008.htm css2.1/20110323/float-replaced-width-004.htm fast/reflections/reflection-with-zoom.html css2.1/20110323/absolute-replaced-width-076.htm svg/W3C-SVG-1.1/struct-image-04-t.svg css2.1/20110323/float-replaced-width-003.htm css3/filters/effect-drop-shadow.html css2.1/20110323/absolute-replaced-height-009.htm css3/blending/mix-blend-mode-isolated-group-2.html css3/filters/effect-hue-rotate-hw.html css3/blending/background-blend-mode-opaque-layer.html virtual/spinvalidation/paint/invalidation/background-image-paint-invalidation-large-abspos-div.html css2.1/20110323/absolute-replaced-height-003.htm css2.1/20110323/inline-block-replaced-height-002.htm css2.1/20110323/absolute-replaced-width-022.htm css3/filters/effect-grayscale-hw.html css2.1/20110323/absolute-replaced-width-062.htm css3/blending/effect-background-blend-mode-stacking.html virtual/display_list_2d_canvas/fast/canvas/canvas-toBlob-webp-maximum-quality.html css2.1/20110323/absolute-replaced-width-055.htm virtual/display_list_2d_canvas/fast/canvas/canvas-toBlob-jpeg-maximum-quality.html fast/canvas/canvas-toBlob-webp-maximum-quality.html css2.1/20110323/absolute-replaced-width-043.htm fast/borders/border-image-omit-right-slice.html ietestcenter/css3/bordersbackgrounds/none-as-image-layer.htm virtual/display_list_2d_canvas/fast/canvas/canvas-toDataURL-jpeg-maximum-quality.html css3/filters/effect-contrast-hw.html css3/filters/effect-saturate-hw.html css3/filters/composited-reflected.html compositing/reflections/simple-composited-reflections.html virtual/gpu/fast/canvas/canvas-toDataURL-webp.html images/embed-image.html virtual/gpu/fast/canvas/canvas-toBlob-jpeg-maximum-quality.html images/object-image.html css2.1/20110323/absolute-replaced-width-048.htm css3/blending/background-blend-mode-image-svg.html css2.1/20110323/absolute-replaced-width-008.htm images/paint-subrect.html tables/mozilla/bugs/bug82946-2.html css2.1/20110323/inline-block-replaced-width-006.htm css3/filters/effect-sepia.html ietestcenter/css3/bordersbackgrounds/color-behind-images.htm virtual/gpu-rasterization/images/object-image.html images/png-color-profile-ignore-gamma.html mhtml/image_document.mht fast/css/background-shorthand-invalid-url.html css3/blending/background-blend-mode-different-image-formats.html css2.1/20110323/float-replaced-width-011.htm css2.1/20110323/absolute-replaced-height-030.htm css2.1/20110323/absolute-replaced-width-020.htm virtual/display_list_2d_canvas/fast/canvas/canvas-toDataURL-webp.html virtual/gpu-rasterization/images/color-profile-munsell-adobe-to-srgb.html fast/inline/inline-box-background-long-image.html css2.1/20110323/block-replaced-width-006.htm css2.1/20110323/absolute-replaced-width-064.htm css2.1/20110323/absolute-replaced-width-034.htm css2.1/20110323/absolute-replaced-height-016.htm virtual/spinvalidation/paint/invalidation/svg/scrolling-embedded-svg-file-image-repaint-problem.html transforms/transformed-document-element.html css3/filters/effect-brightness-hw.html css3/filters/effect-saturate.html css3/filters/effect-hue-rotate.html css3/filters/effect-opacity-hw.html css2.1/20110323/absolute-replaced-height-010.htm css3/filters/effect-drop-shadow-hw.html virtual/display_list_2d_canvas/fast/canvas/canvas-toDataURL-webp-maximum-quality.html virtual/gpu/fast/canvas/canvas-toDataURL-webp-maximum-quality.html virtual/gpu-rasterization/images/paint-subrect.html css2.1/20110323/absolute-replaced-height-002.htm virtual/gpu/fast/canvas/canvas-toDataURL-jpeg-maximum-quality.html fast/canvas/canvas-toBlob-jpeg-medium-quality.html images/color-profile-image-profile-match.html fast/reflections/reflection-masks-outset.html css3/filters/regions-expanding.html virtual/layout_ng/css2.1/20110323/float-replaced-width-003.htm virtual/layout_ng/css2.1/20110323/float-replaced-width-001.htm virtual/gpu-rasterization/images/color-profile-munsell-srgb-to-srgb.html fast/canvas/canvas-toDataURL-webp-maximum-quality.html css2.1/20110323/inline-block-replaced-width-001.htm css2.1/20110323/absolute-replaced-width-036.htm css2.1/20110323/absolute-replaced-width-027.htm virtual/gpu/fast/canvas/canvas-toBlob-webp-maximum-quality.html ietestcenter/css3/bordersbackgrounds/order-of-images.htm css2.1/20110323/inline-replaced-width-001.htm css3/filters/crash-filter-change.html paint/invalidation/compositing/page-scale-repaint.html paint/invalidation/background-image-paint-invalidation-small-document.html virtual/layout_ng/css2.1/20110323/inline-replaced-width-001.htm ietestcenter/css3/bordersbackgrounds/background_position_three_four_values.htm css2.1/20110323/absolute-replaced-width-015.htm css3/filters/effect-blur.html css2.1/20110323/inline-replaced-height-001.htm svg/W3C-SVG-1.1/struct-image-08-t.svg css3/filters/effect-grayscale.html fast/canvas/canvas-toDataURL-jpeg-maximum-quality.html virtual/layout_ng/css2.1/20110323/block-replaced-width-001.htm css3/blending/mix-blend-mode-isolated-group-3.html css3/blending/background-blend-mode-default-value.html fast/writing-mode/block-level-images.html css2.1/20110323/absolute-replaced-width-057.htm css2.1/20110323/absolute-replaced-width-006.htm css3/blending/background-blend-mode-image-color.html ietestcenter/css3/bordersbackgrounds/background-size-aspect-ratio.htm fast/reflections/reflection-direction.html css2.1/20110323/absolute-replaced-height-024.htm css2.1/20110323/absolute-replaced-height-001.htm paint/invalidation/background-image-paint-invalidation-large-abspos-div.html virtual/threaded/compositing/visibility/visibility-image-layers.html css3/filters/effect-sepia-hw.html css2.1/20110323/block-replaced-height-001.htm images/color-profile-munsell-srgb-to-srgb.html css3/filters/effect-combined-hw.html virtual/layout_ng/css2.1/20110323/inline-replaced-height-001.htm css2.1/20110323/absolute-replaced-width-071.htm css2.1/20110323/absolute-replaced-width-013.htm css3/filters/effect-opacity.html css2.1/20110323/float-replaced-width-002.htm css3/blending/background-blend-mode-crossfade-image-gradient.html css2.1/20110323/inline-replaced-width-006.htm virtual/layout_ng/css2.1/20110323/block-replaced-width-006.htm css3/blending/background-blend-mode-gradient-image.html css2.1/20110323/inline-block-replaced-height-003.htm compositing/visibility/visibility-image-layers.html images/color-profile-munsell-adobe-to-srgb.html virtual/gpu-rasterization/images/png-color-profile-ignore-gamma.html css2.1/20110323/block-replaced-width-001.htm paint/invalidation/svg/scrolling-embedded-svg-file-image-repaint-problem.html css2.1/20110323/float-replaced-height-003.htm compositing/masks/direct-image-mask.html css3/blending/effect-background-blend-mode-tiled.html css2.1/20110323/absolute-replaced-width-001.htm virtual/layout_ng/css2.1/20110323/float-replaced-width-005.htm css2.1/20110323/width-replaced-element-001.htm.
Message was sent while issue was closed.
Thanks, yeah I was expecting to need to do a lot of rebaselines. Not sure how this passed the commit queue...
Message was sent while issue was closed.
Description was changed from ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209} ========== to ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209} ==========
The CQ bit was checked by msarett@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
The CQ bit was checked by msarett@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was checked by msarett@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was checked by msarett@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: This issue passed the CQ dry run.
The CQ bit was checked by msarett@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from msarett@chromium.org, brianosman@google.com Link to the patchset: https://codereview.chromium.org/2583543003/#ps80001 (title: "More rebaselines")
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": 80001, "attempt_start_ts": 1482360514906410,
"parent_rev": "7b73b9bbf318336f2bb2df0d366805781c239362", "commit_rev":
"250b86794ef20c9462739e615bd230a6a5dbf85b"}
Message was sent while issue was closed.
Description was changed from ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209} ========== to ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209} Review-Url: https://codereview.chromium.org/2583543003 ==========
Message was sent while issue was closed.
Committed patchset #5 (id:80001)
Message was sent while issue was closed.
Description was changed from ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Cr-Commit-Position: refs/heads/master@{#439209} Review-Url: https://codereview.chromium.org/2583543003 ========== to ========== Use more accurate sRGB, Adobe, matrices TBR=fmalita@chromium.org BUG=657839 Committed: https://crrev.com/556786009fd3fdbb395ab09432004a3afbc82b0c Committed: https://crrev.com/7bf60df60dea0a5d295549957e23381898e0af33 Cr-Original-Commit-Position: refs/heads/master@{#439209} Cr-Commit-Position: refs/heads/master@{#440253} ==========
Message was sent while issue was closed.
Patchset 5 (id:??) landed as https://crrev.com/7bf60df60dea0a5d295549957e23381898e0af33 Cr-Commit-Position: refs/heads/master@{#440253} |
