Title: [285351] branches/safari-613.1.7-branch/Source/WebCore
- Revision
- 285351
- Author
- [email protected]
- Date
- 2021-11-05 12:53:38 -0700 (Fri, 05 Nov 2021)
Log Message
Cherry-pick r285231. rdar://problem/85082384
[ iOS ] TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS is crashing
https://bugs.webkit.org/show_bug.cgi?id=232676
Patch by Alex Christensen <[email protected]> on 2021-11-03
Reviewed by Eric Carlson.
In r284743 we changed types from raw pointers to WeakPtr, but accessing them in layoutSublayers is not thread safe.
Remove some unsafe early returns.
* platform/ios/VideoFullscreenInterfaceAVKit.mm:
(-[WebAVPlayerLayer layoutSublayers]):
(-[WebAVPlayerLayer resolveBounds]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@285231 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-613.1.7-branch/Source/WebCore/ChangeLog (285350 => 285351)
--- branches/safari-613.1.7-branch/Source/WebCore/ChangeLog 2021-11-05 19:33:46 UTC (rev 285350)
+++ branches/safari-613.1.7-branch/Source/WebCore/ChangeLog 2021-11-05 19:53:38 UTC (rev 285351)
@@ -1,3 +1,36 @@
+2021-11-05 Russell Epstein <[email protected]>
+
+ Cherry-pick r285231. rdar://problem/85082384
+
+ [ iOS ] TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS is crashing
+ https://bugs.webkit.org/show_bug.cgi?id=232676
+
+ Patch by Alex Christensen <[email protected]> on 2021-11-03
+ Reviewed by Eric Carlson.
+
+ In r284743 we changed types from raw pointers to WeakPtr, but accessing them in layoutSublayers is not thread safe.
+ Remove some unsafe early returns.
+
+ * platform/ios/VideoFullscreenInterfaceAVKit.mm:
+ (-[WebAVPlayerLayer layoutSublayers]):
+ (-[WebAVPlayerLayer resolveBounds]):
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@285231 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2021-11-03 Alex Christensen <[email protected]>
+
+ [ iOS ] TestWebKitAPI.WebKitLegacy.AudioSessionCategoryIOS is crashing
+ https://bugs.webkit.org/show_bug.cgi?id=232676
+
+ Reviewed by Eric Carlson.
+
+ In r284743 we changed types from raw pointers to WeakPtr, but accessing them in layoutSublayers is not thread safe.
+ Remove some unsafe early returns.
+
+ * platform/ios/VideoFullscreenInterfaceAVKit.mm:
+ (-[WebAVPlayerLayer layoutSublayers]):
+ (-[WebAVPlayerLayer resolveBounds]):
+
2021-11-03 Alan Coon <[email protected]>
Cherry-pick r285110. rdar://problem/84994696
Modified: branches/safari-613.1.7-branch/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm (285350 => 285351)
--- branches/safari-613.1.7-branch/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm 2021-11-05 19:33:46 UTC (rev 285350)
+++ branches/safari-613.1.7-branch/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm 2021-11-05 19:53:38 UTC (rev 285351)
@@ -289,9 +289,6 @@
if ([_videoSublayer superlayer] != self)
return;
- if (![_avPlayerController delegate])
- return;
-
[_videoSublayer setPosition:CGPointMake(CGRectGetMidX(self.bounds), CGRectGetMidY(self.bounds))];
if (self.videoDimensions.height <= 0 || self.videoDimensions.width <= 0)
@@ -328,8 +325,6 @@
- (void)resolveBounds
{
[NSObject cancelPreviousPerformRequestsWithTarget:self selector:@selector(resolveBounds) object:nil];
- if (![_avPlayerController delegate])
- return;
if ([_videoSublayer superlayer] != self)
return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
