New issue
Advanced search Search tips

Issue 630989 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 569301
issue 630986



Sign in to add a comment

Change RTCStatsCallback from a callback interface to a callback function

Project Member Reported by hbos@chromium.org, Jul 25 2016

Issue description

RTCStatsCallback[1] should really be a callback function and not a callback interface.

The current implementation of callback interface only supports the callback function case so in practice it works as intended, but when callback function is supported ( issue 569301 ) it should be used instead. Especially if  issue 630986  is resolved and callback interface starts working as intended.

Very low priority. The callback interface (if implemented correctly) adds one more means of calling the the function, and this is for a non-standardized callback function that is meant to go away soon.

[1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/modules/peerconnection/RTCStatsCallback.idl?q=RTCStatsCallback.idl&sq=package:chromium&dr
 
Components: Blink>WebRTC
Status: Available (was: Untriaged)
Owner: guidou@chromium.org
Status: Assigned (was: Available)
Cc: guidou@chromium.org
Components: Blink>Bindings
Owner: yukishiino@chromium.org
Status: Fixed (was: Assigned)

Sign in to add a comment