Summary: | WebRTC: Update RTCIceCandidate | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Bergkvist <adam.bergkvist> | ||||||||
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | alex, commit-queue, eric.carlson | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 143211 | ||||||||||
Attachments: |
|
Description
Adam Bergkvist
2016-03-16 00:34:07 PDT
Created attachment 274183 [details]
Proposed patch
Created attachment 274185 [details]
Proposed patch
With mac build fix (unused param)
Comment on attachment 274185 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=274185&action=review > Source/WebCore/ChangeLog:9 > + In short: The "candidate" init dictionary member is required. At least one of dictionary Nit: At least one of *the* ... > Source/WebCore/Modules/mediastream/RTCIceCandidate.cpp:61 > + if (!intConversionOk || result > 65535) { Nit: you can use USHRT_MAX here. Created attachment 274271 [details]
Patch for landing
Thanks for your review Eric. Both comments addressed in patch for landing. Comment on attachment 274271 [details] Patch for landing Clearing flags on attachment: 274271 Committed r198325: <http://trac.webkit.org/changeset/198325> |