Revert "Always try to reduce opList splitting in DDL contexts/drawingManagers"
This reverts commit 2cce805f1f8216ebdc56b247389d8aa05a51df11.
Reason for revert: Chrome
Original change's description:
> Always try to reduce opList splitting in DDL contexts/drawingManagers
>
> This may get us in trouble w/ local DDL testing (since we run all our GMs through DDLs). For Chrome this shouldn't yet be a problem (since they are only using DDLs for compositing).
>
> This does mean we're on a tight timeline to land predictive intermediate flushes before Chrome starts using DDLs for rasterization.
>
> Change-Id: I0bb95c075cff3ee49498ff267d76c3a61d16373e
> Reviewed-on: https://skia-review.googlesource.com/c/skia/+/199722
> Reviewed-by: Brian Salomon <bsalomon@google.com>
> Commit-Queue: Robert Phillips <robertphillips@google.com>
TBR=bsalomon@google.com,robertphillips@google.com
Change-Id: Idb2dbda1a41844b2541526d504b117fd4cd628cc
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/200505
Reviewed-by: Robert Phillips <robertphillips@google.com>
Commit-Queue: Robert Phillips <robertphillips@google.com>
diff --git a/src/gpu/GrLegacyDirectContext.cpp b/src/gpu/GrLegacyDirectContext.cpp
index 7a5fad0..d5bce1b 100644
--- a/src/gpu/GrLegacyDirectContext.cpp
+++ b/src/gpu/GrLegacyDirectContext.cpp
@@ -86,18 +86,7 @@
sortOpLists = true;
}
- // For now, this is only turned on for direct rendering when explicitly enabled.
- // Once predictive intermediate flushes are implemented it should be enabled whenever
- // sorting is enabled.
- bool reduceOpListSplitting = false; // sortOpLists
- if (GrContextOptions::Enable::kNo == this->options().fReduceOpListSplitting) {
- reduceOpListSplitting = false;
- } else if (GrContextOptions::Enable::kYes == this->options().fReduceOpListSplitting) {
- reduceOpListSplitting = true;
- }
-
- this->setupDrawingManager(this->explicitlyAllocateGPUResources(),
- sortOpLists, reduceOpListSplitting);
+ this->setupDrawingManager(this->explicitlyAllocateGPUResources(), sortOpLists);
SkASSERT(this->caps());