New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 6 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 644851
issue 914491



Sign in to add a comment
link

Issue 795132: cc: Skia renderer needs color conversion support

Reported by ccameron@chromium.org, Dec 14 2017 Project Member

Issue description

Color color conversion happens at two steps in the display pipeline
Place 1: Raster time
  Here we try to raster into whatever the "right space" is. This can be the output display space, or sRGB (for canvas), etc.
Place 2: Display time
  Here we convert whatever inputs we have -- video, tiles, canvases, etc, into the display's color space. This is only done for GPU compositing.

For software compositing, color conversion at composite time is too slow to contemplate, so we don't do it. We raster everything into sRGB (see LayerTreeHostImpl::GetRasterColorSpace), and then display everything as sRGB.

Skia currently has 2 color conversion APIs:
API 1: Set a SkColorSpace on your output SkSurfaces, input SkImages, and input/output SkBitmaps, etc, and color conversion is done **on the fly** everywhere. This also opts you into linear blending, which is not compatible with the web.
API 2: Wrap all of your canvases in a SkColorSpaceXformCanvas, and tag your input SkImages only. This does not opt you into linear blending, but it does not do conversion on the fly -- it calls SkImage::makeColorSpace on all input images, making a color-converted copy of the images that weren't already in the destination space.

We currently use API 2 for rasterization, and we (at least try to) make all SkImages have the same color space as the destination, so that no color-conversion-copies are made. Making color-conversion-copies of SkImages is expensive, and whenever we don't succeed in pre-converting all images, performance bugs appear.

Using API 1 for compositing is not possible, because linear blending is not compatible with existing web content.

Using API 2 for compositing is not possible, because we will always have input images that are not in the output color space (e.g, canvas is sRGB, video is lots of things).

Ultimately we will want an API which allows for
- on-the-fly (in-shader) conversion
- without linear blending
This may be similar to the idea of a kNonLinearBlending_ColorSpaceFlag that was pursued in  issue 697675 , but was abandoned in favor of the SkColorSpaceXformCanvas approach due to complexity of the implementation.

Another issue that may need to be addressed is that there are also color spaces (HDR video, for example, or YUV video) that do not have a SkColorSpace representation.
 

Comment 1 by vmi...@chromium.org, Dec 15 2017

Cc: brianosman@chromium.org
It sounds like brianosman is looking to make color correction with non-linear blending a 1st class citizen and replacing SkColorSpaceXformCanvas.

Comment 2 by ccameron@chromium.org, Dec 15 2017

That's good news! We abandoned kNonLinearBlending_ColorSpaceFlag because it was incurring too much complexity, but getting all of the edge cases covered in SkColorSpaceXformCanvas is turning out to be no picnic either.

Comment 3 by schenney@chromium.org, Dec 15 2017

Components: -Blink>Compositing Internals>Compositing
Correct me if this is in fact a Blink Compositing issue.

Comment 4 by ccameron@chromium.org, Dec 15 2017

Btw, Chrome's compositor supports arbitrary 3D LUTs for color spaces, however, according to UMAs, this is overkill.

On Windows/Stable/63.0.3239.84,
- 88.64% of users have sRGB color spaces
- 4.54% have exact SkMatrix44-and-SkColorSpaceTransferFn
- 4.06% can be approximated to <1/255 by SkMatrix44-and-SkColorSpaceTransferFn
- 0.21% need a 3D LUT
- 1.99% fail to parse entirely

I'd advocate
- SkMatrix44-and-SkColorSpaceTransferFn is probably enough
- SkMatrix44-and-per-channel SkColorSpaceTransferFn is almost certainly enough
- HDR on Windows will require extended-sRGB support
- HDR on CrOS may require some other strange functions

Comment 5 by brianosman@chromium.org, Dec 15 2017

Cc: mtklein@chromium.org
Just adding mtklein@, who is also working on replacing SkColorSpaceXformCanvas.

Comment 6 by rjkroege@chromium.org, Feb 12 2018

Cc: penghuang@chromium.org

Comment 7 by weiliangc@chromium.org, Dec 12

Owner: weiliangc@chromium.org
Status: Started (was: Available)
Looking at color conversion right now by sprinkling color space every where in code...

Comment 8 by backer@chromium.org, Dec 12

Blocking: 914491

Comment 9 by weiliangc@chromium.org, Dec 12

For my current CL, I noticed that not all color space in gfx is supported by SkColorSpace https://cs.chromium.org/chromium/src/ui/gfx/color_space.cc?sq=package:chromium&g=0&l=447

I am wondering what needs to be done in those cases.

