Bug 238338 – Web Inspector: Blank inspector in a page with container queries

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[jec]

Created attachment 455657 [details]
screenshot of blank inspector

TP 142

1. Open https://jec.fyi/demo/css-container-queries
2. Open Inspector
3. Inspector is blank [NOT OK]

Another page with CQ failed too: https://designcember.com

Expected Inspector to work as usual.

[Radar WebKit Bug Importer]

<rdar://problem/90779886>

[Patrick Angle]

Thank you for reporting this issue - the team in investigating!

[Patrick Angle]

Created attachment 455677 [details]
Patch v1.0

[Patrick Angle]

Comment on attachment 455677 [details]
Patch v1.0

Just want to note that even once this patch lands, Web Inspector will not actually show container query rules. That work is being tracked in bug 238346.

[Devin Rousso]

Comment on attachment 455677 [details]
Patch v1.0

r=me, unfortunate that we have so many unchecked dereferences :(

I did do a bit of digging, and I wonder if something like `window.getMatchedCSSRules(element)` will cause a crash if `element` has matching container queries, since `StyleRuleType::Container` is not handled inside `StyleRuleBase::createCSSOMWrapper`.  Might be worthing filing a separate bug and replacing that FIXME with it (and maybe including similar FIXME in relevant inspector code too).

[Patrick Angle]

(In reply to Devin Rousso from comment #5)
> Comment on attachment 455677 [details]
> Patch v1.0
> 
> r=me, unfortunate that we have so many unchecked dereferences :(
> 
> I did do a bit of digging, and I wonder if something like
> `window.getMatchedCSSRules(element)` will cause a crash if `element` has
> matching container queries, since `StyleRuleType::Container` is not handled
> inside `StyleRuleBase::createCSSOMWrapper`.  Might be worthing filing a
> separate bug and replacing that FIXME with it (and maybe including similar
> FIXME in relevant inspector code too).

I've just verified and there is no crash in that case, and the correct immediate rule is returned, but it's `parentRule` is null instead of being the actual container query rule.

[EWS]

Committed r291824 (248848@main): <https://commits.webkit.org/248848@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 455677 [details].