diff options
| author | Akhil Jaiswal <quic_akhijais@quicinc.com> | 2023-09-02 11:03:16 +0530 |
|---|---|---|
| committer | Murtuza Raja <quic_mraja@quicinc.com> | 2023-09-11 23:44:08 +0530 |
| commit | ca7349a398d992d359e7b05f5a15cd5a3e37e6bb (patch) | |
| tree | e84f5d6e0ac22fff5e4e51ea2d6b825fc9be7cf0 | |
| parent | 331b5c932e560f521151ebe7d9239addcfb1d729 (diff) | |
RenderEngine:don't blur when using protected content
Bug: 270314344
Bug: 213116783
Test: manual (ExoPlayer)
Change-Id: Ifcacbd4c44aee1a504a7c46f3fece51004ed7f37
CRs-Fixed: 3605143
| -rw-r--r-- | libs/renderengine/skia/SkiaGLRenderEngine.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libs/renderengine/skia/SkiaGLRenderEngine.cpp b/libs/renderengine/skia/SkiaGLRenderEngine.cpp index 0caa9f2fbd..bda344888e 100644 --- a/libs/renderengine/skia/SkiaGLRenderEngine.cpp +++ b/libs/renderengine/skia/SkiaGLRenderEngine.cpp @@ -836,7 +836,8 @@ void SkiaGLRenderEngine::drawLayersInternal( SkCanvas* canvas = dstCanvas; SkiaCapture::OffscreenState offscreenCaptureState; const LayerSettings* blurCompositionLayer = nullptr; - if (mBlurFilter) { + // TODO (b/270314344): Enable blurs in protected context. + if (mBlurFilter && !mInProtectedContext) { bool requiresCompositionLayer = false; for (const auto& layer : layers) { // if the layer doesn't have blur or it is not visible then continue @@ -930,7 +931,8 @@ void SkiaGLRenderEngine::drawLayersInternal( const auto [bounds, roundRectClip] = getBoundsAndClip(layer.geometry.boundaries, layer.geometry.roundedCornersCrop, layer.geometry.roundedCornersRadius); - if (mBlurFilter && layerHasBlur(layer, ctModifiesAlpha)) { + // TODO (b/270314344): Enable blurs in protected context. + if (mBlurFilter && layerHasBlur(layer, ctModifiesAlpha) && !mInProtectedContext) { std::unordered_map<uint32_t, sk_sp<SkImage>> cachedBlurs; // if multiple layers have blur, then we need to take a snapshot now because |