Closed Bug 801297 Opened 12 years ago Closed 12 years ago

Fix createAnswer API

Categories

(Core :: WebRTC: Signaling, defect)

Product:
Core
Component:
WebRTC: Signaling
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED
Milestone:
mozilla19

People

(Reporter: ekr, Assigned: anant)

Details

Attachments

(1 file)

Fix createAnswer and add exception handling
6.15 KB, patch
jesup
: review+
Details | Diff | Splinter Review 
The current createAnswer API does not match the specification
Assignee: nobody → anant
createAnswer only takes a single mandatory argument (success callback). I also added try/catch blocks around content callbacks, and exposed the remoteDescription attribute.
Attachment #671505 - Flags: review?(rjesup)
Attachment #671505 - Flags: review?(rjesup) → review+
> 967fa8cc6213 Anant Narayanan — Merge with inbound

Please rebase rather than merge if someone pushes before you :-)
https://hg.mozilla.org/mozilla-central/rev/76c3e2baba71
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Keywords: verifyme
Verified through mochitest automation for basic pc that already exists.
Status: RESOLVED → VERIFIED
Flags: in-testsuite+
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: