Commit f0f7f730 authored by Vasiliy Telezhnikov's avatar Vasiliy Telezhnikov Committed by Commit Bot

Don't use WrappedSkImageFactory for WebView.

WrappedSkImageFactory doesn't support multithreading and so can't be
used if shared image is used if the Display context is on another
thread.

Change-Id: Iac39c4657ab968b9c729c6ba29e3ac985e36a4cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2013809
Auto-Submit: Vasiliy Telezhnikov <vasilyt@chromium.org>
Reviewed-by: default avatarJonathan Backer <backer@chromium.org>
Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#734483}
parent 57ec7405
......@@ -368,9 +368,9 @@ SharedImageBackingFactory* SharedImageFactory::GetFactoryByUsage(
constexpr auto kWrappedSkImageUsage = SHARED_IMAGE_USAGE_RASTER |
SHARED_IMAGE_USAGE_OOP_RASTERIZATION |
SHARED_IMAGE_USAGE_DISPLAY;
bool using_wrapped_sk_image = wrapped_sk_image_factory_ &&
(usage == kWrappedSkImageUsage) &&
!using_interop_factory;
bool using_wrapped_sk_image =
wrapped_sk_image_factory_ && (usage == kWrappedSkImageUsage) &&
!using_interop_factory && !share_between_threads;
using_interop_factory |= vulkan_usage && !using_wrapped_sk_image;
if (gmb_type != gfx::EMPTY_BUFFER) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment