Title: [285247] branches/safari-612.3.3.1-branch/Source
- Revision
- 285247
- Author
- [email protected]
- Date
- 2021-11-03 17:12:21 -0700 (Wed, 03 Nov 2021)
Log Message
Revert "Cherry-pick r285241. rdar://problem/84990898"
This reverts commit r285243.
Modified Paths
Diff
Modified: branches/safari-612.3.3.1-branch/Source/WTF/ChangeLog (285246 => 285247)
--- branches/safari-612.3.3.1-branch/Source/WTF/ChangeLog 2021-11-04 00:05:38 UTC (rev 285246)
+++ branches/safari-612.3.3.1-branch/Source/WTF/ChangeLog 2021-11-04 00:12:21 UTC (rev 285247)
@@ -1,47 +1,3 @@
-2021-11-03 Russell Epstein <[email protected]>
-
- Cherry-pick r285241. rdar://problem/84990898
-
- [macOS] MediaSession coordinator enabled in UIProcess, disabled in WebContent
- https://bugs.webkit.org/show_bug.cgi?id=232687
- <rdar://84990898>
-
- Reviewed by Eric Carlson.
-
- Source/WebKit:
-
- The default value of defaultMediaSessionCoordinatorEnabled() will be true if the UIProcess has
- the correct entitlement. However, this value is not passed across the XPC boundary unless it has
- been overridden, which means defaultMediaSessionCoordinatorEnabled() will control the setting
- in the WebContent process, which is not entitled.
-
- Have the WebContent process check its parent process's entitlement rather than its own when
- determining the default value for defaultMediaSessionCoordinatorEnabled().
-
- * Shared/WebPreferencesDefaultValues.cpp:
- (WebKit::defaultMediaSessionCoordinatorEnabled):
-
- Source/WTF:
-
- Have the MediaSessionPlaylistEnabled preference use the same defaults as MediaSessionCoordinatorEnabled.
-
- * Scripts/Preferences/WebPreferencesExperimental.yaml:
-
-
- git-svn-id: https://svn.webkit.org/repository/webkit/trunk@285241 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
- 2021-11-03 Jer Noble <[email protected]>
-
- [macOS] MediaSession coordinator enabled in UIProcess, disabled in WebContent
- https://bugs.webkit.org/show_bug.cgi?id=232687
- <rdar://84990898>
-
- Reviewed by Eric Carlson.
-
- Have the MediaSessionPlaylistEnabled preference use the same defaults as MediaSessionCoordinatorEnabled.
-
- * Scripts/Preferences/WebPreferencesExperimental.yaml:
-
2021-11-02 Russell Epstein <[email protected]>
Cherry-pick r283732. rdar://problem/84944569
Modified: branches/safari-612.3.3.1-branch/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml (285246 => 285247)
--- branches/safari-612.3.3.1-branch/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml 2021-11-04 00:05:38 UTC (rev 285246)
+++ branches/safari-612.3.3.1-branch/Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml 2021-11-04 00:12:21 UTC (rev 285247)
@@ -780,9 +780,9 @@
WebKitLegacy:
default: false
WebKit:
- default: WebKit::defaultMediaSessionCoordinatorEnabled()
+ default: false
WebCore:
- default: true
+ default: false
ModelElementEnabled:
type: bool
Modified: branches/safari-612.3.3.1-branch/Source/WebKit/ChangeLog (285246 => 285247)
--- branches/safari-612.3.3.1-branch/Source/WebKit/ChangeLog 2021-11-04 00:05:38 UTC (rev 285246)
+++ branches/safari-612.3.3.1-branch/Source/WebKit/ChangeLog 2021-11-04 00:12:21 UTC (rev 285247)
@@ -1,56 +1,5 @@
2021-11-03 Russell Epstein <[email protected]>
- Cherry-pick r285241. rdar://problem/84990898
-
- [macOS] MediaSession coordinator enabled in UIProcess, disabled in WebContent
- https://bugs.webkit.org/show_bug.cgi?id=232687
- <rdar://84990898>
-
- Reviewed by Eric Carlson.
-
- Source/WebKit:
-
- The default value of defaultMediaSessionCoordinatorEnabled() will be true if the UIProcess has
- the correct entitlement. However, this value is not passed across the XPC boundary unless it has
- been overridden, which means defaultMediaSessionCoordinatorEnabled() will control the setting
- in the WebContent process, which is not entitled.
-
- Have the WebContent process check its parent process's entitlement rather than its own when
- determining the default value for defaultMediaSessionCoordinatorEnabled().
-
- * Shared/WebPreferencesDefaultValues.cpp:
- (WebKit::defaultMediaSessionCoordinatorEnabled):
-
- Source/WTF:
-
- Have the MediaSessionPlaylistEnabled preference use the same defaults as MediaSessionCoordinatorEnabled.
-
- * Scripts/Preferences/WebPreferencesExperimental.yaml:
-
-
- git-svn-id: https://svn.webkit.org/repository/webkit/trunk@285241 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
- 2021-11-03 Jer Noble <[email protected]>
-
- [macOS] MediaSession coordinator enabled in UIProcess, disabled in WebContent
- https://bugs.webkit.org/show_bug.cgi?id=232687
- <rdar://84990898>
-
- Reviewed by Eric Carlson.
-
- The default value of defaultMediaSessionCoordinatorEnabled() will be true if the UIProcess has
- the correct entitlement. However, this value is not passed across the XPC boundary unless it has
- been overridden, which means defaultMediaSessionCoordinatorEnabled() will control the setting
- in the WebContent process, which is not entitled.
-
- Have the WebContent process check its parent process's entitlement rather than its own when
- determining the default value for defaultMediaSessionCoordinatorEnabled().
-
- * Shared/WebPreferencesDefaultValues.cpp:
- (WebKit::defaultMediaSessionCoordinatorEnabled):
-
-2021-11-03 Russell Epstein <[email protected]>
-
Cherry-pick r285177. rdar://problem/84987165
Terminate unresponsive network process by crashing it
Modified: branches/safari-612.3.3.1-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.cpp (285246 => 285247)
--- branches/safari-612.3.3.1-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.cpp 2021-11-04 00:05:38 UTC (rev 285246)
+++ branches/safari-612.3.3.1-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.cpp 2021-11-04 00:12:21 UTC (rev 285247)
@@ -36,7 +36,6 @@
#endif
#if ENABLE(MEDIA_SESSION_COORDINATOR)
-#import "WebProcess.h"
#import <wtf/cocoa/Entitlements.h>
#endif
@@ -324,10 +323,7 @@
static dispatch_once_t onceToken;
static bool enabled { false };
dispatch_once(&onceToken, ^{
- if (isInWebProcess())
- enabled = WebProcess::singleton().parentProcessHasEntitlement("com.apple.developer.group-session.urlactivity");
- else
- enabled = WTF::processHasEntitlement("com.apple.developer.group-session.urlactivity");
+ enabled = WTF::processHasEntitlement("com.apple.developer.group-session.urlactivity");
});
return enabled;
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes