rtc: Apply condition to avoid blk_skip for non-screen
This is a follow-up to:
https://aomedia-review.googlesource.com/c/aom/+/174561
Although the artifact was observed for screen content,
the same condition should apply to non-screen.
Stats changed overall small and neutral for video
speeds 7 - 10.
Change-Id: I196f0bf3e51eb717f40135eff2bc718c8566f2c9
diff --git a/av1/encoder/nonrd_opt.c b/av1/encoder/nonrd_opt.c
index 5783ee6..499e994 100644
--- a/av1/encoder/nonrd_opt.c
+++ b/av1/encoder/nonrd_opt.c
@@ -829,11 +829,10 @@
}
pd->dst = *orig_dst;
- // TODO(marpan): This is to remove bad artifact observed under screen.
+ // TODO(marpan): This is to remove bad visual artifact.
// For now avoid use of the blk_skip feature (set in block_yrd),
// unless it's spatially flat or stationary block.
- if (cpi->oxcf.tune_cfg.content == AOM_CONTENT_SCREEN &&
- x->source_variance > 0 &&
+ if (x->source_variance > 0 &&
x->content_state_sb.source_sad_nonrd != kZeroSad)
allow_blk_skip = 0;