Summary: | Add SPI for overriding WKWebView's clipping behavior for _visibleContentRect | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Quesada <david_quesada> | ||||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue, simon.fraser, thorton | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
David Quesada
2017-05-11 16:40:00 PDT
Created attachment 309821 [details]
Patch
Comment on attachment 309821 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=309821&action=review > Source/WebKit2/UIProcess/API/Cocoa/WKWebView.mm:2178 > +- (UIView *)_enclosingView I think this name is too generic; it could easily conflict with a UIKit method, and it has a very particular usage, so we should name it accordingly. How about _enclosingViewForExposedRectComputation? (In reply to Simon Fraser (smfr) from comment #2) > Comment on attachment 309821 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=309821&action=review > > > Source/WebKit2/UIProcess/API/Cocoa/WKWebView.mm:2178 > > +- (UIView *)_enclosingView > > I think this name is too generic; it could easily conflict with a UIKit > method, and it has a very particular usage, so we should name it accordingly. > > How about _enclosingViewForExposedRectComputation? Sure. Created attachment 309960 [details]
Patch v2
Comment on attachment 309960 [details] Patch v2 Clearing flags on attachment: 309960 Committed r216802: <http://trac.webkit.org/changeset/216802> All reviewed patches have been landed. Closing bug. |