Changeset 100847 in webkit
- Timestamp:
- Nov 18, 2011 10:05:05 PM (12 years ago)
- Location:
- trunk
- Files:
-
- 7 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r100845 r100847 1 2011-11-18 Vineet Chaudhary <vineet.chaudhary@motorola.com> 2 3 https://bugs.webkit.org/show_bug.cgi?id=72591 4 Remove document.width / document.height 5 6 Reviewed by Darin Adler. 7 8 As support for document.width / document.height has been 9 removed it should use document.body.clientWidth and document.body.clientHeight 10 respectively. 11 12 * fast/dom/document-width-height-force-layout-expected.txt: 13 * fast/dom/document-width-height-force-layout.html: 14 Modified test case to use document.body.clientWidth and document.body.clientHeight. 15 * fullscreen/video-specified-size-expected.txt: 16 * fullscreen/video-specified-size.html: 17 Modified test case to use document.body.clientWidth. 18 1 19 2011-11-18 Yuta Kitamura <yutak@chromium.org> 2 20 -
trunk/LayoutTests/fast/dom/document-width-height-force-layout-expected.txt
r30635 r100847 1 This tests that document. width and document.height cause a layout on use. The test has passed if they have positive values.1 This tests that document.body.clientWidth and document.body.clientHeight cause a layout on use. The test has passed if they have positive values. 2 2 3 document. width = 8004 document. height = 6003 document.body.clientWidth = 800 4 document.body.clientHeight = 600 5 5 -
trunk/LayoutTests/fast/dom/document-width-height-force-layout.html
r25591 r100847 12 12 layoutTestController.dumpAsText(); 13 13 14 log("document. width = " + document.width);15 log("document. height = " + document.height);14 log("document.body.clientWidth = " + document.body.clientWidth); 15 log("document.body.clientHeight = " + document.body.clientHeight); 16 16 } 17 17 </script> 18 18 </head> 19 19 <body onload="test()"> 20 <p>This tests that document. width and document.height cause a layout on use. The test has passed if they have positive values.</p>20 <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> 21 21 <pre id="console"></pre> 22 22 </body> -
trunk/LayoutTests/fullscreen/video-specified-size-expected.txt
r83654 r100847 1 EVENT(webkitfullscreenchange) TEST(video.clientWidth==document. width) OK1 EVENT(webkitfullscreenchange) TEST(video.clientWidth==document.body.clientWidth) OK 2 2 END OF TEST 3 3 -
trunk/LayoutTests/fullscreen/video-specified-size.html
r83654 r100847 6 6 setSrcById("video", findMediaFile("video", "../media/content/test")); 7 7 var video = document.getElementById('video'); 8 9 8 // Bail out early if the full screen API is not enabled or is missing: 10 9 if (Element.prototype.webkitRequestFullScreen == undefined) { … … 12 11 endTest(); 13 12 } else { 14 waitForEventTestAndEnd(document, 'webkitfullscreenchange', "video.clientWidth==document. width");13 waitForEventTestAndEnd(document, 'webkitfullscreenchange', "video.clientWidth==document.body.clientWidth"); 15 14 runWithKeyDown(function(){video.webkitRequestFullScreen()}); 16 15 } -
trunk/Source/WebCore/ChangeLog
r100846 r100847 1 2011-11-18 Vineet Chaudhary <vineet.chaudhary@motorola.com> 2 3 https://bugs.webkit.org/show_bug.cgi?id=72591 4 Remove document.width / document.height 5 6 Reviewed by Darin Adler. 7 8 Removed document.width/document.height from JS bindings, 9 but keeping the same for ObjC bindings. Also it should use 10 document.body.clientWidth and document.body.clientHeight instead. 11 12 * html/HTMLDocument.idl: 13 1 14 2011-11-18 Martin Robinson <mrobinson@igalia.com> 2 15 -
trunk/Source/WebCore/html/HTMLDocument.idl
r92327 r100847 45 45 void releaseEvents(); 46 46 47 #if defined(LANGUAGE_OBJECTIVE_C) && LANGUAGE_OBJECTIVE_C 47 48 readonly attribute long width; 48 49 readonly attribute long height; 50 #endif 49 51 attribute [ConvertNullToNullString] DOMString dir; 50 52 attribute [ConvertNullToNullString] DOMString designMode;
Note: See TracChangeset
for help on using the changeset viewer.