From 25eb47ccf18f0764eb9a94d586eb8b74ffd712d3 Mon Sep 17 00:00:00 2001 From: Tommi Date: Thu, 29 Aug 2019 16:39:05 +0200 Subject: Make the RtpHeaderParserImpl available to tests and tools only. There are a few reasons for making this test only: * The code is only used by tests and utilities. * The pure interface has only a single implementation so an interface isn't really needed. (a followup change could remove it altogether) * The implementation always incorporates locking regardless of how the class gets used. See e.g. previous use in the Packet class. * The implementation is a layer on top of RtpUtility::RtpHeaderParser which is sufficient for most production cases. Change-Id: Ide6d50567cf8ae5127a2eb04cceeb10cf317ec36 Bug: none Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/150658 Commit-Queue: Tommi Reviewed-by: Karl Wiberg Cr-Commit-Position: refs/heads/master@{#29010} --- examples/BUILD.gn | 1 + 1 file changed, 1 insertion(+) (limited to 'examples') diff --git a/examples/BUILD.gn b/examples/BUILD.gn index ce60643e0c..0d54d303c2 100644 --- a/examples/BUILD.gn +++ b/examples/BUILD.gn @@ -665,6 +665,7 @@ if (is_linux || is_win) { "../system_wrappers:field_trial", "../test:field_trial", "../test:platform_video_capturer", + "../test:rtp_test_utils", "//third_party/abseil-cpp/absl/memory", "//third_party/abseil-cpp/absl/types:optional", ] -- cgit v1.2.3