content/renderer/DEPS - Issue 11413065: Adding first WebRTC browser test to content_browsertests

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

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

Side by Side Diff: content/renderer/DEPS

Issue 11413065: Adding first WebRTC browser test to content_browsertests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src

Patch Set: Created 8 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

OLD	NEW
1 include_rules = [	1 include_rules = [

2 "+content/public/renderer",	2 "+content/public/renderer",

3 "+content/public/browser/native_web_keyboard_event.h", # For tests.	3 "+content/public/browser/native_web_keyboard_event.h", # For tests.

4 "+content/public/browser/web_ui_controller_factory.h", # For tests.	4 "+content/public/browser/web_ui_controller_factory.h", # For tests.

	5 "+content/public/browser/web_contents.h", # For tests.
	jam 2012/11/19 19:14:51 nit: i realize there are two existing includes abo nit: i realize there are two existing includes above, but these are incorrect and shouldn't be used as examples. your test actually has no dependencies on content/renderer, so put it in content/browser phoglund_chromium 2012/11/20 09:02:03 Done. Show quoted text On 2012/11/19 19:14:51, John Abd-El-Malek wrote: > nit: i realize there are two existing includes above, but these are incorrect > and shouldn't be used as examples. > > your test actually has no dependencies on content/renderer, so put it in > content/browser Done.
5 "+jingle/glue",	6 "+jingle/glue",

6 "+media", # For audio input/output and audio/video decoding.	7 "+media", # For audio input/output and audio/video decoding.

7 "+third_party/hyphen/hyphen.h",	8 "+third_party/hyphen/hyphen.h",

8 "+third_party/libjingle",	9 "+third_party/libjingle",

9 "+third_party/mach_override",	10 "+third_party/mach_override",

10 "+third_party/tcmalloc",	11 "+third_party/tcmalloc",

11 "+v8/include",	12 "+v8/include",

12 ]	13 ]

OLD	NEW