Issue #10491 add HTMLInputElement attributes that reflect content ide…

…ntically it uses a new version of string-cache servo/string-cache#148
servo · Apr 11, 2016 · 9b8f183 · 9b8f183
1 parent bc2237e
commit 9b8f183
Show file tree

Hide file tree

Showing 24 changed files with 371 additions and 1,633 deletions.
diff --git a/components/gfx/Cargo.toml b/components/gfx/Cargo.toml
@@ -27,7 +27,7 @@ serde = "0.7"
 serde_macros = "0.7"
 servo-skia = "0.20130412.0"
 smallvec = "0.1"
-string_cache = {version = "0.2.11", features = ["heap_size"]}
+string_cache = {version = "0.2.12", features = ["heap_size"]}
 time = "0.1.12"
 unicode-script = { version = "0.1", features = ["harfbuzz"] }
 url = {version = "0.5.7", features = ["heap_size"]}

diff --git a/components/layout/Cargo.toml b/components/layout/Cargo.toml
@@ -76,7 +76,7 @@ serde = "0.7"
 serde_json = "0.7"
 serde_macros = "0.7"
 smallvec = "0.1"
-string_cache = {version = "0.2.11", features = ["heap_size"]}
+string_cache = {version = "0.2.12", features = ["heap_size"]}
 time = "0.1"
 unicode-bidi = "0.2"
 unicode-script = { version = "0.1", features = ["harfbuzz"] }

diff --git a/components/script/Cargo.toml b/components/script/Cargo.toml
@@ -91,7 +91,7 @@ rustc-serialize = "0.3"
 selectors = {version = "0.5", features = ["heap_size"]}
 serde = "0.7"
 smallvec = "0.1"
-string_cache = {version = "0.2.11", features = ["heap_size", "unstable"]}
+string_cache = {version = "0.2.12", features = ["heap_size", "unstable"]}
 time = "0.1.12"
 unicase = "1.0"
 url = {version = "0.5.7", features = ["heap_size"]}

diff --git a/components/script/dom/htmlinputelement.rs b/components/script/dom/htmlinputelement.rs
@@ -305,6 +305,25 @@ impl LayoutHTMLInputElementHelpers for LayoutJS<HTMLInputElement> {
 }
 
 impl HTMLInputElementMethods for HTMLInputElement {
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-accept
+    make_getter!(Accept, "accept");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-accept
+    make_setter!(SetAccept, "accept");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-alt
+    make_getter!(Alt, "alt");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-alt
+    make_setter!(SetAlt, "alt");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-dirName
+    make_getter!(DirName, "dirname");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-dirName
+    make_setter!(SetDirName, "dirname");
+
     // https://html.spec.whatwg.org/multipage/#dom-fe-disabled
     make_bool_getter!(Disabled, "disabled");
 
@@ -458,12 +477,54 @@ impl HTMLInputElementMethods for HTMLInputElement {
     // https://html.spec.whatwg.org/multipage/#attr-fs-formnovalidate
     make_bool_setter!(SetFormNoValidate, "formnovalidate");
 
+    // https://html.spec.whatwg.org/multipage/#attr-input-max
+    make_getter!(Max, "max");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-max
+    make_setter!(SetMax, "max");
+
     // https://html.spec.whatwg.org/multipage/#dom-input-maxlength
     make_int_getter!(MaxLength, "maxlength", DEFAULT_MAX_LENGTH);
 
     // https://html.spec.whatwg.org/multipage/#dom-input-maxlength
     make_limited_int_setter!(SetMaxLength, "maxlength", DEFAULT_MAX_LENGTH);
 
+    // https://html.spec.whatwg.org/multipage/#attr-input-min
+    make_getter!(Min, "min");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-min
+    make_setter!(SetMin, "min");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-multiple
+    make_bool_getter!(Multiple, "multiple");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-multiple
+    make_bool_setter!(SetMultiple, "multiple");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-pattern
+    make_getter!(Pattern, "pattern");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-pattern
+    make_setter!(SetPattern, "pattern");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-required
+    make_bool_getter!(Required, "required");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-required
+    make_bool_setter!(SetRequired, "required");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-src
+    make_getter!(Src, "src");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-src
+    make_setter!(SetSrc, "src");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-step
+    make_getter!(Step, "step");
+
+    // https://html.spec.whatwg.org/multipage/#attr-input-step
+    make_setter!(SetStep, "step");
+
     // https://html.spec.whatwg.org/multipage/#dom-input-indeterminate
     fn Indeterminate(&self) -> bool {
         self.upcast::<Element>().state().contains(IN_INDETERMINATE_STATE)

diff --git a/components/script/dom/webidls/HTMLInputElement.webidl b/components/script/dom/webidls/HTMLInputElement.webidl
@@ -5,13 +5,13 @@
 
 // https://html.spec.whatwg.org/multipage/#htmlinputelement
 interface HTMLInputElement : HTMLElement {
-  //         attribute DOMString accept;
-  //         attribute DOMString alt;
+           attribute DOMString accept;
+           attribute DOMString alt;
   //         attribute DOMString autocomplete;
   //         attribute boolean autofocus;
            attribute boolean defaultChecked;
            attribute boolean checked;
-  //         attribute DOMString dirName;
+           attribute DOMString dirName;
            attribute boolean disabled;
   readonly attribute HTMLFormElement? form;
   //readonly attribute FileList? files;
@@ -24,21 +24,21 @@ interface HTMLInputElement : HTMLElement {
              attribute boolean indeterminate;
   //         attribute DOMString inputMode;
   //readonly attribute HTMLElement? list;
-  //         attribute DOMString max;
+           attribute DOMString max;
           [SetterThrows]
           attribute long maxLength;
-  //         attribute DOMString min;
+           attribute DOMString min;
   //         attribute long minLength;
-  //         attribute boolean multiple;
+           attribute boolean multiple;
            attribute DOMString name;
-  //         attribute DOMString pattern;
+           attribute DOMString pattern;
            attribute DOMString placeholder;
            attribute boolean readOnly;
-  //         attribute boolean required;
+           attribute boolean required;
              [SetterThrows]
              attribute unsigned long size;
-  //         attribute DOMString src;
-  //         attribute DOMString step;
+           attribute DOMString src;
+           attribute DOMString step;
            attribute DOMString type;
            attribute DOMString defaultValue;
 [TreatNullAs=EmptyString, SetterThrows]

diff --git a/components/servo/Cargo.lock b/components/servo/Cargo.lock
diff --git a/components/style/Cargo.toml b/components/style/Cargo.toml
@@ -40,7 +40,7 @@ selectors = {version = "0.5", features = ["heap_size", "unstable"]}
 serde = {version = "0.7", features = ["nightly"]}
 serde_macros = "0.7"
 smallvec = "0.1"
-string_cache = {version = "0.2.11", features = ["heap_size"]}
+string_cache = {version = "0.2.12", features = ["heap_size"]}
 time = "0.1"
 url = {version = "0.5.7", features = ["heap_size"]}
 
diff --git a/components/util/Cargo.toml b/components/util/Cargo.toml
@@ -44,7 +44,7 @@ rustc-serialize = "0.3"
 serde = "0.7"
 serde_macros = "0.7"
 smallvec = "0.1"
-string_cache = {version = "0.2.11", features = ["heap_size"]}
+string_cache = {version = "0.2.12", features = ["heap_size"]}
 url = {version = "0.5.7", features = ["heap_size", "serde_serialization"]}
 
 [target.x86_64-pc-windows-gnu.dependencies]

diff --git a/ports/cef/Cargo.lock b/ports/cef/Cargo.lock