Closed Bug 1112700 Opened 10 years ago Closed 10 years ago

Browser Toolbox Inspector crash: Assertion failure: aPropID != eCSSPropertyExtra_variable

Categories

(Core :: CSS Parsing and Computation, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: jryans, Assigned: miker)

References

Details

(Keywords: crash, regression)

Attachments

(1 file)

STR: 1. Start a debug build of latest from fx-team[1] 2. Open the Browser Toolbox 3. Select the inspector tab [1]: http://hg.mozilla.org/integration/fx-team/rev/66cc38d90f25 Assertion failure: aPropID != eCSSPropertyExtra_variable, at ../../../layout/style/nsCSSParser.cpp:9521 #01: (anonymous namespace)::CSSParserImpl::ParseProperty(nsCSSProperty)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x37144cb] #02: (anonymous namespace)::CSSParserImpl::IsValueValidForProperty(nsCSSProperty, nsAString_internal const&)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x36d9b56] #03: nsCSSParser::IsValueValidForProperty(nsCSSProperty, nsAString_internal const&)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x36d9a11] #04: inDOMUtils::CssPropertyIsValid(nsAString_internal const&, nsAString_internal const&, bool*)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x3c26344] #05: NS_InvokeByIndex[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x19f142] #06: CallMethodHelper::Invoke()[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0xf43038] #07: CallMethodHelper::Call()[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0xf33107] #08: XPCWrappedNative::CallMethod(XPCCallContext&, XPCWrappedNative::CallMode)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0xf119c4] #09: XPC_WN_CallMethod(JSContext*, unsigned int, JS::Value*)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0xf13b98] #10: js::CallJSNative(JSContext*, bool (*)(JSContext*, unsigned int, JS::Value*), JS::CallArgs const&)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5c108f4] #11: js::Invoke(JSContext*, JS::CallArgs, js::MaybeConstruct)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5b905ad] #12: Interpret(JSContext*, js::RunState&)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5bb1603] #13: js::RunScript(JSContext*, js::RunState&)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5ba4cfa] #14: js::Invoke(JSContext*, JS::CallArgs, js::MaybeConstruct)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5b906f1] #15: js::InvokeConstructor(JSContext*, JS::CallArgs)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5bbcd15] #16: js::InvokeConstructor(JSContext*, JS::Value, unsigned int, JS::Value const*, JS::MutableHandle<JS::Value>)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x5bbcfc1] #17: js::jit::DoCallFallback(JSContext*, js::jit::BaselineFrame*, js::jit::ICCall_Fallback*, unsigned int, JS::Value*, JS::MutableHandle<JS::Value>)[/Users/jryans/projects/mozilla/gecko-dev-2/obj-firefox-debug/dist/NightlyDebug.app/Contents/MacOS/XUL +0x55cfd0b]
Seems like this is in heycam's wheelhouse, given that it's CSS-variable code.
Flags: needinfo?(cam)
(In reply to Daniel Holbert [:dholbert] from comment #1) > Seems like this is in heycam's wheelhouse, given that it's CSS-variable code. His wheelhouse, my code. In fact, this is best fixed from inDOMUtils.cpp... I am onto it.
Flags: needinfo?(cam)
No longer blocks: 1111601
Depends on: 1111601
Assignee: nobody → mratcliffe
CSS vars accept any value so returning true from DOMUtils.CssPropertyIsValid() is the obvious fix... this codepath must never have worked. r=me in the interests of getting the browser toolbox up and running again.
Attachment #8538387 - Flags: review+
Comment on attachment 8538387 [details] [diff] [review] 1112700-fix-assertion-aPropID-not-eCSSPropertyExtra_variable.patch https://hg.mozilla.org/integration/fx-team/rev/a24e89b3b206
Attachment #8538387 - Flags: checkin+
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: