HTMLImageElement.hspace / vspace attributes should be unsigned
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Sep 2016 22:02:39 +0000 (22:02 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Sep 2016 22:02:39 +0000 (22:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=161754

Reviewed by Alex Christensen.

LayoutTests/imported/w3c:

Rebaseline W3C test now that more checks are passing.

* web-platform-tests/html/dom/reflection-embedded-expected.txt:

Source/WebCore:

HTMLImageElement.hspace / vspace attributes should be unsigned:
- https://html.spec.whatwg.org/#HTMLImageElement-partial

Firefox agrees with the specification, however, those were signed in
webKit.

No new tests, rebaselined existing test.

* html/HTMLImageElement.idl:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@205665 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/html/dom/reflection-embedded-expected.txt
LayoutTests/platform/ios-simulator/imported/w3c/web-platform-tests/html/dom/reflection-embedded-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/html/HTMLImageElement.idl

index 03ff3b1..882693c 100644 (file)
@@ -1,5 +1,16 @@
 2016-09-08  Chris Dumez  <cdumez@apple.com>
 
+        HTMLImageElement.hspace / vspace attributes should be unsigned
+        https://bugs.webkit.org/show_bug.cgi?id=161754
+
+        Reviewed by Alex Christensen.
+
+        Rebaseline W3C test now that more checks are passing.
+
+        * web-platform-tests/html/dom/reflection-embedded-expected.txt:
+
+2016-09-08  Chris Dumez  <cdumez@apple.com>
+
         HTMLImageElement.width / height attributes should be unsigned
         https://bugs.webkit.org/show_bug.cgi?id=161730
 
index 07e2194..2448950 100644 (file)
@@ -1426,11 +1426,11 @@ PASS img.hspace: IDL get with DOM attribute unset
 PASS img.hspace: setAttribute() to -2147483649 followed by getAttribute() 
 PASS img.hspace: setAttribute() to -2147483649 followed by IDL get 
 PASS img.hspace: setAttribute() to -2147483648 followed by getAttribute() 
-FAIL img.hspace: setAttribute() to -2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.hspace: setAttribute() to -2147483648 followed by IDL get 
 PASS img.hspace: setAttribute() to -36 followed by getAttribute() 
-FAIL img.hspace: setAttribute() to -36 followed by IDL get assert_equals: expected 0 but got -36
+PASS img.hspace: setAttribute() to -36 followed by IDL get 
 PASS img.hspace: setAttribute() to -1 followed by getAttribute() 
-FAIL img.hspace: setAttribute() to -1 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.hspace: setAttribute() to -1 followed by IDL get 
 PASS img.hspace: setAttribute() to 0 followed by getAttribute() 
 PASS img.hspace: setAttribute() to 0 followed by IDL get 
 PASS img.hspace: setAttribute() to 1 followed by getAttribute() 
@@ -1448,7 +1448,7 @@ PASS img.hspace: setAttribute() to 4294967296 followed by IDL get
 PASS img.hspace: setAttribute() to "" followed by getAttribute() 
 PASS img.hspace: setAttribute() to "" followed by IDL get 
 PASS img.hspace: setAttribute() to "-1" followed by getAttribute() 
-FAIL img.hspace: setAttribute() to "-1" followed by IDL get assert_equals: expected 0 but got -1
+PASS img.hspace: setAttribute() to "-1" followed by IDL get 
 PASS img.hspace: setAttribute() to "-0" followed by getAttribute() 
 PASS img.hspace: setAttribute() to "-0" followed by IDL get 
 PASS img.hspace: setAttribute() to "0" followed by getAttribute() 
@@ -1545,21 +1545,21 @@ PASS img.hspace: IDL set to "-0" should not throw
 PASS img.hspace: IDL set to "-0" followed by getAttribute() 
 PASS img.hspace: IDL set to "-0" followed by IDL get 
 PASS img.hspace: IDL set to 2147483648 should not throw 
-FAIL img.hspace: IDL set to 2147483648 followed by getAttribute() assert_equals: expected "0" but got "-2147483648"
-FAIL img.hspace: IDL set to 2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.hspace: IDL set to 2147483648 followed by getAttribute() 
+PASS img.hspace: IDL set to 2147483648 followed by IDL get 
 PASS img.hspace: IDL set to 4294967295 should not throw 
-FAIL img.hspace: IDL set to 4294967295 followed by getAttribute() assert_equals: expected "0" but got "-1"
-FAIL img.hspace: IDL set to 4294967295 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.hspace: IDL set to 4294967295 followed by getAttribute() 
+PASS img.hspace: IDL set to 4294967295 followed by IDL get 
 PASS img.vspace: typeof IDL attribute 
 PASS img.vspace: IDL get with DOM attribute unset 
 PASS img.vspace: setAttribute() to -2147483649 followed by getAttribute() 
 PASS img.vspace: setAttribute() to -2147483649 followed by IDL get 
 PASS img.vspace: setAttribute() to -2147483648 followed by getAttribute() 
-FAIL img.vspace: setAttribute() to -2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.vspace: setAttribute() to -2147483648 followed by IDL get 
 PASS img.vspace: setAttribute() to -36 followed by getAttribute() 
-FAIL img.vspace: setAttribute() to -36 followed by IDL get assert_equals: expected 0 but got -36
+PASS img.vspace: setAttribute() to -36 followed by IDL get 
 PASS img.vspace: setAttribute() to -1 followed by getAttribute() 
-FAIL img.vspace: setAttribute() to -1 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.vspace: setAttribute() to -1 followed by IDL get 
 PASS img.vspace: setAttribute() to 0 followed by getAttribute() 
 PASS img.vspace: setAttribute() to 0 followed by IDL get 
 PASS img.vspace: setAttribute() to 1 followed by getAttribute() 
@@ -1577,7 +1577,7 @@ PASS img.vspace: setAttribute() to 4294967296 followed by IDL get
 PASS img.vspace: setAttribute() to "" followed by getAttribute() 
 PASS img.vspace: setAttribute() to "" followed by IDL get 
 PASS img.vspace: setAttribute() to "-1" followed by getAttribute() 
-FAIL img.vspace: setAttribute() to "-1" followed by IDL get assert_equals: expected 0 but got -1
+PASS img.vspace: setAttribute() to "-1" followed by IDL get 
 PASS img.vspace: setAttribute() to "-0" followed by getAttribute() 
 PASS img.vspace: setAttribute() to "-0" followed by IDL get 
 PASS img.vspace: setAttribute() to "0" followed by getAttribute() 
@@ -1674,11 +1674,11 @@ PASS img.vspace: IDL set to "-0" should not throw
 PASS img.vspace: IDL set to "-0" followed by getAttribute() 
 PASS img.vspace: IDL set to "-0" followed by IDL get 
 PASS img.vspace: IDL set to 2147483648 should not throw 
-FAIL img.vspace: IDL set to 2147483648 followed by getAttribute() assert_equals: expected "0" but got "-2147483648"
-FAIL img.vspace: IDL set to 2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.vspace: IDL set to 2147483648 followed by getAttribute() 
+PASS img.vspace: IDL set to 2147483648 followed by IDL get 
 PASS img.vspace: IDL set to 4294967295 should not throw 
-FAIL img.vspace: IDL set to 4294967295 followed by getAttribute() assert_equals: expected "0" but got "-1"
-FAIL img.vspace: IDL set to 4294967295 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.vspace: IDL set to 4294967295 followed by getAttribute() 
+PASS img.vspace: IDL set to 4294967295 followed by IDL get 
 PASS img.longDesc: typeof IDL attribute 
 PASS img.longDesc: IDL get with DOM attribute unset 
 PASS img.longDesc: setAttribute() to "" followed by getAttribute() 
index 681c62a..b3e0657 100644 (file)
@@ -1426,11 +1426,11 @@ PASS img.hspace: IDL get with DOM attribute unset
 PASS img.hspace: setAttribute() to -2147483649 followed by getAttribute() 
 PASS img.hspace: setAttribute() to -2147483649 followed by IDL get 
 PASS img.hspace: setAttribute() to -2147483648 followed by getAttribute() 
-FAIL img.hspace: setAttribute() to -2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.hspace: setAttribute() to -2147483648 followed by IDL get 
 PASS img.hspace: setAttribute() to -36 followed by getAttribute() 
-FAIL img.hspace: setAttribute() to -36 followed by IDL get assert_equals: expected 0 but got -36
+PASS img.hspace: setAttribute() to -36 followed by IDL get 
 PASS img.hspace: setAttribute() to -1 followed by getAttribute() 
-FAIL img.hspace: setAttribute() to -1 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.hspace: setAttribute() to -1 followed by IDL get 
 PASS img.hspace: setAttribute() to 0 followed by getAttribute() 
 PASS img.hspace: setAttribute() to 0 followed by IDL get 
 PASS img.hspace: setAttribute() to 1 followed by getAttribute() 
@@ -1448,7 +1448,7 @@ PASS img.hspace: setAttribute() to 4294967296 followed by IDL get
 PASS img.hspace: setAttribute() to "" followed by getAttribute() 
 PASS img.hspace: setAttribute() to "" followed by IDL get 
 PASS img.hspace: setAttribute() to "-1" followed by getAttribute() 
-FAIL img.hspace: setAttribute() to "-1" followed by IDL get assert_equals: expected 0 but got -1
+PASS img.hspace: setAttribute() to "-1" followed by IDL get 
 PASS img.hspace: setAttribute() to "-0" followed by getAttribute() 
 PASS img.hspace: setAttribute() to "-0" followed by IDL get 
 PASS img.hspace: setAttribute() to "0" followed by getAttribute() 
@@ -1545,21 +1545,21 @@ PASS img.hspace: IDL set to "-0" should not throw
 PASS img.hspace: IDL set to "-0" followed by getAttribute() 
 PASS img.hspace: IDL set to "-0" followed by IDL get 
 PASS img.hspace: IDL set to 2147483648 should not throw 
-FAIL img.hspace: IDL set to 2147483648 followed by getAttribute() assert_equals: expected "0" but got "-2147483648"
-FAIL img.hspace: IDL set to 2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.hspace: IDL set to 2147483648 followed by getAttribute() 
+PASS img.hspace: IDL set to 2147483648 followed by IDL get 
 PASS img.hspace: IDL set to 4294967295 should not throw 
-FAIL img.hspace: IDL set to 4294967295 followed by getAttribute() assert_equals: expected "0" but got "-1"
-FAIL img.hspace: IDL set to 4294967295 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.hspace: IDL set to 4294967295 followed by getAttribute() 
+PASS img.hspace: IDL set to 4294967295 followed by IDL get 
 PASS img.vspace: typeof IDL attribute 
 PASS img.vspace: IDL get with DOM attribute unset 
 PASS img.vspace: setAttribute() to -2147483649 followed by getAttribute() 
 PASS img.vspace: setAttribute() to -2147483649 followed by IDL get 
 PASS img.vspace: setAttribute() to -2147483648 followed by getAttribute() 
-FAIL img.vspace: setAttribute() to -2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.vspace: setAttribute() to -2147483648 followed by IDL get 
 PASS img.vspace: setAttribute() to -36 followed by getAttribute() 
-FAIL img.vspace: setAttribute() to -36 followed by IDL get assert_equals: expected 0 but got -36
+PASS img.vspace: setAttribute() to -36 followed by IDL get 
 PASS img.vspace: setAttribute() to -1 followed by getAttribute() 
-FAIL img.vspace: setAttribute() to -1 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.vspace: setAttribute() to -1 followed by IDL get 
 PASS img.vspace: setAttribute() to 0 followed by getAttribute() 
 PASS img.vspace: setAttribute() to 0 followed by IDL get 
 PASS img.vspace: setAttribute() to 1 followed by getAttribute() 
@@ -1577,7 +1577,7 @@ PASS img.vspace: setAttribute() to 4294967296 followed by IDL get
 PASS img.vspace: setAttribute() to "" followed by getAttribute() 
 PASS img.vspace: setAttribute() to "" followed by IDL get 
 PASS img.vspace: setAttribute() to "-1" followed by getAttribute() 
-FAIL img.vspace: setAttribute() to "-1" followed by IDL get assert_equals: expected 0 but got -1
+PASS img.vspace: setAttribute() to "-1" followed by IDL get 
 PASS img.vspace: setAttribute() to "-0" followed by getAttribute() 
 PASS img.vspace: setAttribute() to "-0" followed by IDL get 
 PASS img.vspace: setAttribute() to "0" followed by getAttribute() 
@@ -1674,11 +1674,11 @@ PASS img.vspace: IDL set to "-0" should not throw
 PASS img.vspace: IDL set to "-0" followed by getAttribute() 
 PASS img.vspace: IDL set to "-0" followed by IDL get 
 PASS img.vspace: IDL set to 2147483648 should not throw 
-FAIL img.vspace: IDL set to 2147483648 followed by getAttribute() assert_equals: expected "0" but got "-2147483648"
-FAIL img.vspace: IDL set to 2147483648 followed by IDL get assert_equals: expected 0 but got -2147483648
+PASS img.vspace: IDL set to 2147483648 followed by getAttribute() 
+PASS img.vspace: IDL set to 2147483648 followed by IDL get 
 PASS img.vspace: IDL set to 4294967295 should not throw 
-FAIL img.vspace: IDL set to 4294967295 followed by getAttribute() assert_equals: expected "0" but got "-1"
-FAIL img.vspace: IDL set to 4294967295 followed by IDL get assert_equals: expected 0 but got -1
+PASS img.vspace: IDL set to 4294967295 followed by getAttribute() 
+PASS img.vspace: IDL set to 4294967295 followed by IDL get 
 PASS img.longDesc: typeof IDL attribute 
 PASS img.longDesc: IDL get with DOM attribute unset 
 PASS img.longDesc: setAttribute() to "" followed by getAttribute() 
index 10b4901..38925b2 100644 (file)
@@ -1,5 +1,22 @@
 2016-09-08  Chris Dumez  <cdumez@apple.com>
 
+        HTMLImageElement.hspace / vspace attributes should be unsigned
+        https://bugs.webkit.org/show_bug.cgi?id=161754
+
+        Reviewed by Alex Christensen.
+
+        HTMLImageElement.hspace / vspace attributes should be unsigned:
+        - https://html.spec.whatwg.org/#HTMLImageElement-partial
+
+        Firefox agrees with the specification, however, those were signed in
+        webKit.
+
+        No new tests, rebaselined existing test.
+
+        * html/HTMLImageElement.idl:
+
+2016-09-08  Chris Dumez  <cdumez@apple.com>
+
         Update parseHTMLNonNegativeInteger() to return an unsigned value
         https://bugs.webkit.org/show_bug.cgi?id=161759
 
index 1c43067..40dc62c 100644 (file)
@@ -28,7 +28,7 @@
     [Reflect, TreatNullAs=EmptyString] attribute DOMString border;
     attribute DOMString? crossOrigin;
     attribute unsigned long height;
-    [Reflect] attribute long hspace;
+    [Reflect] attribute unsigned long hspace;
     [Reflect] attribute boolean isMap;
     [Reflect, URL] attribute USVString longDesc;
     [Reflect, URL] attribute USVString src;
@@ -36,7 +36,7 @@
     [Reflect] attribute DOMString sizes;
     readonly attribute USVString currentSrc;
     [Reflect] attribute DOMString useMap;
-    [Reflect] attribute long vspace;
+    [Reflect] attribute unsigned long vspace;
     attribute unsigned long width;
 
     // Extensions