Add missing deps on //chrome/common:service_process_mojom.
//chrome/browser correctly lists //chrome/common:service_process_mojom
in its public_deps. However, //chrome/browser/extensions and
//chrome/browser/ui are listed in allow_circular_includes_from, allowing
them to include headers from //chrome/browser without a dependency. Add
explicit dependencies from //chrome/browser/extensions and
//chrome/browser/ui to //chrome/common:service_process_mojom.
Bug: 805916
Change-Id: I83c2c7588e8056d413b9cccc8f5eceeb203986d7
Reviewed-on: https://chromium-review.googlesource.com/890686
Commit-Queue: Lei Zhang <[email protected]>
Reviewed-by: Lei Zhang <[email protected]>
Cr-Commit-Position: refs/heads/master@{#532517}diff --git a/chrome/browser/extensions/BUILD.gn b/chrome/browser/extensions/BUILD.gn
index 649ea9d..b0bddfd0 100644
--- a/chrome/browser/extensions/BUILD.gn
+++ b/chrome/browser/extensions/BUILD.gn
@@ -1111,6 +1111,7 @@
"api/cloud_print_private/cloud_print_private_api.cc",
"api/cloud_print_private/cloud_print_private_api.h",
]
+ deps += [ "//chrome/common:service_process_mojom" ]
}
if (enable_service_discovery) {
