New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 620445 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature



Sign in to add a comment

Refactoring: Make RtpPayloadType enum in Cast Streaming.

Project Member Reported by x...@chromium.org, Jun 15 2016

Issue description

Change |rtp_payload_type| to enum to specify both the type (video/audio) and codec of the RTP stream.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1a96dfbda016edeb50e631e20fb3cd28bda49908

commit 1a96dfbda016edeb50e631e20fb3cd28bda49908
Author: xjz <xjz@chromium.org>
Date: Fri Jul 01 18:41:11 2016

Use enum for RtpPayloadType.
Change |rtp_payload_type| to enum to specify both the type
(video/audio) and encoding codec of the RTP streaming.

BUG= 620445 

Review-Url: https://codereview.chromium.org/2068133005
Cr-Commit-Position: refs/heads/master@{#403490}

[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/chrome/browser/media/cast_transport_host_filter_unittest.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/chrome/common/cast_messages.h
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/chrome/renderer/extensions/cast_streaming_native_handler.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/chrome/renderer/media/cast_rtp_stream.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/chrome/renderer/media/cast_rtp_stream.h
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/cast_config.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/cast_config.h
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/net/cast_transport_config.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/net/cast_transport_config.h
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/net/cast_transport_impl_unittest.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/net/rtp/rtp_sender.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/receiver/frame_receiver.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/sender/audio_sender_unittest.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/test/end2end_unittest.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/test/receiver.cc
[modify] https://crrev.com/1a96dfbda016edeb50e631e20fb3cd28bda49908/media/cast/test/utility/default_config.cc

Comment 2 by x...@chromium.org, Jul 20 2016

Status: Fixed (was: Started)

Sign in to add a comment