21241 2008-09-30 07:29:13 -0700 REGRESSION (r36977): getRGBColorValue().red returning incorrect value 2008-10-08 17:29:29 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Mac OS X 10.5 RESOLVED FIXED P2 Normal --- 1 ap sam ggaren marchant sam zwarich oldest_to_newest 93363 0 ap 2008-09-30 07:29:13 -0700 I'm getting an assertion failure with today's ToT, see attached test case. 93364 1 23940 ap 2008-09-30 07:30:19 -0700 Created attachment 23940 test case (assertion failure) ASSERTION FAILED: getValue (/Users/ap/Safari/OpenSource/WebKitBuild/Debug/JavaScriptCore.framework/PrivateHeaders/PropertySlot.h:136 void JSC::PropertySlot::setCustom(JSC::JSValue*, JSC::JSValue* (*)(JSC::ExecState*, const JSC::Identifier&, const JSC::PropertySlot&))) 0 com.apple.WebCore 0x035dbb1b JSC::PropertySlot::setCustom(JSC::JSValue*, JSC::JSValue* (*)(JSC::ExecState*, JSC::Identifier const&, JSC::PropertySlot const&)) + 125 (PropertySlot.h:136) 1 com.apple.WebCore 0x039ab857 bool JSC::getStaticValueSlot<WebCore::JSRGBColor, WebCore::DOMObject>(JSC::ExecState*, JSC::HashTable const*, WebCore::JSRGBColor*, JSC::Identifier const&, JSC::PropertySlot&) + 197 (lookup.h:199) 2 com.apple.WebCore 0x039ab5d3 WebCore::JSRGBColor::getOwnPropertySlot(JSC::ExecState*, JSC::Identifier const&, JSC::PropertySlot&) + 55 (JSRGBColor.cpp:55) 3 com.apple.JavaScriptCore 0x008f09fb JSC::JSValue::get(JSC::ExecState*, JSC::Identifier const&, JSC::PropertySlot&) const + 187 (JSObject.h:430) 93894 2 ap 2008-10-03 07:58:14 -0700 OK, looks like this is just how bug 21235 looks in debug builds. 94595 3 zwarich 2008-10-08 13:33:04 -0700 This is also detectable in Release builds, because the actual value returned is the getter function, rather than a CSSPrimitiveValue. 94599 4 zwarich 2008-10-08 13:40:31 -0700 *** Bug 21235 has been marked as a duplicate of this bug. *** 94635 5 zwarich 2008-10-08 16:00:56 -0700 This is caused by r36977: http://trac.webkit.org/changeset/36977 I am assigning this to Sam, because he knows the fix. 94646 6 sam 2008-10-08 17:15:13 -0700 Fixed in 37432. 94650 7 marchant 2008-10-08 17:29:29 -0700 Thanks! 23940 2008-09-30 07:30:19 -0700 2008-09-30 07:30:19 -0700 test case (assertion failure) red-color.html text/html 217 ap PGJvZHk+CjxzcGFuIHN0eWxlPSJjb2xvcjojZjAwZmZmIj5mb288L3NwYW4+CjxzY3JpcHQ+CmFs ZXJ0KGRvY3VtZW50LmRlZmF1bHRWaWV3LmdldENvbXB1dGVkU3R5bGUoZG9jdW1lbnQuZ2V0RWxl bWVudHNCeVRhZ05hbWUoInNwYW4iKVswXSwgIiIpLmdldFByb3BlcnR5Q1NTVmFsdWUoImNvbG9y IikuZ2V0UkdCQ29sb3JWYWx1ZSgpLnJlZCk7Cjwvc2NyaXB0Pgo8L2JvZHk+Cg==