Auto merge of #5923 - nox:limited-unsigned-long, r=jdm

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/5923)
<!-- Reviewable:end -->

1 files changed, 17 insertions, 0 deletions

diff --git a/components/script/dom/attr.rs b/components/script/dom/attr.rs
index 88384aabe5f..3e3979dbaef 100644
--- a/components/script/dom/attr.rs
+++ b/components/script/dom/attr.rs
@@ -54,8 +54,25 @@ impl AttrValue {
         AttrValue::TokenList(tokens, atoms)
     }
 
+    // https://html.spec.whatwg.org/multipage/#reflecting-content-attributes-in-idl-attributes:idl-unsigned-long
     pub fn from_u32(string: DOMString, default: u32) -> AttrValue {
         let result = parse_unsigned_integer(string.chars()).unwrap_or(default);
+        let result = if result > 2147483647 {
+            default
+        } else {
+            result
+        };
+        AttrValue::UInt(string, result)
+    }
+
+    // https://html.spec.whatwg.org/multipage/#limited-to-only-non-negative-numbers-greater-than-zero
+    pub fn from_limited_u32(string: DOMString, default: u32) -> AttrValue {
+        let result = parse_unsigned_integer(string.chars()).unwrap_or(default);
+        let result = if result == 0 || result > 2147483647 {
+            default
+        } else {
+            result
+        };
         AttrValue::UInt(string, result)
     }