cast: cleanup rtp header and parsing

media/cast/BUILD.gn

Index: media/cast/BUILD.gn
diff --git a/media/cast/BUILD.gn b/media/cast/BUILD.gn
index 15054363f60041d27dc7b921a5b50d44825d0187..28884894187619f3d994825bd9f5d73685144a16 100644
--- a/media/cast/BUILD.gn
+++ b/media/cast/BUILD.gn
@@ -17,6 +17,7 @@ source_set("common") {
     "cast_config.cc",
     "cast_config.h",
     "cast_defines.h",
+    "cast_defines.cc",
     "cast_environment.cc",
     "cast_environment.h",
     "common/clock_drift_smoother.cc",
@@ -87,12 +88,11 @@ source_set("net") {
     "net/rtp/packet_storage.h",
     "net/rtp/receiver_stats.cc",
     "net/rtp/receiver_stats.h",
+    "net/rtp/rtp_defines.h",
     "net/rtp/rtp_packetizer.cc",
     "net/rtp/rtp_packetizer.h",
     "net/rtp/rtp_parser.cc",
     "net/rtp/rtp_parser.h",
-    "net/rtp/rtp_receiver_defines.cc",
-    "net/rtp/rtp_receiver_defines.h",
     "net/rtp/rtp_sender.cc",
     "net/rtp/rtp_sender.h",
     "net/udp_transport.cc",
@@ -306,12 +306,12 @@ test("cast_unittests") {
     "net/rtp/mock_rtp_payload_feedback.h",
     "net/rtp/packet_storage_unittest.cc",
     "net/rtp/receiver_stats_unittest.cc",
-    "net/rtp/rtp_header_parser.cc",
-    "net/rtp/rtp_header_parser.h",
+    "net/rtp/rtp_defines.h",

[miu, 2015/10/01 00:36:22]

Since this target depends on the :net target (via :receiver or :sender), do we
need to include rtp_defines.h in the sources list here?  (I could be mistaken,
but the build should work if the sources are listed in upstream targets.)

[Irfan, 2015/10/01 19:26:52]

Done.

     "net/rtp/rtp_packet_builder.cc",
     "net/rtp/rtp_packetizer_unittest.cc",
+    "net/rtp/rtp_parser.cc"

[miu, 2015/10/01 00:36:22]

ditto: Looks like rtp_parser.* is included in :net.

[Irfan, 2015/10/01 19:26:52]

Done.

+    "net/rtp/rtp_parser.h",
     "net/rtp/rtp_parser_unittest.cc",
-    "net/rtp/rtp_receiver_defines.h",
     "net/udp_transport_unittest.cc",
     "receiver/audio_decoder_unittest.cc",
     "receiver/frame_receiver_unittest.cc",