Commit 7490a086 authored by Sylvain Defresne's avatar Sylvain Defresne Committed by Commit Bot

[ios] Allow use of chrome_browser_provider_forward.h from provider

The provider are not permitted to have dependency on //ios/chrome,
however they were able to forward-declare ChromeBrowserProvider so
they ended up with indirect dependency.

To allow moving the class to the global namespace, allow them to
include the forwarding header since this does not increase the
amount of "bad" dependency.

This is a reland of https://crrev.com/c/2010796 with the correct
path (sorry for the spam).

Bug: 1042208, 1043657
Change-Id: I9abe0f492ab1ff86ca095dc22beb437459e8e239
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2010984
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: default avatarMark Cogan <marq@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#733549}
parent 0c53a6fc
...@@ -10,4 +10,8 @@ include_rules = [ ...@@ -10,4 +10,8 @@ include_rules = [
# Do not allow inclusion of forwarding header in the provider code as the # Do not allow inclusion of forwarding header in the provider code as the
# provider is not allowed to depends on //ios/chrome. # provider is not allowed to depends on //ios/chrome.
"-ios/public/provider/chrome/browser/browser_state/chrome_browser_state.h", "-ios/public/provider/chrome/browser/browser_state/chrome_browser_state.h",
# To allow moving ChromeBrowserState to the global namespace, permit the
# use of chrome_browser_state_forward.h.
"+ios/chrome/browser/browser_state/chrome_browser_state_forward.h",
] ]
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment