content/DEPS - Issue 1921553003: Move permission.mojom from WebKit/public/platform/ to components/

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: content/DEPS

Issue 1921553003: Move permission.mojom from WebKit/public/platform/ to components/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: gn check fix Created 4 years, 8 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: content/DEPS

diff --git a/content/DEPS b/content/DEPS

index 1d6f952b9ba7fde072427e6cb6236ec54110e767..f4c300ffaff73d2335f961a71cf2bb6c8aa6cc9c 100644

--- a/content/DEPS

+++ b/content/DEPS

@@ -1,6 +1,6 @@

# Do NOT add chrome to the list below. We shouldn't be including files

-# from src/chrome in src/content. Also, we may only depend on files in

-# src/components that are shared with the mojo html_viewer.

+# from src/chrome in src/content. Also, we may only depend on mojo interface

jam 2016/05/03 03:32:46 actually you can just remove the second sentence a

xhwang 2016/05/03 05:04:11 Done.

+# files in src/components that are shared with other modules.

include_rules = [

# The subdirectories in content/ will manually allow their own include

# directories in content/ so we disallow all of them.

@@ -110,6 +110,9 @@ include_rules = [

# For generated JNI includes.

"+jni",

+ # For permission mojo interfaces.

+ "+components/permissions",

jam 2016/05/03 03:32:46 put this up near the other components/ lines

xhwang 2016/05/03 05:04:11 Moved up. But I don't see other components/ lines.

]

# content -> content/shell dependency is not allowed, except for browser tests.

« chrome/browser/DEPS ('K') | « components/permissions/permission_status.mojom ('k') | content/browser/BUILD.gn » ('j') | content/browser/DEPS » ('J')