New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 4 users
Status: Available
Owner: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Sign in to add a comment
Convert options and statistics from explicitly-typed fields to dictionaries (e.g. chromium's base::Value or win's VARIANT)
Reported by, Oct 21 2013 Back to list
Today webrtc & libjingle use structs to pass options and statistics around.
This causes a lot of unnecessary work that is effectively generic serializing/deserializing, as well as verbosity in code that needs to mention each field explicitly instead of simply doing a walk of the dictionary.  

chromium's base::Value (or a similar nestable/mergable dictionary) could replace the use of protobufs in options and explicit-parameter-passing in stats.  

Example uses: http://cr/55119537-p10 and

3rd-party docs:
Comment 1 by, Dec 17 2014
Labels: Area-PeerConnection
Project Member Comment 2 by, Jan 6 2015
We'd like to focus on rewriting VideoOptions and AudioOptions to use base::Value or similar.  

We should not add any more VideoOptions until this is cleaned up.
Project Member Comment 3 by, Jan 6 2015
Labels: EngTriaged mstone-44
Project Member Comment 4 by, Jan 7 2015
(Note that you're with this labling essentially saying that we don't add any VideoOptions before M44, that sounds like it might be unreasonable to me.)
Project Member Comment 5 by, Feb 1 2016
Labels: -Mstone-44
I don't see any recent CLs linked to this bug, so I don't think it's been fixed. I'm therefore leaving this in an open state, but I am removing the milestone label since this bug hasn't been updated in quite some time.
Project Member Comment 6 by, Nov 8 2016
Labels: Pri-3
Project Member Comment 7 by, Dec 14 2016
Owner: ----
Project Member Comment 8 by, Sep 19
Sign in to add a comment