Title: [232366] tags/Safari-606.1.19/Source/WebKit
- Revision
- 232366
- Author
- [email protected]
- Date
- 2018-05-31 13:19:41 -0700 (Thu, 31 May 2018)
Log Message
Cherry-pick r232299. rdar://problem/40691662
Fix a few issues in WKFullScreenViewController
https://bugs.webkit.org/show_bug.cgi?id=186067
<rdar://problem/40630944>
Reviewed by Darin Adler.
The check in setInterface() is checking the wrong pointer:
* UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
(WKFullScreenViewControllerPlaybackSessionModelClient::setInterface):
The check in -_effectiveFullscreenInsetTop is backwards:
(-[WKFullScreenViewController _effectiveFullscreenInsetTop]):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232299 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: tags/Safari-606.1.19/Source/WebKit/ChangeLog (232365 => 232366)
--- tags/Safari-606.1.19/Source/WebKit/ChangeLog 2018-05-31 19:51:30 UTC (rev 232365)
+++ tags/Safari-606.1.19/Source/WebKit/ChangeLog 2018-05-31 20:19:41 UTC (rev 232366)
@@ -1,5 +1,40 @@
2018-05-31 Kocsen Chung <[email protected]>
+ Cherry-pick r232299. rdar://problem/40691662
+
+ Fix a few issues in WKFullScreenViewController
+ https://bugs.webkit.org/show_bug.cgi?id=186067
+ <rdar://problem/40630944>
+
+ Reviewed by Darin Adler.
+
+ The check in setInterface() is checking the wrong pointer:
+ * UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
+ (WKFullScreenViewControllerPlaybackSessionModelClient::setInterface):
+
+ The check in -_effectiveFullscreenInsetTop is backwards:
+ (-[WKFullScreenViewController _effectiveFullscreenInsetTop]):
+
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232299 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2018-05-30 Jer Noble <[email protected]>
+
+ Fix a few issues in WKFullScreenViewController
+ https://bugs.webkit.org/show_bug.cgi?id=186067
+ <rdar://problem/40630944>
+
+ Reviewed by Darin Adler.
+
+ The check in setInterface() is checking the wrong pointer:
+ * UIProcess/ios/fullscreen/WKFullScreenViewController.mm:
+ (WKFullScreenViewControllerPlaybackSessionModelClient::setInterface):
+
+ The check in -_effectiveFullscreenInsetTop is backwards:
+ (-[WKFullScreenViewController _effectiveFullscreenInsetTop]):
+
+2018-05-31 Kocsen Chung <[email protected]>
+
Revert r231903. rdar://problem/40690985
2018-05-31 Kocsen Chung <[email protected]>
Modified: tags/Safari-606.1.19/Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm (232365 => 232366)
--- tags/Safari-606.1.19/Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm 2018-05-31 19:51:30 UTC (rev 232365)
+++ tags/Safari-606.1.19/Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenViewController.mm 2018-05-31 20:19:41 UTC (rev 232366)
@@ -67,7 +67,7 @@
return;
if (m_interface && m_interface->playbackSessionModel())
- interface->playbackSessionModel()->removeClient(*this);
+ m_interface->playbackSessionModel()->removeClient(*this);
m_interface = interface;
if (m_interface && m_interface->playbackSessionModel())
interface->playbackSessionModel()->addClient(*this);
@@ -360,7 +360,7 @@
@dynamic _effectiveFullscreenInsetTop;
- (CGFloat)_effectiveFullscreenInsetTop
{
- if (!self.prefersStatusBarHidden)
+ if (self.prefersStatusBarHidden)
return 0;
CGRect cancelFrame = _cancelButton.get().frame;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
