Use DlRegionSorted as the algorithm

Author: knopp

display_list/benchmarking/dl_region_benchmarks.cc

@@ -27,26 +27,12 @@ class SkRegionAdapter {
   SkRegion region_;
 };
 
-class DlRegion2Adapter {
+class DlRegionAdapter {
  public:
   void addRect(const SkIRect& rect) { rects_.push_back(rect); }
 
   std::vector<SkIRect> getRects() {
-    flutter::DlRegion2 region;
-    region.addRects(std::move(rects_));
-    return region.getRects(false);
-  }
-
- private:
-  std::vector<SkIRect> rects_;
-};
-
-class DlRegionSortedAdapter {
- public:
-  void addRect(const SkIRect& rect) { rects_.push_back(rect); }
-
-  std::vector<SkIRect> getRects() {
-    flutter::DlRegionSorted region;
+    flutter::DlRegion region;
     region.addRects(std::move(rects_));
     return region.getRects(false);
   }
@@ -84,51 +70,23 @@ static void BM_RegionBenchmarkSkRegion(benchmark::State& state, int maxSize) {
 }
 
 static void BM_RegionBenchmarkDlRegion(benchmark::State& state, int maxSize) {
-  RunRegionBenchmark<DlRegion>(state, maxSize);
-}
-
-static void BM_RegionBenchmarkDlRegionSorted(benchmark::State& state,
-                                             int maxSize) {
-  RunRegionBenchmark<DlRegionSortedAdapter>(state, maxSize);
-}
-
-static void BM_RegionBenchmarkDlRegion2(benchmark::State& state, int maxSize) {
-  RunRegionBenchmark<DlRegion2Adapter>(state, maxSize);
+  RunRegionBenchmark<DlRegionAdapter>(state, maxSize);
 }
 
 BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion, Tiny, 30)
     ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion2, Tiny, 30)
-    ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegionSorted, Tiny, 30)
-    ->Unit(benchmark::kMicrosecond);
 BENCHMARK_CAPTURE(BM_RegionBenchmarkSkRegion, Tiny, 30)
     ->Unit(benchmark::kMicrosecond);
-
 BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion, Small, 100)
     ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion2, Small, 100)
-    ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegionSorted, Small, 100)
-    ->Unit(benchmark::kMicrosecond);
 BENCHMARK_CAPTURE(BM_RegionBenchmarkSkRegion, Small, 100)
     ->Unit(benchmark::kMicrosecond);
-
 BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion, Medium, 400)
     ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion2, Medium, 400)
-    ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegionSorted, Medium, 400)
-    ->Unit(benchmark::kMicrosecond);
 BENCHMARK_CAPTURE(BM_RegionBenchmarkSkRegion, Medium, 400)
     ->Unit(benchmark::kMicrosecond);
-
 BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion, Large, 1500)
     ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegion2, Large, 1500)
-    ->Unit(benchmark::kMicrosecond);
-BENCHMARK_CAPTURE(BM_RegionBenchmarkDlRegionSorted, Large, 1500)
-    ->Unit(benchmark::kMicrosecond);
 BENCHMARK_CAPTURE(BM_RegionBenchmarkSkRegion, Large, 1500)
     ->Unit(benchmark::kMicrosecond);