Bug 120861

Summary: Adding username to RTCIceServer
Product: WebKit Reporter: Thiago de Barros Lacerda <thiago.lacerda>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric.carlson, glenn, gyuyoung.kim, hta, hugo.lima, jer.noble, lauro.neto, luciano.wolf, noam, sergio, tommyw
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Thiago de Barros Lacerda
Reported 2013-09-06 06:59:31 PDT
According to the specification (http://dev.w3.org/2011/webrtc/editor/webrtc.html#idl-def-RTCIceServer) RTCIceServer has a username field.
Attachments
Patch (3.35 KB, patch)
2013-09-06 07:18 PDT, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Patch (3.40 KB, patch)
2013-09-06 13:19 PDT, Thiago de Barros Lacerda 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Thiago de Barros Lacerda
Comment 1 2013-09-06 07:18:12 PDT
Created attachment 210756 [details] Patch
Noam Rosenthal
Comment 2 2013-09-06 07:33:17 PDT
Comment on attachment 210756 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=210756&action=review Overall comments > Source/WebCore/ChangeLog:8 > + No new tests (OOPS!). Don't leave this OOPS in. Are there current tests for RTC?
Thiago de Barros Lacerda
Comment 3 2013-09-06 13:19:45 PDT
Created attachment 210788 [details] Patch
Eric Carlson
Comment 4 2013-09-06 13:34:25 PDT
This looks like it was merged from https://chromium.googlesource.com/chromium/blink/+/415abb016c34a595d8c2d3b0d62d8b804d734ee2%5E%21. Please mention this if this is so.
Eric Carlson
Comment 5 2013-09-06 13:35:01 PDT
*** Bug 120877 has been marked as a duplicate of this bug. ***
Thiago de Barros Lacerda
Comment 6 2013-09-06 16:07:47 PDT
(In reply to comment #4) > This looks like it was merged from https://chromium.googlesource.com/chromium/blink/+/415abb016c34a595d8c2d3b0d62d8b804d734ee2%5E%21. Please mention this if this is so. Hi Eric. I haven't seen this patch from blink, but, indeed, it does the same thing
Thiago de Barros Lacerda
Comment 7 2013-09-08 14:23:09 PDT
(In reply to comment #6) > (In reply to comment #4) > > This looks like it was merged from https://chromium.googlesource.com/chromium/blink/+/415abb016c34a595d8c2d3b0d62d8b804d734ee2%5E%21. Please mention this if this is so. > > Hi Eric. I haven't seen this patch from blink, but, indeed, it does the same thing (In reply to comment #6) > (In reply to comment #4) > > This looks like it was merged from https://chromium.googlesource.com/chromium/blink/+/415abb016c34a595d8c2d3b0d62d8b804d734ee2%5E%21. Please mention this if this is so. > > Hi Eric. I haven't seen this patch from blink, but, indeed, it does the same thing Also I haven't looked the blink's source code when doing this patch. Should I mention it anyway?
Eric Carlson
Comment 8 2013-09-08 16:36:53 PDT
(In reply to comment #7) > > Also I haven't looked the blink's source code when doing this patch. Should I mention it anyway? No, there is no need.
WebKit Commit Bot
Comment 9 2013-09-09 06:46:22 PDT
Comment on attachment 210788 [details] Patch Clearing flags on attachment: 210788 Committed r155346: <http://trac.webkit.org/changeset/155346>
WebKit Commit Bot
Comment 10 2013-09-09 06:46:26 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.