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

Side by Side Diff: chrome/browser/google_apis/DEPS

Issue 18659002: Remove use of TestBrowserThreadBundle from unit tests in c/b/google_apis. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/google_apis/base_requests_server_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 include_rules = [ 1 include_rules = [
2 "-chrome", 2 "-chrome",
3 "-content", 3 "-content",
4 "+chrome/browser/google_apis", 4 "+chrome/browser/google_apis",
5 5
6 # BrowserThread should be gone: crbug.com/256112 6 # BrowserThread should be gone: crbug.com/256112
7 "!content/public/browser/browser_thread.h", 7 "!content/public/browser/browser_thread.h",
8 ] 8 ]
9 9
10 # Exceptions are temporarily needed. crbug.com/146989 10 # Exceptions are temporarily needed. crbug.com/146989
11 specific_include_rules = { 11 specific_include_rules = {
12 ".*requests.*unittest\.cc": [ 12 ".*requests.*unittest\.cc": [
13 # This is necessary for AuthService. See below. 13 # This is necessary for AuthService. See below.
14 "!chrome/test/base/testing_profile.h", 14 "!chrome/test/base/testing_profile.h",
15 # TestBrowserThreadBundle should be gone: crbug.com/256109
16 "!content/public/test/test_browser_thread_bundle.h",
17 ], 15 ],
18 # AuthService should be gone. crbug.com/162157 16 # AuthService should be gone. crbug.com/162157
19 "auth_service\.(h|cc)": [ 17 "auth_service\.(h|cc)": [
20 "!chrome/browser/profiles/profile.h", 18 "!chrome/browser/profiles/profile.h",
21 "!chrome/browser/signin/token_service_factory.h", 19 "!chrome/browser/signin/token_service_factory.h",
22 "!chrome/browser/signin/token_service.h", 20 "!chrome/browser/signin/token_service.h",
23 "!chrome/common/chrome_notification_types.h", 21 "!chrome/common/chrome_notification_types.h",
24 "!content/public/browser/notification_details.h", 22 "!content/public/browser/notification_details.h",
25 "!content/public/browser/notification_observer.h", 23 "!content/public/browser/notification_observer.h",
26 "!content/public/browser/notification_registrar.h", 24 "!content/public/browser/notification_registrar.h",
27 "!content/public/browser/notification_source.h", 25 "!content/public/browser/notification_source.h",
28 "!content/public/browser/notification_types.h", 26 "!content/public/browser/notification_types.h",
29 ], 27 ],
30 } 28 }
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/google_apis/base_requests_server_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698