https://bugs.webkit.org/show_bug.cgi?id=72591
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 19 Nov 2011 06:05:05 +0000 (06:05 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 19 Nov 2011 06:05:05 +0000 (06:05 +0000)
Remove document.width / document.height

Patch by Vineet Chaudhary <vineet.chaudhary@motorola.com> on 2011-11-18
Reviewed by Darin Adler.

Source/WebCore:

Removed document.width/document.height from JS bindings,
but keeping the same for ObjC bindings. Also it should use
document.body.clientWidth and document.body.clientHeight instead.

* html/HTMLDocument.idl:

LayoutTests:

As support for document.width / document.height has been
removed it should use document.body.clientWidth and document.body.clientHeight
respectively.

* fast/dom/document-width-height-force-layout-expected.txt:
* fast/dom/document-width-height-force-layout.html:
Modified test case to use document.body.clientWidth and document.body.clientHeight.
* fullscreen/video-specified-size-expected.txt:
* fullscreen/video-specified-size.html:
Modified test case to use document.body.clientWidth.

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

LayoutTests/ChangeLog
LayoutTests/fast/dom/document-width-height-force-layout-expected.txt
LayoutTests/fast/dom/document-width-height-force-layout.html
LayoutTests/fullscreen/video-specified-size-expected.txt
LayoutTests/fullscreen/video-specified-size.html
Source/WebCore/ChangeLog
Source/WebCore/html/HTMLDocument.idl

index 76e9f31..1f1a827 100644 (file)
@@ -1,3 +1,21 @@
+2011-11-18  Vineet Chaudhary  <vineet.chaudhary@motorola.com>
+
+        https://bugs.webkit.org/show_bug.cgi?id=72591
+        Remove document.width / document.height
+
+        Reviewed by Darin Adler.
+
+        As support for document.width / document.height has been
+        removed it should use document.body.clientWidth and document.body.clientHeight
+        respectively.
+
+        * fast/dom/document-width-height-force-layout-expected.txt:
+        * fast/dom/document-width-height-force-layout.html:
+        Modified test case to use document.body.clientWidth and document.body.clientHeight.
+        * fullscreen/video-specified-size-expected.txt:
+        * fullscreen/video-specified-size.html:
+        Modified test case to use document.body.clientWidth.
+
 2011-11-18  Yuta Kitamura  <yutak@chromium.org>
 
         [Qt] Unreviewed, add one failing test to the Skipped list.
index 0f3d5d0..931a296 100644 (file)
@@ -1,5 +1,5 @@
-This tests that document.width and document.height cause a layout on use. The test has passed if they have positive values.
+This tests that document.body.clientWidth and document.body.clientHeight cause a layout on use. The test has passed if they have positive values.
 
-document.width = 800
-document.height = 600
+document.body.clientWidth = 800
+document.body.clientHeight = 600
 
index 956b377..e438840 100644 (file)
             if (window.layoutTestController)
                 layoutTestController.dumpAsText();
 
-            log("document.width = " + document.width);
-            log("document.height = " + document.height);
+            log("document.body.clientWidth = " + document.body.clientWidth);
+            log("document.body.clientHeight = " + document.body.clientHeight);
         }
     </script>
 </head>
 <body onload="test()">
-    <p>This tests that document.width and document.height cause a layout on use.  The test has passed if they have positive values.</p>
+    <p>This tests that document.body.clientWidth and document.body.clientHeight cause a layout on use.  The test has passed if they have positive values.</p>
     <pre id="console"></pre>
 </body>
 </html>
index 6b8700d..19bff9f 100644 (file)
@@ -1,3 +1,3 @@
-EVENT(webkitfullscreenchange) TEST(video.clientWidth==document.width) OK
+EVENT(webkitfullscreenchange) TEST(video.clientWidth==document.body.clientWidth) OK
 END OF TEST
 
index 818706e..fadc343 100644 (file)
@@ -5,14 +5,13 @@
     <script>
         setSrcById("video", findMediaFile("video", "../media/content/test"));
         var video = document.getElementById('video');
-
         // Bail out early if the full screen API is not enabled or is missing:
         if (Element.prototype.webkitRequestFullScreen == undefined) {
             logResult(false, "Element.prototype.webkitRequestFullScreen == undefined");
             endTest();
         } else {
-            waitForEventTestAndEnd(document, 'webkitfullscreenchange', "video.clientWidth==document.width");
+            waitForEventTestAndEnd(document, 'webkitfullscreenchange', "video.clientWidth==document.body.clientWidth");
             runWithKeyDown(function(){video.webkitRequestFullScreen()});
         }
     </script>
-</body>
\ No newline at end of file
+</body>
index 0513bc6..0397639 100644 (file)
@@ -1,3 +1,16 @@
+2011-11-18  Vineet Chaudhary  <vineet.chaudhary@motorola.com>
+
+        https://bugs.webkit.org/show_bug.cgi?id=72591
+        Remove document.width / document.height
+
+        Reviewed by Darin Adler.
+
+        Removed document.width/document.height from JS bindings,
+        but keeping the same for ObjC bindings. Also it should use
+        document.body.clientWidth and document.body.clientHeight instead.
+
+        * html/HTMLDocument.idl:
+
 2011-11-18  Martin Robinson  <mrobinson@igalia.com>
 
         Fix the GTK+ build.
index 36ce0a2..d2400a8 100644 (file)
@@ -44,8 +44,10 @@ module html {
         void captureEvents();
         void releaseEvents();
 
+#if defined(LANGUAGE_OBJECTIVE_C) && LANGUAGE_OBJECTIVE_C
         readonly attribute long width;
         readonly attribute long height;
+#endif
                  attribute [ConvertNullToNullString] DOMString dir;
                  attribute [ConvertNullToNullString] DOMString designMode;
         readonly attribute DOMString compatMode;