Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(344)

Side by Side Diff: third_party/WebKit/Source/modules/mediasource/TrackDefault.idl

Issue 2380383003: Use sequence<T> instead of T[] where possible and document exceptions (Closed)
Patch Set: more Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Per 11 Nov 2014 Editor's Draft 5 // Per 11 Nov 2014 Editor's Draft
6 // https://dvcs.w3.org/hg/html-media/raw-file/tip/media-source/media-source.html #idl-def-TrackDefault 6 // https://dvcs.w3.org/hg/html-media/raw-file/tip/media-source/media-source.html #idl-def-TrackDefault
7 7
8 enum TrackDefaultType { "audio", "video", "text" }; 8 enum TrackDefaultType { "audio", "video", "text" };
9 9
10 [ 10 [
11 Constructor(TrackDefaultType type, DOMString language, DOMString label, sequ ence<DOMString> kinds, optional DOMString byteStreamTrackID = ""), 11 Constructor(TrackDefaultType type, DOMString language, DOMString label, sequ ence<DOMString> kinds, optional DOMString byteStreamTrackID = ""),
12 RaisesException=Constructor, 12 RaisesException=Constructor,
13 RuntimeEnabled=MediaSourceExperimental, 13 RuntimeEnabled=MediaSourceExperimental,
14 ] interface TrackDefault { 14 ] interface TrackDefault {
15 readonly attribute TrackDefaultType type; 15 readonly attribute TrackDefaultType type;
16 readonly attribute DOMString byteStreamTrackID; 16 readonly attribute DOMString byteStreamTrackID;
17 readonly attribute DOMString language; 17 readonly attribute DOMString language;
18 readonly attribute DOMString label; 18 readonly attribute DOMString label;
19 // TODO(wolenetz): |kinds| should be FrozenArray<DOMString>.
wolenetz 2016/10/03 20:53:03 LGTM % reference https://crbug.com/619665 here ple
foolip 2016/10/04 10:02:20 Referencing https://crbug.com/619665 and https://g
19 readonly attribute DOMString[] kinds; 20 readonly attribute DOMString[] kinds;
20 }; 21 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698