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

Unified Diff: Source/core/html/HTMLOptionElement.idl

Issue 1203943002: HTMLOptionElement constructor arguments default values are not per spec (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: Source/core/html/HTMLOptionElement.idl
diff --git a/Source/core/html/HTMLOptionElement.idl b/Source/core/html/HTMLOptionElement.idl
index 1c800da12b7091b8ed28e8dae84ae564ab438126..bcbeed389d434e96643d011d113b2dfc204b556d 100644
--- a/Source/core/html/HTMLOptionElement.idl
+++ b/Source/core/html/HTMLOptionElement.idl
@@ -20,12 +20,11 @@
// https://html.spec.whatwg.org/#the-option-element
-// TODO(philipj): The constructor arguments default values are not per spec.
[
NamedConstructor=Option(optional DOMString data = null,
optional DOMString value = null,
- [Default=Undefined] optional boolean defaultSelected,
- [Default=Undefined] optional boolean selected),
+ optional boolean defaultSelected = false,
+ optional boolean selected = false),
ConstructorCallWith=Document,
RaisesException=Constructor
] interface HTMLOptionElement : HTMLElement {

Powered by Google App Engine
This is Rietveld 408576698