From e7985f22fa866a656425a7165a6c23a746ef4ffe Mon Sep 17 00:00:00 2001 From: Kevin Lubick Date: Tue, 8 Feb 2022 11:24:32 -0500 Subject: [PATCH] [debugger] Remove dependency cycle DebugCanvas.h and DebugLayerManger.h both included each other. DebugCanvas.h already had a forward declaration of DebugLayerManager, so that seemed the easiest way to break the cycle. Change-Id: Ie8ce5eeca7aba2cdbbcbf92f9fdc42eccc319a2e Bug: skia:12541 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/505641 Reviewed-by: Nathaniel Nifong --- tools/debugger/BUILD.bazel | 1 - tools/debugger/DebugCanvas.h | 1 - 2 files changed, 2 deletions(-) diff --git a/tools/debugger/BUILD.bazel b/tools/debugger/BUILD.bazel index 904063e11c..f21e42532b 100644 --- a/tools/debugger/BUILD.bazel +++ b/tools/debugger/BUILD.bazel @@ -5,7 +5,6 @@ generated_cc_atom( hdrs = ["DebugCanvas.h"], visibility = ["//:__subpackages__"], deps = [ - ":DebugLayerManager_hdr", ":DrawCommand_hdr", "//include/core:SkCanvasVirtualEnforcer_hdr", "//include/core:SkCanvas_hdr", diff --git a/tools/debugger/DebugCanvas.h b/tools/debugger/DebugCanvas.h index 9ae1bf51ca..c67f0c4b89 100644 --- a/tools/debugger/DebugCanvas.h +++ b/tools/debugger/DebugCanvas.h @@ -16,7 +16,6 @@ #include "include/pathops/SkPathOps.h" #include "include/private/SkTArray.h" #include "tools/UrlDataManager.h" -#include "tools/debugger/DebugLayerManager.h" #include "tools/debugger/DrawCommand.h" #include