Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit fbc887d

Browse files
reed-at-googleSkia Commit-Bot
authored andcommitted
Revert "Remove (unnecessary?) flush in RenderYUVAToRGBA"
This reverts commit 49b293a. Reason for revert: speculative to fix chrome roll Original change's description: > Remove (unnecessary?) flush in RenderYUVAToRGBA > > I don't think we need a flush here. > > Change-Id: I87bf41b491835578e7df9bea4e15ea24dd551c1e > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/196645 > Reviewed-by: Greg Daniel <[email protected]> > Commit-Queue: Robert Phillips <[email protected]> [email protected],[email protected],[email protected] Change-Id: I49305a5a2bcd5fe112b5f55b64511fd05d6c8548 No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://skia-review.googlesource.com/c/skia/+/197305 Reviewed-by: Mike Reed <[email protected]> Commit-Queue: Mike Reed <[email protected]> Auto-Submit: Mike Reed <[email protected]>
1 parent 0a57971 commit fbc887d

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/image/SkImage_GpuBase.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -361,6 +361,10 @@ bool SkImage_GpuBase::RenderYUVAToRGBA(GrContext* ctx, GrRenderTargetContext* re
361361
paint.addColorFragmentProcessor(std::move(fp));
362362

363363
renderTargetContext->drawRect(GrNoClip(), std::move(paint), GrAA::kNo, SkMatrix::I(), rect);
364+
365+
// DDL TODO: in the promise image version we must not flush here
366+
ctx->priv().flushSurfaceWrites(renderTargetContext->asSurfaceProxy());
367+
364368
return true;
365369
}
366370

0 commit comments

Comments
 (0)