Create //webrtc/api:libjingle_peerconnection_api + refactorings.

webrtc/api/mediatype.h

+/*
+ *  Copyright 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.
+ */
+
+#ifndef WEBRTC_API_MEDIATYPE_H_
+#define WEBRTC_API_MEDIATYPE_H_
+
+#include <string>
+
+namespace cricket {
+
+enum MediaType {
+  MEDIA_TYPE_AUDIO,
+  MEDIA_TYPE_VIDEO,
+  MEDIA_TYPE_DATA
+};
+
+std::string MediaTypeToString(MediaType type);
+
+}  // namespace cricket
+
+#endif  // WEBRTC_API_MEDIATYPE_H_

[Taylor Brandstetter, 2016/11/18 21:10:31]

We could call this file "mediatypes.h" (plural) and use it for various
media-related enums and value types. For example, cricket::MediaConfig could
also go here.

[the sun, 2016/11/22 12:59:50]

Could we even call it media_types.h? I believe most files in the repo uses _
between words in file names - it would be nice if we could make it consistent,
at least when adding new files.

[ossu, 2016/11/22 15:36:16]

We could. I was about to say everything else in api/ has names that aren't
separated by underscores, but it seems to be a bit of a mixed bag: the
peerconnection stuff is like that, but stuff under call uses underscores. I
believe this should be normalized across api/ and I prefer underscores as well.