Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Unified Diff: webrtc/api/DEPS

Issue 2668523004: Move AudioDecoder and related stuff to the api/ directory (Closed)
Patch Set: sort #includes + git cl format Created 3 years, 11 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: 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",
],
}

Powered by Google App Engine
This is Rietveld 408576698