Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6259)

Unified Diff: chrome/browser/signin/DEPS

Issue 15024007: Eliminate BrowserProcess dependency from sign-in production code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge LKGR Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/signin/DEPS
diff --git a/chrome/browser/signin/DEPS b/chrome/browser/signin/DEPS
index 9f4d6abab99b1cd5dd64c3f453e9e2b03218f05e..4c69ecb7d95d2f5b874b1d7bfb0adc44cf662d58 100644
--- a/chrome/browser/signin/DEPS
+++ b/chrome/browser/signin/DEPS
@@ -8,7 +8,6 @@ include_rules = [
"+chrome/browser/signin",
# TODO(joi): Get this list to zero.
- "!chrome/browser/browser_process.h",
"!chrome/browser/policy/cloud/user_policy_signin_service.h",
"!chrome/browser/policy/cloud/user_policy_signin_service_factory.h",
"!chrome/browser/profiles/profile.h",
@@ -47,6 +46,7 @@ include_rules = [
specific_include_rules = {
r".*_[a-z]*test\.cc": [
# TODO(joi): Get this list to zero.
+ "!chrome/browser/browser_process.h",
"!chrome/browser/prefs/browser_prefs.h",
"!chrome/browser/sync/profile_sync_service_mock.h",
"!chrome/browser/ui/browser.h",
@@ -56,7 +56,10 @@ specific_include_rules = {
],
# These files are staying in //chrome so no need to limit.
- r"chrome_signin_manager_delegate\.(h|cc)": [
+ r"(chrome_signin_manager_delegate|"
+ r"signin_names_io_thread.*|"
+ r"signin_manager_factory)"
+ r"\.(h|cc)": [
"+chrome/browser",
"+chrome/common",
],

Powered by Google App Engine
This is Rietveld 408576698