Skip to content

Commit

Permalink
[Code health] Enable gn check for //extensions/browser/api/declarative:*
Browse files Browse the repository at this point in the history
This CL removes the cyclic dependency between
//extensions/browser/api/declarative_content:* and
//extensions/browser/api/declarative:* by unifying the two targets to
make enabling gn check for
//extensions/browser/api/declarative:* possible.

Bug: 1158942
Change-Id: Ib580c0f6c4a5e6b9eb3bae73d30dbd5f82c340f0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2708084
Reviewed-by: Erik Staab <estaab@chromium.org>
Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Commit-Queue: Ghazale Hosseinabadi <ghazale@chromium.org>
Cr-Commit-Position: refs/heads/master@{#856541}
  • Loading branch information
Ghazale Hosseinabadi authored and Chromium LUCI CQ committed Feb 23, 2021
1 parent be6a070 commit 8412abc
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 23 deletions.
2 changes: 0 additions & 2 deletions .gn
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,6 @@ no_check_targets = [
"//chrome/browser/storage_access_api:*", # 2 errors
"//chrome/browser/touch_to_fill/android:*", # 8 errors
"//chrome/test:*", # 2682 errors

"//extensions/browser/api/declarative:*", # 20 errors
"//extensions/browser/api/declarative_net_request:*", # 18 errors
"//extensions/browser/api/declarative_webrequest:*", # 29 errors
"//extensions/browser/api/web_request:*", # 37 errors
Expand Down
1 change: 0 additions & 1 deletion extensions/browser/api/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,6 @@ group("api_implementations") {
"//extensions/browser/api/bluetooth_socket",
"//extensions/browser/api/cast_channel",
"//extensions/browser/api/declarative",
"//extensions/browser/api/declarative_content",
"//extensions/browser/api/declarative_net_request",
"//extensions/browser/api/declarative_webrequest",
"//extensions/browser/api/dns",
Expand Down
6 changes: 6 additions & 0 deletions extensions/browser/api/declarative/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ assert(enable_extensions,

source_set("declarative") {
sources = [
"../declarative_content/content_rules_registry.h",
"declarative_api.cc",
"declarative_api.h",
"declarative_rule.h",
Expand All @@ -22,7 +23,12 @@ source_set("declarative") {
]

deps = [
"//components/url_matcher",
"//content/public/browser",
"//extensions/browser/api",
"//extensions/browser/api/declarative_webrequest",
"//extensions/browser/api/web_request",
"//extensions/common",
"//extensions/common/api",
]

Expand Down
20 changes: 0 additions & 20 deletions extensions/browser/api/declarative_content/BUILD.gn

This file was deleted.

0 comments on commit 8412abc

Please sign in to comment.