[webkit-reviews] review granted: [Bug 171013] Expose obscured insets to web content (as "safe area insets") : [Attachment 307521] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Apr 19 17:32:35 PDT 2017

Wenson Hsieh <wenson_hsieh at apple.com> has granted Tim Horton
<thorton at apple.com>'s request for review:
Bug 171013: Expose obscured insets to web content (as "safe area insets")

Attachment 307521: Patch


--- Comment #4 from Wenson Hsieh <wenson_hsieh at apple.com> ---
Comment on attachment 307521
  --> https://bugs.webkit.org/attachment.cgi?id=307521

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


> Source/WebCore/dom/ConstantPropertyMap.cpp:96
> +void ConstantPropertyMap::didChangeObscuredInsets()

imo, updateObscuredInsetsFromPage would be a more informative name here, or
maybe just updateObscuredInsets, and then pass a const ref to the insets

> Source/WebCore/page/Page.cpp:2292
> +void Page::setObscuredInsets(FloatBoxExtent insets)

Nit - I think this can be a const &

> Source/WebCore/page/Page.h:333
> +    WEBCORE_EXPORT void setObscuredInsets(FloatBoxExtent);

Nit - const FloatBoxExtent&?

> Source/WebCore/page/Page.h:692
>      // This is only used for history scroll position restoration.

Is this comment still valid? (now that it's referring to

More information about the webkit-reviews mailing list