Chromium Code Reviews| Index: webrtc/api/DEPS |
| diff --git a/webrtc/api/DEPS b/webrtc/api/DEPS |
| index 4543a44d751936ac56990e0963eae51f7d1cacca..0b3778bde22a8204659458ced5130cec5680fffb 100644 |
| --- a/webrtc/api/DEPS |
| +++ b/webrtc/api/DEPS |
| @@ -12,9 +12,11 @@ specific_include_rules = { |
| "+webrtc/voice_engine", |
| ], |
| - # TODO(kwiberg): Remove this exception when audio_decoder_factory.h |
| - # has moved to api/. |
| - "peerconnectioninterface\.h": [ |
| - "+webrtc/modules/audio_coding/codecs/audio_decoder_factory.h", |
| + # We allow .cc files in webrtc/api/ to #include a bunch of stuff |
|
kjellander_webrtc
2017/02/01 14:54:50
Which are the cases for this? It sounds like those
kwiberg-webrtc
2017/02/01 20:15:02
Files in api/ must be allowed to include files out
ossu
2017/02/06 12:54:16
Yeah, if api/ is to be the one and only external a
kjellander_webrtc
2017/02/10 07:58:48
Thanks for explaining. Let's do this.
|
| + # that's off-limits for the .h files. That's because .h files leak |
| + # their #includes to whoever's #including them, but .cc files do not |
| + # since no one #includes them. |
| + ".*\.cc": [ |
| + "+webrtc/modules/audio_coding", |
| ], |
| } |