extensions/DEPS - Issue 2499493004: Communicate ExtensionSettings policy to renderers

Side by Side Diff: extensions/DEPS

Issue 2499493004: Communicate ExtensionSettings policy to renderers (Closed)

Patch Set: Fix effective TLD wildcard bug, move to Leaky LazyInstance in PermissionsData, removed unnecessary … Created 3 years, 9 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

« chrome/browser/extensions/permissions_updater.cc ('K') | « chrome/test/data/extensions/api_test/content_scripts/permissions/manifest.json ('k') | extensions/browser/renderer_startup_helper.cc » ('j') | extensions/common/extension_messages.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 include_rules = [	1 include_rules = [

2 # Do not add Chrome dependencies. Much work went into removing them.	2 # Do not add Chrome dependencies. Much work went into removing them.

3 "+components/browsing_data",	3 "+components/browsing_data",

4 "+components/crx_file",	4 "+components/crx_file",

5 "+components/guest_view",	5 "+components/guest_view",

6 "+components/prefs",	6 "+components/prefs",

7 "+components/url_matcher",	7 "+components/url_matcher",

8 "+components/version_info",	8 "+components/version_info",

9 "-content",	9 "-content",

10 "+content/grit/content_resources.h",	10 "+content/grit/content_resources.h",

(...skipping 29 matching lines...) Expand all Loading...
40 # Temporarily allowed testing includes. See above.	40 # Temporarily allowed testing includes. See above.

41 # TODO(jamescook): Remove these. http://crbug.com/162530	41 # TODO(jamescook): Remove these. http://crbug.com/162530

42 "+chrome/browser/apps/app_browsertest_util.h",	42 "+chrome/browser/apps/app_browsertest_util.h",

43 "+chrome/browser/extensions/api/management/management_api.h",	43 "+chrome/browser/extensions/api/management/management_api.h",

44 "+chrome/browser/extensions/api/permissions/permissions_api.h",	44 "+chrome/browser/extensions/api/permissions/permissions_api.h",

45 "+chrome/browser/extensions/extension_apitest.h",	45 "+chrome/browser/extensions/extension_apitest.h",

46 "+chrome/browser/extensions/extension_function_test_utils.h",	46 "+chrome/browser/extensions/extension_function_test_utils.h",

47 "+chrome/browser/extensions/extension_service.h",	47 "+chrome/browser/extensions/extension_service.h",

48 "+chrome/browser/extensions/extension_service_test_base.h",	48 "+chrome/browser/extensions/extension_service_test_base.h",

49 "+chrome/browser/extensions/extension_test_message_listener.h",	49 "+chrome/browser/extensions/extension_test_message_listener.h",

	50 "+chrome/browser/extensions/extension_with_management_policy_apitest.h",
	Devlin 2017/03/29 21:36:50 Why do we need this? Why do we need this? nrpeter 2017/03/30 00:06:06 We need to use this in tests, isn't this necessary Show quoted text On 2017/03/29 21:36:50, Devlin wrote: > Why do we need this? We need to use this in tests, isn't this necessary to do so? Devlin 2017/03/30 00:33:53 Only if the tests are in the //extensions layer, w Show quoted text On 2017/03/30 00:06:06, nrpeter wrote: > On 2017/03/29 21:36:50, Devlin wrote: > > Why do we need this? > > We need to use this in tests, isn't this necessary to do so? Only if the tests are in the //extensions layer, which it doesn't seem like any of these are. nrpeter 2017/03/31 21:43:35 Done. Show quoted text On 2017/03/30 00:33:53, Devlin wrote: > On 2017/03/30 00:06:06, nrpeter wrote: > > On 2017/03/29 21:36:50, Devlin wrote: > > > Why do we need this? > > > > We need to use this in tests, isn't this necessary to do so? > > Only if the tests are in the //extensions layer, which it doesn't seem like any > of these are. Done.
50 "+chrome/browser/extensions/test_extension_dir.h",	51 "+chrome/browser/extensions/test_extension_dir.h",

51 "+chrome/browser/extensions/test_extension_prefs.h",	52 "+chrome/browser/extensions/test_extension_prefs.h",

52 "+chrome/browser/extensions/test_extension_system.h",	53 "+chrome/browser/extensions/test_extension_system.h",

53 "+chrome/browser/ui/browser.h",	54 "+chrome/browser/ui/browser.h",

54 "+chrome/common/chrome_switches.h",	55 "+chrome/common/chrome_switches.h",

55 "+chrome/common/extensions/features/feature_channel.h",	56 "+chrome/common/extensions/features/feature_channel.h",

56 "+chrome/test/base/chrome_render_view_test.h",	57 "+chrome/test/base/chrome_render_view_test.h",

57 "+chrome/test/base/testing_profile.h",	58 "+chrome/test/base/testing_profile.h",

58 "+chrome/test/base/ui_test_utils.h",	59 "+chrome/test/base/ui_test_utils.h",

59 ],	60 ],

60 }	61 }

OLD	NEW