[Webkit-unassigned] [Bug 238661] Web Inspector: Use computedStyleMap() instead of deprecated getPropertyCSSValue()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=238661

--- Comment #7 from Devin Rousso <drousso at apple.com> ---
Comment on attachment 456340
  --> https://bugs.webkit.org/attachment.cgi?id=456340
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=456340&action=review

>>>>> Source/WebKit/WebProcess/Inspector/WebInspectorUI.cpp:65
>>>>> +    page.corePage()->settings().setCSSTypedOMEnabled(true);
>>>> 
>>>> Since this is not something enabled by default, we should ensure that the frontend doesn't break if this is disabled.  Could we create a helper function that still uses the old behavior if this isn't enabled?  Something like:
>>>> ```
>>>> Object.defineProperty(Element.prototype, "getComputedCSSPropertyNumberValue", {
>>>>     value(property) {
>>>>         let result = undefined;
>>>>         result ??= this.computedStyleMap?.().get(property)?.value;
>>>>         result ??= window.getComputedStyle(this).getPropertyCSSValue(property)?.getFloatValue(CSSPrimitiveValue.CSS_PX);
>>>>         return result;   
>>>>     },
>>>> });
>>>> ```
>>> 
>>> Doesn't the setting ensure it's always enabled in the inspector frontend? or do you mean adding the helper instead of changing the setting?
>> 
>> Oh sorry I wasn't entirely clear.  This relies on *both* the setting and `ENABLE_CSS_TYPED_OM`.  I was more concerned with about if `ENABLE_CSS_TYPED_OM` is not enabled (for whatever reason).  If/When the build flag is removed, then we can remove the fallback code.
> 
> I'm not familiar with xcode build system, but I think CSS_TYPED_OM is enabled unconditionally in PlatformEnableCocoa.h (or can that be overridden with xcode magic?). And I enabled it by default for GTK and WPE ports too.

Yes, but on cocoa platforms that's predicated by `#if !defined(ENABLE_CSS_TYPED_OM)`.  I admit it's probably unlikely that some build system would override this, but in the off chance that something does I'd hate for Web Inspector to be broken in that situation.  We do something similar for other things that are guarded by flags (e.g. `WebGLRenderingContext`, `indexedDB`, etc.), just in case :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220406/6f8f6729/attachment.htm>