RESOLVED FIXED Bug 85491
MediaStream API: Make PeerConnection00's API fully compliant with the draft 
https://bugs.webkit.org/show_bug.cgi?id=85491
Summary MediaStream API: Make PeerConnection00's API fully compliant with the draft
Tommy Widenflycht
Reported 2012-05-03 06:29:26 PDT
Mainly make the relevant API's use objects (aka Dictionaries) instead of the temporary strings, but also making a few API's exception aware and changing the name of a flag.
Attachments
Patch (16.62 KB, patch)
2012-05-03 06:38 PDT, Tommy Widenflycht 		no flags
DetailsFormatted DiffDiff
Patch (19.50 KB, patch)
2012-05-03 07:34 PDT, Tommy Widenflycht 		no flags
DetailsFormatted DiffDiff
Patch (22.97 KB, patch)
2012-05-04 04:14 PDT, Tommy Widenflycht 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Tommy Widenflycht
Comment 1 2012-05-03 06:38:47 PDT
Created attachment 140000 [details] Patch
WebKit Review Bot
Comment 2 2012-05-03 06:41:50 PDT
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Philippe Normand
Comment 3 2012-05-03 06:43:52 PDT
Comment on attachment 140000 [details] Patch Attachment 140000 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/12610242
Tommy Widenflycht
Comment 4 2012-05-03 07:34:41 PDT
Created attachment 140010 [details] Patch
Adam Barth
Comment 5 2012-05-03 10:50:36 PDT
Comment on attachment 140010 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=140010&action=review > Source/WebCore/Modules/mediastream/PeerConnection00.cpp:260 > + if (!ec) > + return; Is this backwards? I thought ec == 0 is success. > LayoutTests/ChangeLog:1 > +2012-05-03 Tommy Widenflycht <tommyw@google.com> Looks like this ChangeLog got corrupted a bit.
Darin Adler
Comment 6 2012-05-03 15:58:22 PDT
Comment on attachment 140010 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=140010&action=review >> Source/WebCore/Modules/mediastream/PeerConnection00.cpp:260 >> + return; > > Is this backwards? I thought ec == 0 is success. More importantly, if this is backwards, why didn’t a test catch that fact?
Tommy Widenflycht
Comment 7 2012-05-04 04:14:32 PDT
Created attachment 140188 [details] Patch
Tommy Widenflycht
Comment 8 2012-05-04 04:17:52 PDT
Comment on attachment 140010 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=140010&action=review >>> Source/WebCore/Modules/mediastream/PeerConnection00.cpp:260 >>> + return; >> >> Is this backwards? I thought ec == 0 is success. > > More importantly, if this is backwards, why didn’t a test catch that fact? Yes, it was backward. And I am ashamed. Test now added. It's so cold in Sweden that my brain froze. >> LayoutTests/ChangeLog:1 >> +2012-05-03 Tommy Widenflycht <tommyw@google.com> > > Looks like this ChangeLog got corrupted a bit. Fixed.
Adam Barth
Comment 9 2012-05-04 10:13:26 PDT
Comment on attachment 140188 [details] Patch Thanks.
WebKit Review Bot
Comment 10 2012-05-04 10:54:37 PDT
Comment on attachment 140188 [details] Patch Clearing flags on attachment: 140188 Committed r116127: <http://trac.webkit.org/changeset/116127>
WebKit Review Bot
Comment 11 2012-05-04 10:54:43 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.