chromecast/DEPS - Issue 1074383002: Introduce VideoConfig/AudioConfig class for CMA backend

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: chromecast/DEPS

Issue 1074383002: Introduce VideoConfig/AudioConfig class for CMA backend (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: single DecoderConfig structure for both Audio and Video Created 5 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 unified diff | Download patch

OLD	NEW
1 # Do not add any includes under chrome/ anywhere in chromecast.	1 # Do not add any includes under chrome/ anywhere in chromecast.

2	2

3 include_rules = [	3 include_rules = [

4 # Chromecast code should explicitly declare its dependencies on each	4 # Chromecast code should explicitly declare its dependencies on each

5 # sub-directory within chromecast/.	5 # sub-directory within chromecast/.

6 "-chromecast",	6 "-chromecast",

7 "+chromecast/base",	7 "+chromecast/base",

	8 "+chromecast/public/media",
	gunsch 2015/04/17 01:38:51 just add "+chromecast/public", any of our code sho just add "+chromecast/public", any of our code should be able to use the public APIs/configs erickung1 2015/04/29 08:52:20 Done. Show quoted text On 2015/04/17 01:38:51, gunsch wrote: > just add "+chromecast/public", any of our code should be able to use the public > APIs/configs Done. gunsch 2015/04/29 17:42:52 Not done Show quoted text On 2015/04/29 08:52:20, erickung1 wrote: > On 2015/04/17 01:38:51, gunsch wrote: > > just add "+chromecast/public", any of our code should be able to use the > public > > APIs/configs > > Done. Not done erickung1 2015/04/29 20:49:15 This is removed after the rebase Show quoted text On 2015/04/29 17:42:52, gunsch wrote: > On 2015/04/29 08:52:20, erickung1 wrote: > > On 2015/04/17 01:38:51, gunsch wrote: > > > just add "+chromecast/public", any of our code should be able to use the > > public > > > APIs/configs > > > > Done. > > Not done This is removed after the rebase
8	9

9 # Other Chromecast-wide dependencies.	10 # Other Chromecast-wide dependencies.

10 "+content/public/common",	11 "+content/public/common",

11 "+crypto",	12 "+crypto",

12 "+grit/chromecast_settings.h",	13 "+grit/chromecast_settings.h",

13 "+grit/shell_resources.h",	14 "+grit/shell_resources.h",

14 "+jni",	15 "+jni",

15 "+net",	16 "+net",

16 "+ui",	17 "+ui",

17 ]	18 ]

OLD	NEW

« no previous file with comments | « no previous file | chromecast/media/cma/backend/audio_pipeline_device.h » ('j') | chromecast/media/cma/base/decoder_config_adapter.h » ('J')