WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
169735
Add FIXMEs to update WebRTC
https://bugs.webkit.org/show_bug.cgi?id=169735
Summary
Add FIXMEs to update WebRTC
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
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
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.
Top of Page
Format For Printing
XML
Clone This Bug