 Chromium Code Reviews
 Chromium Code Reviews Issue 1844803002:
  Modify PeerConnection for end-to-end QuicDataChannel usage  (Closed) 
  Base URL: https://chromium.googlesource.com/external/webrtc.git@master
    
  
    Issue 1844803002:
  Modify PeerConnection for end-to-end QuicDataChannel usage  (Closed) 
  Base URL: https://chromium.googlesource.com/external/webrtc.git@master| Index: webrtc/api/webrtcsdp.cc | 
| diff --git a/webrtc/api/webrtcsdp.cc b/webrtc/api/webrtcsdp.cc | 
| index 93d4acc6c3fad5fd2e2612ff22af02de2554baeb..e73d3d9421efd07fd8f87591561d44c4dccbc77c 100644 | 
| --- a/webrtc/api/webrtcsdp.cc | 
| +++ b/webrtc/api/webrtcsdp.cc | 
| @@ -151,6 +151,7 @@ static const char kAttributeSctpPort[] = "sctp-port"; | 
| // Experimental flags | 
| static const char kAttributeXGoogleFlag[] = "x-google-flag"; | 
| static const char kValueConference[] = "conference"; | 
| +static const char kAttributeQuic[] = "quic"; | 
| 
Taylor Brandstetter
2016/04/01 23:23:42
Does it make sense for this to be an attribute? I
 | 
| // Candidate | 
| static const char kCandidateHost[] = "host"; | 
| @@ -820,6 +821,11 @@ std::string SdpSerialize(const JsepSessionDescription& jdesc, | 
| // Time Description. | 
| AddLine(kTimeDescription, &message); | 
| + // QUIC Description. | 
| + if (desc->quic()) { | 
| + AddLine("a=quic", &message); | 
| 
Taylor Brandstetter
2016/04/01 23:23:42
Replace this with:
     InitAttrLine(kAttributeQui
 | 
| + } | 
| + | 
| // Group | 
| if (desc->HasGroup(cricket::GROUP_TYPE_BUNDLE)) { | 
| std::string group_line = kAttrGroup; | 
| @@ -2004,6 +2010,8 @@ bool ParseSessionDescription(const std::string& message, size_t* pos, | 
| return false; | 
| } | 
| session_extmaps->push_back(extmap); | 
| + } else if (HasAttribute(line, kAttributeQuic)) { | 
| + desc->set_quic(true); | 
| } | 
| } |