| Summary: | Add bundle SPI to call WebPage::setTopContentInset | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Alex Christensen <achristensen> | ||||
| Component: | New Bugs | Assignee: | Alex Christensen <achristensen> | ||||
| Status: | NEW --- | ||||||
| Severity: | Normal | CC: | simon.fraser, thorton, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Alex Christensen
2020-04-14 11:20:32 PDT
Created attachment 396442 [details]
Patch
Comment on attachment 396442 [details]
Patch
Please explain how this interacts with WebViewImpl::updateContentInsetsIfAutomatic() - m_automaticallyAdjustsContentInsets is on by default in WKWebView. That's called from WebViewImpl::renewGStateI(), which is called whenever AppKit jiggles the view hierarchy, which is a lot.
When this is called we could also send a message to the WebPageProxy to update it. Would that resolve your concerns? What does it /mean/, though? Does automaticallyAdjustsContentInsets become NO, too? It's all very weird. But definitely the UIProcess needs to know when it changes, yes. |