Bug 238549 – InspectorCSSAgent::getSupportedCSSProperties() should use a real CSS parser context

• Status: NEW
• Component: Web Inspector

[Sam Sneddon [:gsnedders]]

Currently this calls:

CSSParserFastPaths::isValidKeywordPropertyAndValue(propertyID, valueID, strictCSSParserContext())

However, by using a strictCSSParserContext() we're actually calling this with basically everything that can be runtime-disabled disabled, which likely means we're missing some valid values.

[Patrick Angle]

If bug 217802 takes Devin's suggestion to keep a `Page& m_inspectedPage` in `InspectorCSSAgent`, I think we should be able to do something along the lines of `CSSParserContext(m_inspectedPage->mainFrame().document())` (with appropriate checks to fall back to the strict context if the page or document is missing) to get a parsing context for the main document.

[Sam Sneddon [:gsnedders]]

Also, as part of this, we should add to CSSParserFastPaths::isValidKeywordPropertyAndValue:

> ASSERT(isCSSPropertyExposed(propertyId, &context.propertySettings))

Or if that's still not possible (perhaps with `isInternalCSSProperty(propertyId)` if we need to), we should file another bug for that.

[Radar WebKit Bug Importer]

<rdar://problem/91350740>