Skip to content

Commit

Permalink
Add GN restriction that //content isn't used on iOS
Browse files Browse the repository at this point in the history
Similar to the restriction in //chrome/BUILD.gn. If you're trying to
use //content on iOS, you want to be using //ios/web instead. If you
need to use //content on other platforms and also need your feature to
run on iOS, it should be a layered component.

Change-Id: Ibe5f2cb9a3013d5a5160254ba6ee837f0e49378d
Reviewed-on: https://chromium-review.googlesource.com/c/1466983
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#631618}
  • Loading branch information
colinblundell authored and Commit Bot committed Feb 13, 2019
1 parent f0fda2b commit a6a9688
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions content/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ config("content_implementation") {
]
}

assert(!is_ios, "Chromium/iOS shouldn't use anything in //content")

# When targets depend on, e.g. //content/public/browser, what happens? To
# facilitate the complexity here, the "public" targets are groups that forward
# to the right thing depending on the build mode. Say for additional
Expand Down

0 comments on commit a6a9688

Please sign in to comment.