Closed Bug 1112700 Opened 7 years ago Closed 7 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+
https://hg.mozilla.org/mozilla-central/rev/a24e89b3b206
Status: NEW → RESOLVED
Closed: 7 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.