Move non-gencode `onDumpInfo` methods to private section.
`onXxxxx` methods in Skia are generally intended to be non-public.
Change-Id: I26b6bdcee653f41a22c91f79d01b20a274210747
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/309981
Auto-Submit: John Stiles <johnstiles@google.com>
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Brian Salomon <bsalomon@google.com>
diff --git a/src/gpu/ops/GrRegionOp.cpp b/src/gpu/ops/GrRegionOp.cpp
index 6a3818a..ec723a2 100644
--- a/src/gpu/ops/GrRegionOp.cpp
+++ b/src/gpu/ops/GrRegionOp.cpp
@@ -72,19 +72,6 @@
}
}
-#if GR_TEST_UTILS
- SkString onDumpInfo() const override {
- SkString str = SkStringPrintf("# combined: %d\n", fRegions.count());
- for (int i = 0; i < fRegions.count(); ++i) {
- const RegionInfo& info = fRegions[i];
- str.appendf("%d: Color: 0x%08x, Region with %d rects\n", i, info.fColor.toBytes_RGBA(),
- info.fRegion.computeRegionComplexity());
- }
- str += fHelper.dumpInfo();
- return str;
- }
-#endif
-
FixedFunctionFlags fixedFunctionFlags() const override { return fHelper.fixedFunctionFlags(); }
GrProcessorSet::Analysis finalize(
@@ -179,6 +166,19 @@
return CombineResult::kMerged;
}
+#if GR_TEST_UTILS
+ SkString onDumpInfo() const override {
+ SkString str = SkStringPrintf("# combined: %d\n", fRegions.count());
+ for (int i = 0; i < fRegions.count(); ++i) {
+ const RegionInfo& info = fRegions[i];
+ str.appendf("%d: Color: 0x%08x, Region with %d rects\n", i, info.fColor.toBytes_RGBA(),
+ info.fRegion.computeRegionComplexity());
+ }
+ str += fHelper.dumpInfo();
+ return str;
+ }
+#endif
+
struct RegionInfo {
SkPMColor4f fColor;
SkRegion fRegion;