Skip to content

Commit a2ae7ea

Browse files
ExplicitSnapshotsChangeUponNavigation API test often fails
https://bugs.webkit.org/show_bug.cgi?id=172475 <rdar://problem/31688322> Reviewed by Simon Fraser. * TestWebKitAPI/cocoa/TestWKWebView.mm: (-[TestWKWebView _setUpTestWindow:]): Make TestWKWebView's window layer-backed. This is now the default, and it's also easy to accidentally write a test that depends on it (for example, ExplicitSnapshotsChangeUponNavigation adds an NSBox, which becomes layer backed, just like the WKWebView, but with no common layer-backed ancestor). git-svn-id: http://svn.webkit.org/repository/webkit/trunk@217249 268f45cc-cd09-0410-ab3c-d52691b4dbfc
1 parent 51fd262 commit a2ae7ea

File tree

2 files changed

+17
-0
lines changed

2 files changed

+17
-0
lines changed

Tools/ChangeLog

+16
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,19 @@
1+
2017-05-22 Tim Horton <[email protected]>
2+
3+
ExplicitSnapshotsChangeUponNavigation API test often fails
4+
https://bugs.webkit.org/show_bug.cgi?id=172475
5+
<rdar://problem/31688322>
6+
7+
Reviewed by Simon Fraser.
8+
9+
* TestWebKitAPI/cocoa/TestWKWebView.mm:
10+
(-[TestWKWebView _setUpTestWindow:]):
11+
Make TestWKWebView's window layer-backed. This is now the default,
12+
and it's also easy to accidentally write a test that depends on it
13+
(for example, ExplicitSnapshotsChangeUponNavigation adds an NSBox,
14+
which becomes layer backed, just like the WKWebView, but with no
15+
common layer-backed ancestor).
16+
117
2017-05-22 youenn fablet <[email protected]>
218

319
Resync web-platform-tests up to 8df7c9c215678328212f232ce0b5270c505a8563

Tools/TestWebKitAPI/cocoa/TestWKWebView.mm

+1
Original file line numberDiff line numberDiff line change
@@ -191,6 +191,7 @@ - (void)_setUpTestWindow:(NSRect)frame
191191
_hostWindow = adoptNS([[TestWKWebViewHostWindow alloc] initWithContentRect:frame styleMask:NSWindowStyleMaskBorderless backing:NSBackingStoreBuffered defer:NO]);
192192
[_hostWindow setFrameOrigin:NSMakePoint(0, 0)];
193193
[_hostWindow setIsVisible:YES];
194+
[_hostWindow contentView].wantsLayer = YES;
194195
[[_hostWindow contentView] addSubview:self];
195196
[_hostWindow makeKeyAndOrderFront:self];
196197
#else

0 commit comments

Comments
 (0)