Bug 169735

Summary: Add FIXMEs to update WebRTC
Product: WebKit Reporter: Jon Lee <jonlee>
Component: WebKit Misc.Assignee: Jon Lee <jonlee>
Status: RESOLVED FIXED    
Severity: Normal CC: youennf
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 169662    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch youennf: review+

Jon Lee
Reported 2017-03-16 00:26:02 PDT
Add FIXMEs to update WebRTC and add RTCIceTransport{,State}
Attachments
Patch (33.24 KB, patch)
2017-03-16 00:35 PDT, Jon Lee 		no flags
DetailsFormatted DiffDiff
Patch (34.47 KB, patch)
2017-03-16 00:47 PDT, Jon Lee 		no flags
DetailsFormatted DiffDiff
Patch (40.99 KB, patch)
2017-03-16 01:45 PDT, Jon Lee 		no flags
DetailsFormatted DiffDiff
Patch (40.20 KB, patch)
2017-03-16 02:20 PDT, Jon Lee 		no flags
DetailsFormatted DiffDiff
Patch (41.39 KB, patch)
2017-03-16 02:38 PDT, Jon Lee 		youennf: review+
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Jon Lee
Comment 1 2017-03-16 00:35:18 PDT
Created attachment 304614 [details] Patch
Jon Lee
Comment 2 2017-03-16 00:47:03 PDT
Created attachment 304615 [details] Patch
Jon Lee
Comment 3 2017-03-16 01:45:15 PDT
Created attachment 304617 [details] Patch
Jon Lee
Comment 4 2017-03-16 02:20:14 PDT
Created attachment 304622 [details] Patch
Jon Lee
Comment 5 2017-03-16 02:38:55 PDT
Created attachment 304624 [details] Patch
youenn fablet
Comment 6 2017-03-16 09:47:14 PDT
Comment on attachment 304624 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=304624&action=review > Source/WebCore/Modules/mediastream/RTCIceTransport.cpp:27 > +#include "RTCIceTransport.h" Why do we need this file?
Jon Lee
Comment 7 2017-03-16 09:49:21 PDT
Committed r214045: <http://trac.webkit.org/changeset/214045>
Jon Lee
Comment 8 2017-03-16 09:52:20 PDT
(In reply to comment #6) > Comment on attachment 304624 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=304624&action=review > > > Source/WebCore/Modules/mediastream/RTCIceTransport.cpp:27 > > +#include "RTCIceTransport.h" > > Why do we need this file? For a future patch. I can remove for now.
Note You need to log in before you can comment on or make changes to this bug.