Move all codec specific definitions from modules_include

webrtc/modules/include/module_common_constants.h

Index: webrtc/modules/include/module_common_constants.h
diff --git a/webrtc/modules/include/module_common_constants.h b/webrtc/modules/include/module_common_constants.h
new file mode 100644
index 0000000000000000000000000000000000000000..fc53328372d77545288585e71bba965493dfa342
--- /dev/null
+++ b/webrtc/modules/include/module_common_constants.h
@@ -0,0 +1,26 @@
+/*
+ *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
+ *
+ *  Use of this source code is governed by a BSD-style license
+ *  that can be found in the LICENSE file in the root of the source
+ *  tree. An additional intellectual property rights grant can be found
+ *  in the file PATENTS.  All contributing project authors may
+ *  be found in the AUTHORS file in the root of the source tree.
+ */
+
+// This file contains constants that are used by multiple global
+// codec definitions (modules/video_coding/codecs/*/include/*_globals.h)

[stefan-webrtc, 2016/12/13 09:35:49]

The reason this is placed under modules/include/... is because it's used by both
modules/rtp_rtcp and modules/video_coding. This comment makes it sound a bit
like they should be placed under modules/video_coding/codecs/... instead?

[hta-webrtc, 2016/12/13 14:03:03]

Sounds better to me.

+
+#ifndef WEBRTC_MODULES_INCLUDE_MODULE_COMMON_CONSTANTS_H_
+#define WEBRTC_MODULES_INCLUDE_MODULE_COMMON_CONSTANTS_H_
+
+namespace webrtc {
+
+const int16_t kNoPictureId = -1;
+const int16_t kNoTl0PicIdx = -1;
+const uint8_t kNoTemporalIdx = 0xFF;
+const int kNoKeyIdx = -1;
+
+}  // namespace webrtc
+
+#endif  // WEBRTC_MODULES_INCLUDE_MODULE_COMMON_CONSTANTS_H_