This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 25810 - Change all extensable enum to DomStrings
Summary: Change all extensable enum to DomStrings
Status: RESOLVED WONTFIX
Alias: None
Product: WebRTC Working Group
Classification: Unclassified
Component: Media Capture and Streams (show other bugs)
Version: unspecified
Hardware: PC All
: P2 normal
Target Milestone: ---
Assignee: public-media-capture@w3.org
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 25298
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-19 18:46 UTC by Cullen Jennings
Modified: 2014-09-10 06:42 UTC (History)
5 users (show)

See Also:


Attachments

Description Cullen Jennings 2014-05-19 18:46:24 UTC
Given how WebIDL enum work, for any one of our current strings that is an Enum, if future specs could extend them, change to DomString
Comment 1 Harald Alvestrand 2014-07-01 09:21:57 UTC
25793 (for facingMode) is an instance of this problem, so adding as a dependency.

Only enums that are used as function arguments (including those that are in dictionaries used as function arguments) cause this problem.
Comment 2 Harald Alvestrand 2014-07-02 11:11:44 UTC
Updating to reflect that 25793 has been marked a duplicate.
Comment 3 Dominique Hazael-Massieux 2014-08-28 08:21:34 UTC
VideoFacingMode has been resolved different than suggested here (with an "other" value).

These leaves 2 other enums:
* MediaStreamTrackState
* MediaDeviceKind

Both are only set by the user agent, so it's not clear to me that there would be any benefit in changing them from enums.

I suggest closing as WONTFIX.
Comment 4 Jan-Ivar Bruaroey [:jib] 2014-08-28 15:11:39 UTC
(In reply to Dominique Hazael-Massieux from comment #3)
> VideoFacingMode has been resolved different than suggested here (with an
> "other" value).

Not that different. It's still a DOMString. The "other" enum-value just leaves the door open for Bug 19936 to let implementations use enum here in the future.

> These leaves 2 other enums:
> * MediaStreamTrackState
> * MediaDeviceKind
> 
> Both are only set by the user agent, so it's not clear to me that there
> would be any benefit in changing them from enums.
> 
> I suggest closing as WONTFIX.

Good observation. I agree.
Comment 5 Stefan Hakansson LK 2014-09-10 06:42:50 UTC
Closing as "WONTFIX" per Dom's and Jan-Ivar's recommendation.