Comment 10 by bugdroid1@chromium.org, Dec 15

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ed44fc45d0c3b3675efec4c550d28c5de1569c9d

commit ed44fc45d0c3b3675efec4c550d28c5de1569c9d
Author: Yuri Wiitala <miu@chromium.org>
Date: Sat Dec 15 03:14:24 2018

Complete the screen capture color space plumbing.

Adds all remaining "plumbing" of color space information, throughout the
CopyOutputRequest execution pipeline and the FrameSinkVideoCapturer
pipeline. This ensures the color space being used to draw the original
RenderPass within the compositor is being specified in the metadata for
all result images.

DevTools: Remove a hack from the color picker tool, now that the color
space information for the screen capture frame is being provided.

browser_tests/content_browsertests changes: Multilple browser tests were
fixed as a result of this change revealing pre-existing bugs in the
tests: the web page layout of the color regions, how pixels were being
selected for analysis, and YUV→RGB color space conversion inaccuracies.

Blink layout tests: Rebased a number of layout test expectations, as the
the layout tests utilize the screen capture pipeline. I examined all of
these changed expectations to confirm near 0% change: Meaning that just
a tiny number of pixels in an image were imperceivably different because
of adding the missing color space info.

Bug: 758057, 8510131,  809835 , 863103, 884170, 795132
Change-Id: I11056ddc4f501ee338dc3283397703747a395571
Reviewed-on: https://chromium-review.googlesource.com/c/1372894
Commit-Queue: Yuri Wiitala <miu@chromium.org>
Reviewed-by: Jamie Walch <jamiewalch@chromium.org>
Reviewed-by: Ria Jiang <riajiang@chromium.org>
Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616936}
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/chrome/browser/devtools/devtools_eye_dropper.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/chrome/browser/extensions/api/cast_streaming/cast_streaming_apitest.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/chrome/browser/extensions/api/tab_capture/tab_capture_apitest.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/chrome/test/data/extensions/api_test/tab_capture/offscreen_end_to_end.js
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/common/frame_sinks/copy_output_result.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/common/frame_sinks/copy_output_result.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display/gl_renderer_copier.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display/skia_output_surface.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display/skia_renderer.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display/software_renderer.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display_embedder/skia_output_surface_impl.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display_embedder/skia_output_surface_impl.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/frame_sinks/video_capture/frame_sink_video_capturer_impl.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/frame_sinks/video_capture/interprocess_frame_pool.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/frame_sinks/video_capture/interprocess_frame_pool.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/frame_sinks/video_capture/video_capture_overlay.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/frame_sinks/video_capture/video_capture_overlay.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/components/viz/service/frame_sinks/video_capture/video_capture_overlay_unittest.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/aura_window_video_capture_device_browsertest.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/content_capture_device_browsertest_base.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/fake_video_capture_stack.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/frame_test_util.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/frame_test_util.h
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/lame_window_capturer_chromeos.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/content/browser/media/capture/web_contents_video_capture_device_browsertest.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/remoting/host/chromeos/aura_desktop_capturer_unittest.cc
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/editing/selection/4402375-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/editing/selection/replaced-boundaries-1-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/editing/selection/replaced-boundaries-2-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/editing/selection/replaced-boundaries-3-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/editing/selection/transformed-selection-rects-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/paint/invalidation/svg/text-selection-text-05-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/foreignObject-text-clipping-bug-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/selection-doubleclick-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/selection-tripleclick-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-align-02-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-align-03-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-align-04-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-align-06-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-intro-05-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-path-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-spacing-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-text-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-text-03-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-text-06-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-text-08-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/text-selection-ws-02-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/linux/svg/text/textPathBoundsBug-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac-mac10.10/editing/selection/replaced-boundaries-3-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac-mac10.11/editing/selection/replaced-boundaries-3-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/editing/selection/4402375-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/editing/selection/replaced-boundaries-1-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/editing/selection/replaced-boundaries-2-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/editing/selection/replaced-boundaries-3-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/editing/selection/transformed-selection-rects-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/paint/invalidation/svg/text-selection-text-05-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/foreignObject-text-clipping-bug-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/selection-doubleclick-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/selection-tripleclick-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-align-03-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-align-04-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-align-06-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-path-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-spacing-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-text-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-text-03-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-text-04-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-text-06-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-text-08-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-tselect-02-f-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-tspan-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-ws-01-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/text-selection-ws-02-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/mac/svg/text/textPathBoundsBug-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/editing/selection/4402375-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/editing/selection/replaced-boundaries-1-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/editing/selection/replaced-boundaries-2-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/editing/selection/replaced-boundaries-3-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/editing/selection/transformed-selection-rects-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/paint/invalidation/svg/text-selection-text-05-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/foreignObject-text-clipping-bug-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/selection-tripleclick-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-align-02-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-align-03-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-align-04-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-align-05-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-align-06-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-path-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-spacing-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-text-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-text-03-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-text-06-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-text-08-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-tselect-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-tselect-02-f-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-tspan-01-b-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-ws-01-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/text-selection-ws-02-t-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win/svg/text/textPathBoundsBug-expected.png
[delete] https://crrev.com/4c40f61f89ae4d0a196c2c7b14b696bb7b5e0430/third_party/blink/web_tests/platform/win7/editing/selection/replaced-boundaries-1-expected.png
[modify] https://crrev.com/ed44fc45d0c3b3675efec4c550d28c5de1569c9d/third_party/blink/web_tests/platform/win7/svg/text/text-selection-text-08-b-expected.png

