RESOLVED FIXED 150106
_WKThumbnailView should support a snapshot-only mode
https://bugs.webkit.org/show_bug.cgi?id=150106
Summary _WKThumbnailView should support a snapshot-only mode
Conrad Shultz
Reported 2015-10-13 16:23:09 PDT
Currently _WKThumbnailView works by reparenting the layer tree of the associated WKView, then snapshotting it. It would be helpful to have a mode where it can simply snapshot the WKView without impacting the layer tree or blocking interaction with the WKView.
Attachments
Patch (5.81 KB, patch)
2015-10-14 13:51 PDT, Conrad Shultz
thorton: review+
Radar WebKit Bug Importer
Comment 1 2015-10-13 16:23:59 PDT
Conrad Shultz
Comment 2 2015-10-14 13:51:05 PDT
Tim Horton
Comment 3 2015-10-14 14:01:15 PDT
Comment on attachment 263105 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263105&action=review > Source/WebKit2/UIProcess/API/Cocoa/_WKThumbnailView.h:42 > +@property (nonatomic) BOOL exclusivelyUsesSnapshot; I don't know if this is how we should word this, because if e.g. we managed to have a layer tree in two places at once, or something crazy like that, we could get the behavior you want (WKView left undisturbed) without sacrificing the instantaneous-ness/no-flashyness of WKThumbnailView. Maybe it should be about the difference in the WKView's behavior?
Conrad Shultz
Comment 4 2015-10-14 14:55:08 PDT
(In reply to comment #3) > Comment on attachment 263105 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=263105&action=review > > > Source/WebKit2/UIProcess/API/Cocoa/_WKThumbnailView.h:42 > > +@property (nonatomic) BOOL exclusivelyUsesSnapshot; > > I don't know if this is how we should word this, because if e.g. we managed > to have a layer tree in two places at once, or something crazy like that, we > could get the behavior you want (WKView left undisturbed) without > sacrificing the instantaneous-ness/no-flashyness of WKThumbnailView. > > Maybe it should be about the difference in the WKView's behavior? That's a good point, but I am having trouble thinking up a good name. allowsInteractionWithWKView? canCoexistWithWKView?
Conrad Shultz
Comment 5 2015-10-20 18:11:11 PDT
Note You need to log in before you can comment on or make changes to this bug.