Bug 169735 - Add FIXMEs to update WebRTC
Summary: Add FIXMEs to update WebRTC
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jon Lee
URL:
Keywords:
Depends on:
Blocks: 169662
  Show dependency treegraph
 
Reported: 2017-03-16 00:26 PDT by Jon Lee
Modified: 2017-03-16 09:52 PDT (History)
1 user (show)

See Also:


Attachments
Patch (33.24 KB, patch)
2017-03-16 00:35 PDT, Jon Lee
no flags Details | Formatted Diff | Diff
Patch (34.47 KB, patch)
2017-03-16 00:47 PDT, Jon Lee
no flags Details | Formatted Diff | Diff
Patch (40.99 KB, patch)
2017-03-16 01:45 PDT, Jon Lee
no flags Details | Formatted Diff | Diff
Patch (40.20 KB, patch)
2017-03-16 02:20 PDT, Jon Lee
no flags Details | Formatted Diff | Diff
Patch (41.39 KB, patch)
2017-03-16 02:38 PDT, Jon Lee
youennf: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jon Lee 2017-03-16 00:26:02 PDT
Add FIXMEs to update WebRTC and add RTCIceTransport{,State}
Comment 1 Jon Lee 2017-03-16 00:35:18 PDT
Created attachment 304614 [details]
Patch
Comment 2 Jon Lee 2017-03-16 00:47:03 PDT
Created attachment 304615 [details]
Patch
Comment 3 Jon Lee 2017-03-16 01:45:15 PDT
Created attachment 304617 [details]
Patch
Comment 4 Jon Lee 2017-03-16 02:20:14 PDT
Created attachment 304622 [details]
Patch
Comment 5 Jon Lee 2017-03-16 02:38:55 PDT
Created attachment 304624 [details]
Patch
Comment 6 youenn fablet 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?
Comment 7 Jon Lee 2017-03-16 09:49:21 PDT
Committed r214045: <http://trac.webkit.org/changeset/214045>
Comment 8 Jon Lee 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.