Comment 11 by bugdroid1@chromium.org, Dec 17

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/98a8f01a27f97ec27e6315fb006e61cd59192d29

commit 98a8f01a27f97ec27e6315fb006e61cd59192d29
Author: Morten Stenshorne <mstensho@chromium.org>
Date: Mon Dec 17 11:41:19 2018

[LayoutNG] Rebaseline test expectations after color space changes.

https://chromium-review.googlesource.com/c/chromium/src/+/1372894 made
some invisible (to the naked eye) rebaselining to some test
expectations, but not for LayoutNG.

TBR=kojii@chromium.org

Bug: 758057, 8510131,  809835 , 863103, 884170, 795132
Change-Id: Ie6b626c66c0930b1c8c00b7b01a0f01ddd06dada
Reviewed-on: https://chromium-review.googlesource.com/c/1379953
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#617089}
[modify] https://crrev.com/98a8f01a27f97ec27e6315fb006e61cd59192d29/third_party/blink/web_tests/FlagExpectations/enable-blink-features=LayoutNG
[modify] https://crrev.com/98a8f01a27f97ec27e6315fb006e61cd59192d29/third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/editing/selection/4402375-expected.png
[modify] https://crrev.com/98a8f01a27f97ec27e6315fb006e61cd59192d29/third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/editing/selection/transformed-selection-rects-expected.png
[modify] https://crrev.com/98a8f01a27f97ec27e6315fb006e61cd59192d29/third_party/blink/web_tests/flag-specific/enable-blink-features=LayoutNG/svg/text/foreignObject-text-clipping-bug-expected.png

Comment 12 by bugdroid, Jan 27

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/86a74be8804c9ac70c3e5814ecb96181cf4dfb09

commit 86a74be8804c9ac70c3e5814ecb96181cf4dfb09
Author: Weiliang Chen <weiliangc@chromium.org>
Date: Sun Jan 27 23:22:30 2019

viz: Add color space information to SkiaRenderer

Take into account resource's color space information in SkiaRenderer so
color correction gets applied. This is done by plumbing through
color space information.

Since SkColorSpace does not cover everything in gfx::ColorSpace, this
patch does not pass all the color transformation pixel test. Further
investigation is needed for the failure cases.

Bug: 795132
Change-Id: I1712e4625a055fd93af7702509d167e33667671d
Reviewed-on: https://chromium-review.googlesource.com/c/1374460
Commit-Queue: weiliangc <weiliangc@chromium.org>
Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Cr-Commit-Position: refs/heads/master@{#626416}

Comment 13 by bugdroid, Jan 30

Project Member
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cb5797a63f17a4b321179d18f169846ebae99eb9

commit cb5797a63f17a4b321179d18f169846ebae99eb9
Author: Peng Huang <penghuang@chromium.org>
Date: Wed Jan 30 21:19:58 2019

Revert part of https://crrev.com/c/1374460

CL 1374460 breaks window resizing. It doesn't create surface's
characterization correctly. So after resizing, all ddl recored with
wrong characterization can not be drawn on the framebuffer successfully.

Bug: 795132
Change-Id: I4bec6bcd3b990bcc260598a7e38c7b9703391ea1
Reviewed-on: https://chromium-review.googlesource.com/c/1446502
Reviewed-by: weiliangc <weiliangc@chromium.org>
Commit-Queue: Peng Huang <penghuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#627589}
[modify] https://crrev.com/cb5797a63f17a4b321179d18f169846ebae99eb9/components/viz/service/display_embedder/skia_output_surface_impl.cc

Sign in to add a comment