6693 2006-01-20 18:35:48 -0800 offsetHeight returns 0, where IE does not. 2022-08-02 09:00:41 -0700 1 1 1 Unclassified WebKit DOM 420+ Mac OS X 10.4 RESOLVED CONFIGURATION CHANGED http://cnx.rice.edu/content/m11268/latest/ HasReduction, InRadar P2 Major --- 7080 1 alice.barraclough webkit-unassigned ahmad.saleem792 ap bfulgham darin emacemac7 ian joost krishnamurty.podipireddy mitz rniwa webkitbugzilla oldest_to_newest 29046 0 alice.barraclough 2006-01-20 18:35:48 -0800 This bug is also in Radar as <rdar://4415273> Summary: Safari hangs when visiting http://cnx.rice.edu/content/m11268/latest/ Steps to Reproduce: 0. MacOS 10.4.4 (8G32) 1. Launch Safari 2. Type "http://cnx.rice.edu/content/m11268/latest/" into address bar and press return Expected Results: Safari should show the page (or at least make an effort) Actual Results: Safari hangs -- must force quit. Regression: Didn't regress. Notes: Firefox 1.5 does not hang on this page, though it does report some missing fonts. 'pwp-pb15.spx' was successfully uploaded ------------------------------------------- <GMT19-Jan-2006 21:54:53GMT> Paul Placeway: Please assign to Developer ADCBugs for further communications with the developer. <GMT21-Jan-2006 02:20:07GMT> Alice Liu: Denver and Glendale hang for long periods. TOT hangs (see attached sample) with KJS::Collector::markStackObjectsConservatively(void*, void*) KJS::SimpleNumber::is(KJS::JSValue const*) at the top when viewed with HotSpotFinder, and then after a while crashes with this trace: Thread 0 Crashed: 0 <<00000000>> 0xffff87c4 __memcpy + 36 (cpu_capabilities.h:189) 1 com.apple.JavaScriptCore 0x015f6dc4 KJS::UString::UString[in-charge](KJS::UString const&, KJS::UString const&) + 948 (ustring.cpp:474) 2 com.apple.JavaScriptCore 0x0163e7ac KJS::operator+(KJS::UString const&, KJS::UString const&) + 52 (ustring.h:500) 3 com.apple.JavaScriptCore 0x015e8fdc KJS::add(KJS::ExecState*, KJS::JSValue*, KJS::JSValue*, char) + 292 (operations.cpp:225) 4 com.apple.JavaScriptCore 0x015d7b28 KJS::AddNode::evaluate(KJS::ExecState*) + 356 (nodes.cpp:1056) 5 com.apple.JavaScriptCore 0x015e10e0 KJS::AssignResolveNode::evaluate(KJS::ExecState*) + 640 (nodes.cpp:1317) 6 com.apple.JavaScriptCore 0x015d6b3c KJS::ExprStatementNode::execute(KJS::ExecState*) + 220 (nodes.cpp:1605) 7 com.apple.JavaScriptCore 0x015dbd14 KJS::ForNode::execute(KJS::ExecState*) + 1036 (nodes.cpp:1745) 8 com.apple.JavaScriptCore 0x015d59f4 KJS::SourceElementsNode::execute(KJS::ExecState*) + 616 (nodes.cpp:2335) 9 com.apple.JavaScriptCore 0x015d424c KJS::BlockNode::execute(KJS::ExecState*) + 216 (nodes.cpp:1582) 10 com.apple.JavaScriptCore 0x015dcd8c KJS::IfNode::execute(KJS::ExecState*) + 500 (nodes.cpp:1624) 11 com.apple.JavaScriptCore 0x015d59f4 KJS::SourceElementsNode::execute(KJS::ExecState*) + 616 (nodes.cpp:2335) 12 com.apple.JavaScriptCore 0x015d424c KJS::BlockNode::execute(KJS::ExecState*) + 216 (nodes.cpp:1582) 13 com.apple.JavaScriptCore 0x015bc218 KJS::DeclaredFunctionImp::execute(KJS::ExecState*) + 92 (function.cpp:339) 14 com.apple.JavaScriptCore 0x015bb914 KJS::FunctionImp::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&) + 700 (function.cpp:110) 15 com.apple.JavaScriptCore 0x015e7df0 KJS::JSObject::call(KJS::ExecState*, KJS::JSObject*, KJS::List const&) + 288 (object.cpp:96) 16 com.apple.JavaScriptCore 0x015df7c4 KJS::FunctionCallResolveNode::evaluate(KJS::ExecState*) + 820 (nodes.cpp:570) 17 com.apple.JavaScriptCore 0x015d6b3c KJS::ExprStatementNode::execute(KJS::ExecState*) + 220 (nodes.cpp:1605) 18 com.apple.JavaScriptCore 0x015d59f4 KJS::SourceElementsNode::execute(KJS::ExecState*) + 616 (nodes.cpp:2335) 19 com.apple.JavaScriptCore 0x015d424c KJS::BlockNode::execute(KJS::ExecState*) + 216 (nodes.cpp:1582) 20 com.apple.JavaScriptCore 0x015cbd68 KJS::InterpreterImp::evaluate(KJS::UChar const*, int, KJS::JSValue*, KJS::UString const&, int) + 1028 (internal.cpp:682) 21 com.apple.JavaScriptCore 0x015cc5ec KJS::Interpreter::evaluate(KJS::UString const&, int, KJS::UChar const*, int, KJS::JSValue*) + 100 (interpreter.cpp:121) 22 com.apple.WebCore 0x01ca2aac KJSProxyImpl::evaluate(WebCore::DOMString const&, int, WebCore::DOMString const&, WebCore::NodeImpl*) + 296 (kjs_proxy.cpp:63) 23 com.apple.WebCore 0x01e381fc Frame::executeScript(QString, int, WebCore::NodeImpl*, QString const&) + 160 (Frame.cpp:2521) 24 com.apple.WebCore 0x01cecd94 WebCore::HTMLTokenizer::scriptExecution(QString const&, WebCore::HTMLTokenizer::State, QString, int) + 496 (htmltokenizer.cpp:485) 25 com.apple.WebCore 0x01cefc2c WebCore::HTMLTokenizer::scriptHandler(WebCore::HTMLTokenizer::State) + 1568 (htmltokenizer.cpp:425) 26 com.apple.WebCore 0x01cf030c WebCore::HTMLTokenizer::parseSpecial(WebCore::SegmentedString&, WebCore::HTMLTokenizer::State) + 1292 (htmltokenizer.cpp:292) 27 com.apple.WebCore 0x01cf2810 WebCore::HTMLTokenizer::parseTag(WebCore::SegmentedString&, WebCore::HTMLTokenizer::State) + 7732 (htmltokenizer.cpp:1262) 28 com.apple.WebCore 0x01cf3220 WebCore::HTMLTokenizer::write(WebCore::SegmentedString const&, bool) + 1776 (htmltokenizer.cpp:1467) 29 com.apple.WebCore 0x01ced2b0 WebCore::HTMLTokenizer::notifyFinished(WebCore::CachedObject*) + 816 (htmltokenizer.cpp:1784) 30 com.apple.WebCore 0x01e944e0 WebCore::CachedScript::checkNotify() + 140 (CachedScript.cpp:111) 31 com.apple.WebCore 0x01e94670 WebCore::CachedScript::data(QBuffer&, bool) + 276 (CachedScript.cpp:103) 32 com.apple.WebCore 0x01e97368 WebCore::Loader::slotFinished(KIO::Job*, NSData*) + 804 (loader.cpp:168) 33 com.apple.WebCore 0x01d1d368 KWQSlot::callWithData(KIO::Job*, NSData*) const + 108 (KWQSlot.cpp:320) 34 com.apple.WebCore 0x01d1be74 KWQSignal::callWithData(KIO::Job*, NSData*) const + 232 (KWQSignal.cpp:183) 35 com.apple.WebCore 0x01c14898 KIO::TransferJob::emitResult(NSData*) + 72 (KWQKJobClasses.mm:242) 36 com.apple.WebCore 0x01d20440 -[KWQResourceLoader finishJobAndHandle:] + 128 (KWQResourceLoader.mm:95) 37 com.apple.WebCore 0x01d206ec -[KWQResourceLoader finishWithData:] + 200 (KWQResourceLoader.mm:126) 38 com.apple.WebKit 0x01242444 -[WebSubresourceLoader didFinishLoading] + 132 (WebSubresourceLoader.m:218) 39 com.apple.WebKit 0x01251304 -[WebLoader connectionDidFinishLoading:] + 184 (WebLoader.m:663) 40 com.apple.Foundation 0x90b4acdc -[NSURLConnection(NSURLConnectionInternal) _sendDidFinishLoadingCallback] + 188 41 com.apple.Foundation 0x90b48f48 -[NSURLConnection(NSURLConnectionInternal) _sendCallbacks] + 556 42 com.apple.Foundation 0x90b48ca0 _sendCallbacks + 156 43 com.apple.CoreFoundation 0x902b0a68 __CFRunLoopDoSources0 + 384 44 com.apple.CoreFoundation 0x902aff98 __CFRunLoopRun + 452 45 com.apple.CoreFoundation 0x902afa18 CFRunLoopRunSpecific + 268 46 com.apple.HIToolbox 0x920531e0 RunCurrentEventLoopInMode + 264 47 com.apple.HIToolbox 0x92052874 ReceiveNextEventCommon + 380 48 com.apple.HIToolbox 0x920526e0 BlockUntilNextEventMatchingListInMode + 96 49 com.apple.AppKit 0x9253f104 _DPSNextEvent + 384 50 com.apple.AppKit 0x9253edc8 -[NSApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 116 51 com.apple.Safari 0x00030ec4 -[BrowserApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 292 (BrowserApplication.m:152) 52 com.apple.AppKit 0x9253b30c -[NSApplication run] + 472 53 com.apple.AppKit 0x9262be68 NSApplicationMain + 452 54 com.apple.Safari 0x000f7d90 main + 156 (main.m:23) 55 com.apple.Safari 0x00002814 _start + 344 (crt.c:272) 56 com.apple.Safari 0x000026b8 start + 60 29283 1 5819 mitz 2006-01-21 14:13:44 -0800 Created attachment 5819 OffsetHeight test The given page enters an infinite for loop at http://cnx.rice.edu/pmathmlcss.js:35 for ( i = 3; i <= mrowH / (2*opH) ; i += 1) since opH is zero. opH comes from taking the offsetHeight of an empty <span>. FireFox also gives zero for the height of an empty <span>, but doesn't reach that code at all since it doesn't support accessing DOM nodes as JS variables by id. WinIE gives a positive height even for the empty span. 31310 2 darin 2006-02-04 09:36:25 -0800 I know this is a hang, but we don't need separate P1 bugs for each page where there's an infinite JavaScript loop. There are two ways to look at this: 1) we need to "finesse" this difference from WinIE so we don't hang on this page 2) we need to make sure that infinite JavaScript loops don't hang the entire browser Item (2) is something we're planning to do as part of the "tree code" project. It's not clear how to do (1) at all. I think I'd like to bump this down to P2 soon. 31421 3 joost 2006-02-05 01:20:23 -0800 Bug 7080 has been filed for the infinite loop part, renaming this one to track how we handle offsetHeight. Lowering to p2 in the process. 503973 4 115394 webkitbugzilla 2011-11-16 09:35:20 -0800 Created attachment 115394 Prepares an alert message showing the offset discrepency between the same anchor tag vertical offset when inline and when set with display:block 503977 5 webkitbugzilla 2011-11-16 09:40:37 -0800 I'd like to see this one fixed. It also impacts empty anchor elements. Positioning elements programmatically in relation to empty anchor elements is basic and necessary functionality. I'm including a related thread here which may shed additional light on the issue: http://stackoverflow.com/questions/8126648/how-to-get-the-correct-offset-top-value-from-webkit-chrome-safari-with-jquery/8126822#8126822 Thanks for your help! 503988 6 darin 2011-11-16 09:47:52 -0800 The bug is about offsetHeight, but that last comment seems to be about offsetTop. Are you sure this is the same issue? 504023 7 webkitbugzilla 2011-11-16 10:24:48 -0800 (In reply to comment #6) > The bug is about offsetHeight, but that last comment seems to be about offsetTop. Are you sure this is the same issue? Thanks for the head up! Sorry, I should have seen that. I'll search for the correct place for this update or log a new bug. 504241 8 webkitbugzilla 2011-11-16 14:16:57 -0800 (In reply to comment #6) > The bug is about offsetHeight, but that last comment seems to be about offsetTop. Are you sure this is the same issue? Okay, I did another search and didn't find this exact issue. I logged the following bug: https://bugs.webkit.org/show_bug.cgi?id=72524 1888378 9 ahmad.saleem792 2022-08-02 06:52:42 -0700 On "OffsetHeight test" - all browsers: *** Safari 15.6 on macOS 12.5 *** Empty span offsetHeight: 18 Non-empty span offsetHeight: 18 *** Chrome Canary 106 *** Empty span offsetHeight: 19 Non-empty span offsetHeight: 19 *** Firefox Nightly 105 *** Empty span offsetHeight: 0 Non-empty span offsetHeight: 16 _____ On "Prepares..." - all browsers: *** Safari 15.6 on macOS 12.5 *** displayCss=[inline] alteredOffset.top=[427.8125] offsetAlert(calledBy=[jQuery.ready]) anchor.offset().top=[427.8125] subheading.offset().top=[427.8125] displayCss=[inline] alteredOffset.top=[427.8125] offsetAlert(calledBy=[body.onload]) anchor.offset().top=[427.8125] subheading.offset().top=[427.8125] *** Chrome Canary 106 *** displayCss=[inline] alteredOffset.top=[437.328125] offsetAlert(calledBy=[jQuery.ready]) anchor.offset().top=[437.328125] subheading.offset().top=[437.328125] displayCss=[inline] alteredOffset.top=[437.328125] offsetAlert(calledBy=[body.onload]) anchor.offset().top=[437.328125] subheading.offset().top=[437.328125] *** Firefox Nightly 105 *** displayCss=[inline] alteredOffset.top=[452.6333312988281] offsetAlert(calledBy=[jQuery.ready]) anchor.offset().top=[454.23333740234375] subheading.offset().top=[454.23333740234375] displayCss=[inline] alteredOffset.top=[452.6333312988281] offsetAlert(calledBy=[body.onload]) anchor.offset().top=[454.23333740234375] subheading.offset().top=[454.23333740234375] _________ Just wanted to share updated test results. Thanks! 1888404 10 rniwa 2022-08-02 09:00:41 -0700 Looks like it's working now. 5819 2006-01-21 14:13:44 -0800 2006-01-21 14:13:44 -0800 OffsetHeight test 6693 testcase.html text/html 253 mitz PHNwYW4gaWQ9InMiPjwvc3Bhbj48YnI+PHNwYW4gaWQ9InQiPng8L3NwYW4+Cjxicj4KPHNjcmlw dCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgogICAgZG9jdW1lbnQud3JpdGUoJ0VtcHR5IHNwYW4g b2Zmc2V0SGVpZ2h0OiAnK3Mub2Zmc2V0SGVpZ2h0KTsKICAgIGRvY3VtZW50LndyaXRlKCc8YnI+ Jyk7CiAgICBkb2N1bWVudC53cml0ZSgnTm9uLWVtcHR5IHNwYW4gb2Zmc2V0SGVpZ2h0OiAnK3Qu b2Zmc2V0SGVpZ2h0KTsKPC9zY3JpcHQ+Cg== 115394 2011-11-16 09:35:20 -0800 2011-11-16 09:35:20 -0800 Prepares an alert message showing the offset discrepency between the same anchor tag vertical offset when inline and when set with display:block example.html text/html 2188 webkitbugzilla PCFET0NUWVBFIEhUTUwgUFVCTElDICItLy9XM0MvL0RURCBIVE1MIDQuMDEvL0VOIiAiaHR0cDov L3d3dy53My5vcmcvVFIvaHRtbDQvc3RyaWN0LmR0ZCI+DQo8aHRtbD4NCjxoZWFkPg0KCTx0aXRs ZT5XaGF0J3MgVXAgd2l0aCBXZWJraXQgQW5jaG9yIG9mZnNldCgpPzwvdGl0bGU+DQoJPG1ldGEg aHR0cC1lcXVpdj0iQ29udGVudC10eXBlIiBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9VVRG LTgiPg0KCTxzY3JpcHQgdHlwZT0idGV4dC9qYXZhc2NyaXB0IiBzcmM9Imh0dHBzOi8vYWpheC5n b29nbGVhcGlzLmNvbS9hamF4L2xpYnMvanF1ZXJ5LzEuNy4wL2pxdWVyeS5taW4uanMiID48L3Nj cmlwdD4NCgk8c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlwdCI+DQoJCXZhciBvZmZzZXRBbGVy dCA9IGZ1bmN0aW9uKGNhbGxlZEJ5KXsNCgkJCXZhciBhbmNob3IgPSBqUXVlcnkoImEjYW5jaG9y Iik7DQoJCQl2YXIgc3ViaGVhZGluZyA9IGpRdWVyeSgic3BhbiN0YXJnZXRTdWJIZWFkaW5nIik7 DQoJCQl2YXIgZGlzcGxheUNzcyA9IGFuY2hvci5jc3MoImRpc3BsYXkiKTsNCgkJCWFuY2hvci5j c3MoImRpc3BsYXkiLCAiYmxvY2siKTsNCgkJCXZhciBhbHRlcmVkT2Zmc2V0ID0gYW5jaG9yLm9m ZnNldCgpOw0KCQkJYW5jaG9yLmNzcygiZGlzcGxheSIsIChkaXNwbGF5Q3NzID8gZGlzcGxheUNz cyA6ICIiKSk7DQoJCQlpZiAoYW5jaG9yLmxlbmd0aCA9PSAxICYmIHN1YmhlYWRpbmcubGVuZ3Ro ID09IDEpDQoJCQl7DQoJCQkJYWxlcnQoImRpc3BsYXlDc3M9WyIrZGlzcGxheUNzcysiXVxuYWx0 ZXJlZE9mZnNldC50b3A9WyIrYWx0ZXJlZE9mZnNldC50b3ArIl1cbm9mZnNldEFsZXJ0KGNhbGxl ZEJ5PVsiK2NhbGxlZEJ5KyJdKVxuXHRhbmNob3Iub2Zmc2V0KCkudG9wPVsiK2FuY2hvci5vZmZz ZXQoKS50b3ArIl1cblxuc3ViaGVhZGluZy5vZmZzZXQoKS50b3A9WyIrc3ViaGVhZGluZy5vZmZz ZXQoKS50b3ArIl0iKTsNCgkJCX0NCgkJfTsNCg0KCQkkKGRvY3VtZW50KS5yZWFkeShmdW5jdGlv biAoKXtvZmZzZXRBbGVydCgialF1ZXJ5LnJlYWR5Iik7fSk7DQoJPC9zY3JpcHQ+DQo8L2hlYWQ+ DQo8Ym9keSBzdHlsZT0iYmFja2dyb3VuZC1jb2xvcjojYzZjYmQxOyBtYXJnaW46MHB4OyBwYWRk aW5nOjBweDsgYm9yZGVyOm5vbmU7IiBvbmxvYWQ9ImphdmFzY3JpcHQ6b2Zmc2V0QWxlcnQoJ2Jv ZHkub25sb2FkJyk7Ij4NCgk8ZGl2Pg0KCQk8YSBuYW1lPSJ0b3AiPjwvYT48aDI+SGVhZGluZzwv aDI+DQoJCVRoaXMgaXMgdGhlIHBoZW5vbWVtYWx5IGludGVyZXN0aW5nIHRleHQuDQoJCTxiciA+ PGJyID5UaGlzIGlzIHRoZSBwaGVub21lbWFseSBpbnRlcmVzdGluZyB0ZXh0Lg0KCQk8YnIgPjxi ciA+VGhpcyBpcyB0aGUgcGhlbm9tZW1hbHkgaW50ZXJlc3RpbmcgdGV4dC4NCgkJPGJyID48YnIg PkludGVybmFsIExpbmtzOiA8YSBocmVmPSIjYTEiPlN1Yi1oZWFkaW5nIDE8L2E+LCA8YSBocmVm PSIjYTIiPlN1Yi1oZWFkaW5nIDI8L2E+LCA8YSBocmVmPSIjYTMiPlN1Yi1oZWFkaW5nIDM8L2E+ LCA8YSBocmVmPSIjYW5jaG9yIj5UYXJnZXQgU3ViLWhlYWRpbmc8L2E+DQoJCTxiciA+PGJyID48 Yj48YSBpZD0iYTEiPjwvYT5TdWItaGVhZGluZyAxPC9iPg0KCQk8YnIgPlRoaXMgaXMgdGhlIHBo ZW5vbWVtYWx5IGludGVyZXN0aW5nIHRleHQuIA0KCQk8YnI+PGEgaHJlZj0iI3RvcCI+dG9wPC9h Pg0KCQk8YnIgPjxiciA+PGEgaWQ9ImEyIj48L2E+PGI+U3ViLWhlYWRpbmcgMjwvYj4NCgkJPGJy ID5UaGlzIGlzIHRoZSBwaGVub21lbWFseSBpbnRlcmVzdGluZyB0ZXh0Lg0KCQk8YnI+PGEgaHJl Zj0iI3RvcCI+dG9wPC9hPg0KCQk8YnIgPjxiciA+PGI+PGEgaWQ9ImEzIj48L2E+U3ViLWhlYWRp bmcgMzwvYj4NCgkJPGJyID5UaGlzIGlzIHRoZSBwaGVub21lbWFseSBpbnRlcmVzdGluZyB0ZXh0 Lg0KCQk8YnI+PGEgaHJlZj0iI3RvcCI+dG9wPC9hPg0KCQk8YnIgPjxiciA+PGEgaWQ9ImFuY2hv ciI+PC9hPjxzcGFuIGlkPSJ0YXJnZXRTdWJIZWFkaW5nIiBzdHlsZT0iZm9udC13ZWlnaHQ6Ym9s ZDsiPlRhcmdldCBTdWItaGVhZGluZzwvc3Bhbj4NCgkJPGJyID5UaGlzIGlzIHRoZSBwaGVub21l bWFseSBpbnRlcmVzdGluZyB0ZXh0Lg0KCQk8YnI+PGEgaHJlZj0iI3RvcCI+dG9wPC9hPg0KCTwv ZGl2Pg0KPC9ib2R5Pg0KPC9odG1sPg==