<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[209512] trunk</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/209512">209512</a></dd>
<dt>Author</dt> <dd>eric.carlson@apple.com</dd>
<dt>Date</dt> <dd>2016-12-07 18:25:57 -0800 (Wed, 07 Dec 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>[MediaStream][Mac] Revoke sandbox extensions when capture ends
https://bugs.webkit.org/show_bug.cgi?id=165476
Reviewed by Brady Eidson.
Source/WebKit2:
Track media capture by process so it is possible to revoke the sandbox extensions issued to
a web process when capture stops. Allocate WK2 user media permission manager lazily rather
than every time a page is created as most pages won't need one.
* CMakeLists.txt: Add new files.
* UIProcess/UserMediaPermissionRequestManagerProxy.cpp:
(WebKit::UserMediaPermissionRequestManagerProxy::UserMediaPermissionRequestManagerProxy): Register
with process manager.
(WebKit::UserMediaPermissionRequestManagerProxy::~UserMediaPermissionRequestManagerProxy):
(WebKit::UserMediaPermissionRequestManagerProxy::userMediaAccessWasGranted): Move sandbox
extension code to the process manager.
(WebKit::UserMediaPermissionRequestManagerProxy::stopCapture): New.
(WebKit::UserMediaPermissionRequestManagerProxy::startedCaptureSession): New, report state
change to the process manager.
(WebKit::UserMediaPermissionRequestManagerProxy::endedCaptureSession): Ditto.
* UIProcess/UserMediaPermissionRequestManagerProxy.h:
* UIProcess/UserMediaProcessManager.cpp: Added.
(WebKit::ProcessState::ProcessState):
(WebKit::ProcessState::managers):
(WebKit::ProcessState::sandboxExtensionsGranted):
(WebKit::ProcessState::setSandboxExtensionsGranted):
(WebKit::stateMap):
(WebKit::processState):
(WebKit::ProcessState::addRequestManager):
(WebKit::ProcessState::removeRequestManager):
(WebKit::UserMediaProcessManager::singleton):
(WebKit::UserMediaProcessManager::addUserMediaPermissionRequestManagerProxy):
(WebKit::UserMediaProcessManager::removeUserMediaPermissionRequestManagerProxy):
(WebKit::UserMediaProcessManager::willCreateMediaStream):
(WebKit::UserMediaProcessManager::startedCaptureSession):
(WebKit::UserMediaProcessManager::endedCaptureSession):
* UIProcess/UserMediaProcessManager.h: Added.
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::WebPageProxy): Don't allocate m_userMediaPermissionRequestManager.
(WebKit::WebPageProxy::resetState): Set m_userMediaPermissionRequestManager to null.
(WebKit::WebPageProxy::userMediaPermissionRequestManager): Allocate m_userMediaPermissionRequestManager
lazily as most pages don't need it.
(WebKit::WebPageProxy::requestUserMediaPermissionForFrame): Call userMediaPermissionRequestManager().
(WebKit::WebPageProxy::enumerateMediaDevicesForFrame): Ditto.
(WebKit::WebPageProxy::clearUserMediaState): Ditto.
(WebKit::WebPageProxy::isPlayingMediaDidChange): Report capture state changes to the request
manager proxy.
* UIProcess/WebPageProxy.h:
* WebKit2.xcodeproj/project.pbxproj: Add new files.
* WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp: Added.
(WebKit::MediaDeviceSandboxExtensions::MediaDeviceSandboxExtensions):
(WebKit::MediaDeviceSandboxExtensions::encode):
(WebKit::MediaDeviceSandboxExtensions::decode):
(WebKit::MediaDeviceSandboxExtensions::operator[]):
(WebKit::MediaDeviceSandboxExtensions::size):
* WebProcess/MediaStream/MediaDeviceSandboxExtensions.h: Added.
* WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp:
(WebKit::UserMediaPermissionRequestManager::~UserMediaPermissionRequestManager): Sandbox extension
is now in a HashMap, not a Vector.
(WebKit::UserMediaPermissionRequestManager::grantUserMediaDeviceSandboxExtensions): Record
IDs with extensions so they can be revoked later.
(WebKit::UserMediaPermissionRequestManager::revokeUserMediaDeviceSandboxExtensions): New.
(WebKit::UserMediaPermissionRequestManager::grantUserMediaDevicesSandboxExtension): Deleted.
* WebProcess/MediaStream/UserMediaPermissionRequestManager.h:
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::grantUserMediaDeviceSandboxExtensions):
(WebKit::WebPage::revokeUserMediaDeviceSandboxExtensions):
(WebKit::WebPage::grantUserMediaDevicesSandboxExtension): Deleted.
* WebProcess/WebPage/WebPage.h:
* WebProcess/WebPage/WebPage.messages.in:
Tools:
Add a test which loads a page that calls navigator.mediaDevices.enumerateDevices, kills the
page, and loads the same page again to ensure that the WK2 UserMediaPermissionRequestManagerProxy
is reset and recreated when a web page exits and is reloaded.
* TestWebKitAPI/PlatformGTK.cmake:
* TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
* TestWebKitAPI/Tests/WebKit2/EnumerateMediaDevices.cpp: Added.
(TestWebKitAPI::checkUserMediaPermissionCallback):
(TestWebKitAPI::TEST):
* TestWebKitAPI/Tests/WebKit2/enumerateMediaDevices.html: Added.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkSourceWebKit2CMakeListstxt">trunk/Source/WebKit2/CMakeLists.txt</a></li>
<li><a href="#trunkSourceWebKit2ChangeLog">trunk/Source/WebKit2/ChangeLog</a></li>
<li><a href="#trunkSourceWebKit2UIProcessUserMediaPermissionRequestManagerProxycpp">trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp</a></li>
<li><a href="#trunkSourceWebKit2UIProcessUserMediaPermissionRequestManagerProxyh">trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h</a></li>
<li><a href="#trunkSourceWebKit2UIProcessWebPageProxycpp">trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp</a></li>
<li><a href="#trunkSourceWebKit2UIProcessWebPageProxyh">trunk/Source/WebKit2/UIProcess/WebPageProxy.h</a></li>
<li><a href="#trunkSourceWebKit2WebKit2xcodeprojprojectpbxproj">trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj</a></li>
<li><a href="#trunkSourceWebKit2WebProcessMediaStreamUserMediaPermissionRequestManagercpp">trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessMediaStreamUserMediaPermissionRequestManagerh">trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.h</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPageWebPagecpp">trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPageWebPageh">trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h</a></li>
<li><a href="#trunkSourceWebKit2WebProcessWebPageWebPagemessagesin">trunk/Source/WebKit2/WebProcess/WebPage/WebPage.messages.in</a></li>
<li><a href="#trunkToolsChangeLog">trunk/Tools/ChangeLog</a></li>
<li><a href="#trunkToolsTestWebKitAPIPlatformGTKcmake">trunk/Tools/TestWebKitAPI/PlatformGTK.cmake</a></li>
<li><a href="#trunkToolsTestWebKitAPITestWebKitAPIxcodeprojprojectpbxproj">trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj</a></li>
</ul>
<h3>Added Paths</h3>
<ul>
<li><a href="#trunkSourceWebKit2UIProcessUserMediaProcessManagercpp">trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp</a></li>
<li><a href="#trunkSourceWebKit2UIProcessUserMediaProcessManagerh">trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h</a></li>
<li><a href="#trunkSourceWebKit2WebProcessMediaStreamMediaDeviceSandboxExtensionscpp">trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp</a></li>
<li><a href="#trunkSourceWebKit2WebProcessMediaStreamMediaDeviceSandboxExtensionsh">trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h</a></li>
<li><a href="#trunkToolsTestWebKitAPITestsWebKit2EnumerateMediaDevicescpp">trunk/Tools/TestWebKitAPI/Tests/WebKit2/EnumerateMediaDevices.cpp</a></li>
<li><a href="#trunkToolsTestWebKitAPITestsWebKit2enumerateMediaDeviceshtml">trunk/Tools/TestWebKitAPI/Tests/WebKit2/enumerateMediaDevices.html</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebKit2CMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/CMakeLists.txt (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/CMakeLists.txt 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/CMakeLists.txt 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -376,6 +376,7 @@
</span><span class="cx"> UIProcess/UserMediaPermissionCheckProxy.cpp
</span><span class="cx"> UIProcess/UserMediaPermissionRequestManagerProxy.cpp
</span><span class="cx"> UIProcess/UserMediaPermissionRequestProxy.cpp
</span><ins>+ UIProcess/UserMediaProcessManager.cpp
</ins><span class="cx"> UIProcess/VisitedLinkStore.cpp
</span><span class="cx"> UIProcess/WebBackForwardList.cpp
</span><span class="cx"> UIProcess/WebColorPicker.cpp
</span><span class="lines">@@ -588,6 +589,7 @@
</span><span class="cx"> WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp
</span><span class="cx"> WebProcess/InjectedBundle/DOM/InjectedBundleRangeHandle.cpp
</span><span class="cx">
</span><ins>+ WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp
</ins><span class="cx"> WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp
</span><span class="cx">
</span><span class="cx"> WebProcess/Network/NetworkProcessConnection.cpp
</span></span></pre></div>
<a id="trunkSourceWebKit2ChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/ChangeLog (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/ChangeLog 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -1,3 +1,83 @@
</span><ins>+2016-12-07 Eric Carlson <eric.carlson@apple.com>
+
+ [MediaStream][Mac] Revoke sandbox extensions when capture ends
+ https://bugs.webkit.org/show_bug.cgi?id=165476
+
+ Reviewed by Brady Eidson.
+
+ Track media capture by process so it is possible to revoke the sandbox extensions issued to
+ a web process when capture stops. Allocate WK2 user media permission manager lazily rather
+ than every time a page is created as most pages won't need one.
+
+ * CMakeLists.txt: Add new files.
+
+ * UIProcess/UserMediaPermissionRequestManagerProxy.cpp:
+ (WebKit::UserMediaPermissionRequestManagerProxy::UserMediaPermissionRequestManagerProxy): Register
+ with process manager.
+ (WebKit::UserMediaPermissionRequestManagerProxy::~UserMediaPermissionRequestManagerProxy):
+ (WebKit::UserMediaPermissionRequestManagerProxy::userMediaAccessWasGranted): Move sandbox
+ extension code to the process manager.
+ (WebKit::UserMediaPermissionRequestManagerProxy::stopCapture): New.
+ (WebKit::UserMediaPermissionRequestManagerProxy::startedCaptureSession): New, report state
+ change to the process manager.
+ (WebKit::UserMediaPermissionRequestManagerProxy::endedCaptureSession): Ditto.
+ * UIProcess/UserMediaPermissionRequestManagerProxy.h:
+
+ * UIProcess/UserMediaProcessManager.cpp: Added.
+ (WebKit::ProcessState::ProcessState):
+ (WebKit::ProcessState::managers):
+ (WebKit::ProcessState::sandboxExtensionsGranted):
+ (WebKit::ProcessState::setSandboxExtensionsGranted):
+ (WebKit::stateMap):
+ (WebKit::processState):
+ (WebKit::ProcessState::addRequestManager):
+ (WebKit::ProcessState::removeRequestManager):
+ (WebKit::UserMediaProcessManager::singleton):
+ (WebKit::UserMediaProcessManager::addUserMediaPermissionRequestManagerProxy):
+ (WebKit::UserMediaProcessManager::removeUserMediaPermissionRequestManagerProxy):
+ (WebKit::UserMediaProcessManager::willCreateMediaStream):
+ (WebKit::UserMediaProcessManager::startedCaptureSession):
+ (WebKit::UserMediaProcessManager::endedCaptureSession):
+ * UIProcess/UserMediaProcessManager.h: Added.
+
+ * UIProcess/WebPageProxy.cpp:
+ (WebKit::WebPageProxy::WebPageProxy): Don't allocate m_userMediaPermissionRequestManager.
+ (WebKit::WebPageProxy::resetState): Set m_userMediaPermissionRequestManager to null.
+ (WebKit::WebPageProxy::userMediaPermissionRequestManager): Allocate m_userMediaPermissionRequestManager
+ lazily as most pages don't need it.
+ (WebKit::WebPageProxy::requestUserMediaPermissionForFrame): Call userMediaPermissionRequestManager().
+ (WebKit::WebPageProxy::enumerateMediaDevicesForFrame): Ditto.
+ (WebKit::WebPageProxy::clearUserMediaState): Ditto.
+ (WebKit::WebPageProxy::isPlayingMediaDidChange): Report capture state changes to the request
+ manager proxy.
+ * UIProcess/WebPageProxy.h:
+
+ * WebKit2.xcodeproj/project.pbxproj: Add new files.
+
+ * WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp: Added.
+ (WebKit::MediaDeviceSandboxExtensions::MediaDeviceSandboxExtensions):
+ (WebKit::MediaDeviceSandboxExtensions::encode):
+ (WebKit::MediaDeviceSandboxExtensions::decode):
+ (WebKit::MediaDeviceSandboxExtensions::operator[]):
+ (WebKit::MediaDeviceSandboxExtensions::size):
+ * WebProcess/MediaStream/MediaDeviceSandboxExtensions.h: Added.
+
+ * WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp:
+ (WebKit::UserMediaPermissionRequestManager::~UserMediaPermissionRequestManager): Sandbox extension
+ is now in a HashMap, not a Vector.
+ (WebKit::UserMediaPermissionRequestManager::grantUserMediaDeviceSandboxExtensions): Record
+ IDs with extensions so they can be revoked later.
+ (WebKit::UserMediaPermissionRequestManager::revokeUserMediaDeviceSandboxExtensions): New.
+ (WebKit::UserMediaPermissionRequestManager::grantUserMediaDevicesSandboxExtension): Deleted.
+ * WebProcess/MediaStream/UserMediaPermissionRequestManager.h:
+
+ * WebProcess/WebPage/WebPage.cpp:
+ (WebKit::WebPage::grantUserMediaDeviceSandboxExtensions):
+ (WebKit::WebPage::revokeUserMediaDeviceSandboxExtensions):
+ (WebKit::WebPage::grantUserMediaDevicesSandboxExtension): Deleted.
+ * WebProcess/WebPage/WebPage.h:
+ * WebProcess/WebPage/WebPage.messages.in:
+
</ins><span class="cx"> 2016-12-07 Jeremy Jones <jeremyj@apple.com>
</span><span class="cx">
</span><span class="cx"> One esc to exit fullscreen and pointer lock
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaPermissionRequestManagerProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -22,6 +22,7 @@
</span><span class="cx">
</span><span class="cx"> #include "APISecurityOrigin.h"
</span><span class="cx"> #include "APIUIClient.h"
</span><ins>+#include "UserMediaProcessManager.h"
</ins><span class="cx"> #include "WebPageMessages.h"
</span><span class="cx"> #include "WebPageProxy.h"
</span><span class="cx"> #include "WebProcessProxy.h"
</span><span class="lines">@@ -92,8 +93,19 @@
</span><span class="cx"> UserMediaPermissionRequestManagerProxy::UserMediaPermissionRequestManagerProxy(WebPageProxy& page)
</span><span class="cx"> : m_page(page)
</span><span class="cx"> {
</span><ins>+#if ENABLE(MEDIA_STREAM)
+ UserMediaProcessManager::singleton().addUserMediaPermissionRequestManagerProxy(*this);
+#endif
</ins><span class="cx"> }
</span><span class="cx">
</span><ins>+UserMediaPermissionRequestManagerProxy::~UserMediaPermissionRequestManagerProxy()
+{
+#if ENABLE(MEDIA_STREAM)
+ UserMediaProcessManager::singleton().removeUserMediaPermissionRequestManagerProxy(*this);
+#endif
+ invalidateRequests();
+}
+
</ins><span class="cx"> void UserMediaPermissionRequestManagerProxy::invalidateRequests()
</span><span class="cx"> {
</span><span class="cx"> for (auto& request : m_pendingUserMediaRequests.values())
</span><span class="lines">@@ -197,30 +209,8 @@
</span><span class="cx"> fameState.setHasPermissionToUseCaptureDevice(audioDeviceUID, true);
</span><span class="cx"> fameState.setHasPermissionToUseCaptureDevice(videoDeviceUID, true);
</span><span class="cx">
</span><del>- size_t extensionCount = 0;
- unsigned requiredExtensions = SandboxExtensionsGranted::None;
- if (!audioDeviceUID.isEmpty()) {
- requiredExtensions |= SandboxExtensionsGranted::Audio;
- extensionCount++;
- }
- if (!videoDeviceUID.isEmpty()) {
- requiredExtensions |= SandboxExtensionsGranted::Video;
- extensionCount++;
- }
</del><ins>+ UserMediaProcessManager::singleton().willCreateMediaStream(*this, !audioDeviceUID.isEmpty(), !videoDeviceUID.isEmpty());
</ins><span class="cx">
</span><del>- unsigned currentExtensions = m_pageSandboxExtensionsGranted;
- if (!(requiredExtensions & currentExtensions)) {
- ASSERT(extensionCount);
- m_pageSandboxExtensionsGranted = requiredExtensions | currentExtensions;
- SandboxExtension::HandleArray handles;
- handles.allocate(extensionCount);
- if (!videoDeviceUID.isEmpty())
- SandboxExtension::createHandleForGenericExtension("com.apple.webkit.camera", handles[--extensionCount]);
- if (!audioDeviceUID.isEmpty())
- SandboxExtension::createHandleForGenericExtension("com.apple.webkit.microphone", handles[--extensionCount]);
- m_page.process().send(Messages::WebPage::GrantUserMediaDevicesSandboxExtension(handles), m_page.pageID());
- }
-
</del><span class="cx"> m_page.process().send(Messages::WebPage::UserMediaAccessWasGranted(userMediaID, audioDeviceUID, videoDeviceUID), m_page.pageID());
</span><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(userMediaID);
</span><span class="lines">@@ -342,4 +332,34 @@
</span><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+void UserMediaPermissionRequestManagerProxy::stopCapture()
+{
+ if (!m_page.isValid())
+ return;
+
+#if ENABLE(MEDIA_STREAM)
+ m_page.setMuted(WebCore::MediaProducer::CaptureDevicesAreMuted);
+#endif
+}
+
+void UserMediaPermissionRequestManagerProxy::startedCaptureSession()
+{
+ if (!m_page.isValid())
+ return;
+
+#if ENABLE(MEDIA_STREAM)
+ UserMediaProcessManager::singleton().startedCaptureSession(*this);
+#endif
+}
+
+void UserMediaPermissionRequestManagerProxy::endedCaptureSession()
+{
+ if (!m_page.isValid())
+ return;
+
+#if ENABLE(MEDIA_STREAM)
+ UserMediaProcessManager::singleton().endedCaptureSession(*this);
+#endif
+}
+
</ins><span class="cx"> } // namespace WebKit
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaPermissionRequestManagerProxyh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -53,7 +53,10 @@
</span><span class="cx"> class UserMediaPermissionRequestManagerProxy {
</span><span class="cx"> public:
</span><span class="cx"> explicit UserMediaPermissionRequestManagerProxy(WebPageProxy&);
</span><ins>+ ~UserMediaPermissionRequestManagerProxy();
</ins><span class="cx">
</span><ins>+ WebPageProxy& page() const { return m_page; }
+
</ins><span class="cx"> void invalidateRequests();
</span><span class="cx">
</span><span class="cx"> void requestUserMediaPermissionForFrame(uint64_t userMediaID, uint64_t frameID, String userMediaDocumentOriginIdentifier, String topLevelDocumentOriginIdentifier, const WebCore::MediaConstraintsData& audioConstraintsData, const WebCore::MediaConstraintsData& videoConstraintsData);
</span><span class="lines">@@ -68,6 +71,10 @@
</span><span class="cx">
</span><span class="cx"> void clearCachedState();
</span><span class="cx">
</span><ins>+ void startedCaptureSession();
+ void endedCaptureSession();
+ void stopCapture();
+
</ins><span class="cx"> private:
</span><span class="cx"> Ref<UserMediaPermissionRequestProxy> createRequest(uint64_t userMediaID, uint64_t frameID, const String&userMediaDocumentOriginIdentifier, const String& topLevelDocumentOriginIdentifier, const Vector<String>& audioDeviceUIDs, const Vector<String>& videoDeviceUIDs);
</span><span class="cx"> void denyRequest(uint64_t userMediaID, UserMediaPermissionRequestProxy::UserMediaAccessDenialReason, const String& invalidConstraint);
</span><span class="lines">@@ -76,14 +83,6 @@
</span><span class="cx">
</span><span class="cx"> HashMap<uint64_t, RefPtr<UserMediaPermissionRequestProxy>> m_pendingUserMediaRequests;
</span><span class="cx"> HashMap<uint64_t, RefPtr<UserMediaPermissionCheckProxy>> m_pendingDeviceRequests;
</span><del>-
- enum SandboxExtensionsGranted {
- None = 0,
- Video = 1 << 0,
- Audio = 1 << 1
- };
- unsigned m_pageSandboxExtensionsGranted;
-
</del><span class="cx"> HashMap<uint64_t, std::unique_ptr<FrameAuthorizationState>> m_frameStates;
</span><span class="cx">
</span><span class="cx"> WebPageProxy& m_page;
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaProcessManagercpp"></a>
<div class="addfile"><h4>Added: trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp (0 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp (rev 0)
+++ trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -0,0 +1,198 @@
</span><ins>+/*
+ * Copyright (C) 2016 Apple Inc. All rights reserved.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#include "config.h"
+#include "UserMediaProcessManager.h"
+
+#if ENABLE(MEDIA_STREAM)
+
+#include "MediaDeviceSandboxExtensions.h"
+#include "WebPageMessages.h"
+#include "WebPageProxy.h"
+#include "WebProcessProxy.h"
+#include <wtf/HashMap.h>
+#include <wtf/NeverDestroyed.h>
+
+namespace WebKit {
+
+static const char* const audioExtensionPath = "com.apple.webkit.microphone";
+static const char* const videoExtensionPath = "com.apple.webkit.camera";
+
+class ProcessState {
+public:
+ ProcessState() { }
+ ProcessState(const ProcessState&) = delete;
+
+ void addRequestManager(UserMediaPermissionRequestManagerProxy&);
+ void removeRequestManager(UserMediaPermissionRequestManagerProxy&);
+ Vector<UserMediaPermissionRequestManagerProxy*>& managers() { return m_managers; }
+
+ enum SandboxExtensionsGranted {
+ None = 0,
+ Video = 1 << 0,
+ Audio = 1 << 1
+ };
+
+ SandboxExtensionsGranted sandboxExtensionsGranted() { return m_pageSandboxExtensionsGranted; }
+ void setSandboxExtensionsGranted(unsigned granted) { m_pageSandboxExtensionsGranted = static_cast<SandboxExtensionsGranted>(granted); }
+
+private:
+ Vector<UserMediaPermissionRequestManagerProxy*> m_managers;
+ SandboxExtensionsGranted m_pageSandboxExtensionsGranted { SandboxExtensionsGranted::None };
+};
+
+static HashMap<WebProcessProxy*, std::unique_ptr<ProcessState>>& stateMap()
+{
+ static NeverDestroyed<HashMap<WebProcessProxy*, std::unique_ptr<ProcessState>>> map;
+ return map;
+}
+
+static ProcessState& processState(WebProcessProxy& process)
+{
+ auto& state = stateMap().add(&process, nullptr).iterator->value;
+ if (state)
+ return *state;
+
+ state = std::make_unique<ProcessState>();
+ return *state;
+}
+
+void ProcessState::addRequestManager(UserMediaPermissionRequestManagerProxy& proxy)
+{
+ ASSERT(!m_managers.contains(&proxy));
+ m_managers.append(&proxy);
+}
+
+void ProcessState::removeRequestManager(UserMediaPermissionRequestManagerProxy& proxy)
+{
+ ASSERT(m_managers.contains(&proxy));
+ m_managers.removeFirstMatching([&proxy](auto other) {
+ return other == &proxy;
+ });
+}
+
+UserMediaProcessManager& UserMediaProcessManager::singleton()
+{
+ static NeverDestroyed<UserMediaProcessManager> manager;
+ return manager;
+}
+
+void UserMediaProcessManager::addUserMediaPermissionRequestManagerProxy(UserMediaPermissionRequestManagerProxy& proxy)
+{
+ processState(proxy.page().process()).addRequestManager(proxy);
+}
+
+void UserMediaProcessManager::removeUserMediaPermissionRequestManagerProxy(UserMediaPermissionRequestManagerProxy& proxy)
+{
+ endedCaptureSession(proxy);
+
+ auto& state = processState(proxy.page().process());
+ state.removeRequestManager(proxy);
+ if (state.managers().isEmpty()) {
+ auto it = stateMap().find(&proxy.page().process());
+ stateMap().remove(it);
+ }
+}
+
+void UserMediaProcessManager::willCreateMediaStream(UserMediaPermissionRequestManagerProxy& proxy, bool withAudio, bool withVideo)
+{
+ ASSERT(stateMap().contains(&proxy.page().process()));
+
+ auto& state = processState(proxy.page().process());
+ size_t extensionCount = 0;
+ unsigned requiredExtensions = ProcessState::SandboxExtensionsGranted::None;
+
+ if (withAudio) {
+ requiredExtensions |= ProcessState::SandboxExtensionsGranted::Audio;
+ extensionCount++;
+ }
+ if (withVideo) {
+ requiredExtensions |= ProcessState::SandboxExtensionsGranted::Video;
+ extensionCount++;
+ }
+
+ unsigned currentExtensions = state.sandboxExtensionsGranted();
+ if (!(requiredExtensions & currentExtensions)) {
+ SandboxExtension::HandleArray handles;
+ handles.allocate(extensionCount);
+
+ Vector<String> ids;
+ ids.reserveCapacity(extensionCount);
+
+ if (withAudio && requiredExtensions & ProcessState::SandboxExtensionsGranted::Audio) {
+ if (SandboxExtension::createHandleForGenericExtension(audioExtensionPath, handles[--extensionCount])) {
+ ids.append(ASCIILiteral(audioExtensionPath));
+ currentExtensions |= ProcessState::SandboxExtensionsGranted::Audio;
+ }
+ }
+
+ if (withVideo && requiredExtensions & ProcessState::SandboxExtensionsGranted::Video) {
+ if (SandboxExtension::createHandleForGenericExtension(videoExtensionPath, handles[--extensionCount])) {
+ ids.append(ASCIILiteral(videoExtensionPath));
+ currentExtensions |= ProcessState::SandboxExtensionsGranted::Video;
+ }
+ }
+
+ state.setSandboxExtensionsGranted(currentExtensions);
+ proxy.page().process().send(Messages::WebPage::GrantUserMediaDeviceSandboxExtensions(MediaDeviceSandboxExtensions(ids, WTFMove(handles))), proxy.page().pageID());
+ }
+}
+
+void UserMediaProcessManager::startedCaptureSession(UserMediaPermissionRequestManagerProxy& proxy)
+{
+ ASSERT(stateMap().contains(&proxy.page().process()));
+}
+
+void UserMediaProcessManager::endedCaptureSession(UserMediaPermissionRequestManagerProxy& proxy)
+{
+ ASSERT(stateMap().contains(&proxy.page().process()));
+
+ auto& state = processState(proxy.page().process());
+ bool hasAudioCapture = false;
+ bool hasVideoCapture = false;
+ for (auto& manager : state.managers()) {
+ if (manager->page().hasActiveAudioStream())
+ hasAudioCapture = true;
+ if (manager->page().hasActiveVideoStream())
+ hasVideoCapture = true;
+ }
+
+ if (hasAudioCapture && hasVideoCapture)
+ return;
+
+ Vector<String> params;
+ unsigned currentExtensions = state.sandboxExtensionsGranted();
+ if (!hasAudioCapture && currentExtensions & ProcessState::SandboxExtensionsGranted::Audio) {
+ params.append(ASCIILiteral(audioExtensionPath));
+ currentExtensions &= ~ProcessState::SandboxExtensionsGranted::Audio;
+ }
+ if (!hasVideoCapture && currentExtensions & ProcessState::SandboxExtensionsGranted::Video) {
+ params.append(ASCIILiteral(videoExtensionPath));
+ currentExtensions &= ~ProcessState::SandboxExtensionsGranted::Video;
+ }
+
+ if (params.isEmpty())
+ return;
+
+ state.setSandboxExtensionsGranted(currentExtensions);
+ proxy.page().process().send(Messages::WebPage::RevokeUserMediaDeviceSandboxExtensions(params), proxy.page().pageID());
+}
+
+} // namespace WebKit
+
+#endif
</ins></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaProcessManagerh"></a>
<div class="addfile"><h4>Added: trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h (0 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h (rev 0)
+++ trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -0,0 +1,45 @@
</span><ins>+/*
+ * Copyright (C) 2016 Apple Inc. All rights reserved.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#pragma once
+
+#if ENABLE(MEDIA_STREAM)
+
+#include "UserMediaPermissionRequestManagerProxy.h"
+
+namespace WebKit {
+
+class WebProcessProxy;
+
+class UserMediaProcessManager {
+public:
+
+ static UserMediaProcessManager& singleton();
+
+ void addUserMediaPermissionRequestManagerProxy(UserMediaPermissionRequestManagerProxy&);
+ void removeUserMediaPermissionRequestManagerProxy(UserMediaPermissionRequestManagerProxy&);
+
+ void willCreateMediaStream(UserMediaPermissionRequestManagerProxy&, bool withAudio, bool withVideo);
+
+ void startedCaptureSession(UserMediaPermissionRequestManagerProxy&);
+ void endedCaptureSession(UserMediaPermissionRequestManagerProxy&);
+};
+
+} // namespace WebKit
+
+#endif
</ins></span></pre></div>
<a id="trunkSourceWebKit2UIProcessWebPageProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -358,7 +358,6 @@
</span><span class="cx"> #endif
</span><span class="cx"> , m_geolocationPermissionRequestManager(*this)
</span><span class="cx"> , m_notificationPermissionRequestManager(*this)
</span><del>- , m_userMediaPermissionRequestManager(*this)
</del><span class="cx"> , m_activityState(ActivityState::NoFlags)
</span><span class="cx"> , m_viewWasEverInWindow(false)
</span><span class="cx"> #if PLATFORM(IOS)
</span><span class="lines">@@ -5376,8 +5375,9 @@
</span><span class="cx"> #if ENABLE(GEOLOCATION)
</span><span class="cx"> m_geolocationPermissionRequestManager.invalidateRequests();
</span><span class="cx"> #endif
</span><ins>+
</ins><span class="cx"> #if ENABLE(MEDIA_STREAM)
</span><del>- m_userMediaPermissionRequestManager.invalidateRequests();
</del><ins>+ m_userMediaPermissionRequestManager = nullptr;
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> m_notificationPermissionRequestManager.invalidateRequests();
</span><span class="lines">@@ -5692,12 +5692,21 @@
</span><span class="cx"> request->deny();
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+UserMediaPermissionRequestManagerProxy& WebPageProxy::userMediaPermissionRequestManager()
+{
+ if (m_userMediaPermissionRequestManager)
+ return *m_userMediaPermissionRequestManager;
+
+ m_userMediaPermissionRequestManager = std::make_unique<UserMediaPermissionRequestManagerProxy>(*this);
+ return *m_userMediaPermissionRequestManager;
+}
+
</ins><span class="cx"> void WebPageProxy::requestUserMediaPermissionForFrame(uint64_t userMediaID, uint64_t frameID, String userMediaDocumentOriginIdentifier, String topLevelDocumentOriginIdentifier, const WebCore::MediaConstraintsData& audioConstraintsData, const WebCore::MediaConstraintsData& videoConstraintsData)
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(MEDIA_STREAM)
</span><span class="cx"> MESSAGE_CHECK(m_process->webFrame(frameID));
</span><span class="cx">
</span><del>- m_userMediaPermissionRequestManager.requestUserMediaPermissionForFrame(userMediaID, frameID, userMediaDocumentOriginIdentifier, topLevelDocumentOriginIdentifier, audioConstraintsData, videoConstraintsData);
</del><ins>+ userMediaPermissionRequestManager().requestUserMediaPermissionForFrame(userMediaID, frameID, userMediaDocumentOriginIdentifier, topLevelDocumentOriginIdentifier, audioConstraintsData, videoConstraintsData);
</ins><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(userMediaID);
</span><span class="cx"> UNUSED_PARAM(frameID);
</span><span class="lines">@@ -5714,7 +5723,7 @@
</span><span class="cx"> WebFrameProxy* frame = m_process->webFrame(frameID);
</span><span class="cx"> MESSAGE_CHECK(frame);
</span><span class="cx">
</span><del>- m_userMediaPermissionRequestManager.enumerateMediaDevicesForFrame(userMediaID, frameID, userMediaDocumentOriginIdentifier, topLevelDocumentOriginIdentifier);
</del><ins>+ userMediaPermissionRequestManager().enumerateMediaDevicesForFrame(userMediaID, frameID, userMediaDocumentOriginIdentifier, topLevelDocumentOriginIdentifier);
</ins><span class="cx"> #else
</span><span class="cx"> UNUSED_PARAM(userMediaID);
</span><span class="cx"> UNUSED_PARAM(frameID);
</span><span class="lines">@@ -5726,7 +5735,7 @@
</span><span class="cx"> void WebPageProxy::clearUserMediaState()
</span><span class="cx"> {
</span><span class="cx"> #if ENABLE(MEDIA_STREAM)
</span><del>- m_userMediaPermissionRequestManager.clearCachedState();
</del><ins>+ userMediaPermissionRequestManager().clearCachedState();
</ins><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -6409,15 +6418,21 @@
</span><span class="cx">
</span><span class="cx"> WebCore::MediaProducer::MediaStateFlags oldMediaStateHasActiveCapture = m_mediaState & (WebCore::MediaProducer::HasActiveAudioCaptureDevice | WebCore::MediaProducer::HasActiveVideoCaptureDevice);
</span><span class="cx"> WebCore::MediaProducer::MediaStateFlags newMediaStateHasActiveCapture = state & (WebCore::MediaProducer::HasActiveAudioCaptureDevice | WebCore::MediaProducer::HasActiveVideoCaptureDevice);
</span><del>- if (!oldMediaStateHasActiveCapture && newMediaStateHasActiveCapture)
- m_uiClient->didBeginCaptureSession();
- if (oldMediaStateHasActiveCapture && !newMediaStateHasActiveCapture)
- m_uiClient->didEndCaptureSession();
</del><span class="cx">
</span><span class="cx"> MediaProducer::MediaStateFlags playingMediaMask = MediaProducer::IsPlayingAudio | MediaProducer::IsPlayingVideo;
</span><span class="cx"> MediaProducer::MediaStateFlags oldState = m_mediaState;
</span><span class="cx"> m_mediaState = state;
</span><span class="cx">
</span><ins>+#if ENABLE(MEDIA_STREAM)
+ if (!oldMediaStateHasActiveCapture && newMediaStateHasActiveCapture) {
+ m_uiClient->didBeginCaptureSession();
+ userMediaPermissionRequestManager().startedCaptureSession();
+ } else if (oldMediaStateHasActiveCapture && !newMediaStateHasActiveCapture) {
+ m_uiClient->didEndCaptureSession();
+ userMediaPermissionRequestManager().endedCaptureSession();
+ }
+#endif
+
</ins><span class="cx"> activityStateDidChange(ActivityState::IsAudible);
</span><span class="cx">
</span><span class="cx"> playingMediaMask |= MediaProducer::HasActiveAudioCaptureDevice | MediaProducer::HasActiveVideoCaptureDevice;
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessWebPageProxyh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.h (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/WebPageProxy.h 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.h 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -1049,6 +1049,8 @@
</span><span class="cx">
</span><span class="cx"> bool isPlayingAudio() const { return !!(m_mediaState & WebCore::MediaProducer::IsPlayingAudio); }
</span><span class="cx"> void isPlayingMediaDidChange(WebCore::MediaProducer::MediaStateFlags, uint64_t);
</span><ins>+ bool hasActiveAudioStream() const { return m_mediaState & WebCore::MediaProducer::HasActiveAudioCaptureDevice; }
+ bool hasActiveVideoStream() const { return m_mediaState & WebCore::MediaProducer::HasActiveVideoCaptureDevice; }
</ins><span class="cx"> WebCore::MediaProducer::MediaStateFlags mediaStateFlags() const { return m_mediaState; }
</span><span class="cx">
</span><span class="cx"> #if PLATFORM(MAC)
</span><span class="lines">@@ -1295,6 +1297,7 @@
</span><span class="cx"> void reachedApplicationCacheOriginQuota(const String& originIdentifier, uint64_t currentQuota, uint64_t totalBytesNeeded, PassRefPtr<Messages::WebPageProxy::ReachedApplicationCacheOriginQuota::DelayedReply>);
</span><span class="cx"> void requestGeolocationPermissionForFrame(uint64_t geolocationID, uint64_t frameID, String originIdentifier);
</span><span class="cx">
</span><ins>+ UserMediaPermissionRequestManagerProxy& userMediaPermissionRequestManager();
</ins><span class="cx"> void requestUserMediaPermissionForFrame(uint64_t userMediaID, uint64_t frameID, String userMediaDocumentOriginIdentifier, String topLevelDocumentOriginIdentifier, const WebCore::MediaConstraintsData& audioConstraints, const WebCore::MediaConstraintsData& videoConstraints);
</span><span class="cx"> void enumerateMediaDevicesForFrame(uint64_t userMediaID, uint64_t frameID, String userMediaDocumentOriginIdentifier, String topLevelDocumentOriginIdentifier);
</span><span class="cx">
</span><span class="lines">@@ -1688,7 +1691,9 @@
</span><span class="cx"> GeolocationPermissionRequestManagerProxy m_geolocationPermissionRequestManager;
</span><span class="cx"> NotificationPermissionRequestManagerProxy m_notificationPermissionRequestManager;
</span><span class="cx">
</span><del>- UserMediaPermissionRequestManagerProxy m_userMediaPermissionRequestManager;
</del><ins>+#if ENABLE(MEDIA_STREAM)
+ std::unique_ptr<UserMediaPermissionRequestManagerProxy> m_userMediaPermissionRequestManager;
+#endif
</ins><span class="cx">
</span><span class="cx"> WebCore::ActivityState::Flags m_activityState;
</span><span class="cx"> bool m_viewWasEverInWindow;
</span></span></pre></div>
<a id="trunkSourceWebKit2WebKit2xcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -58,6 +58,9 @@
</span><span class="cx"> 07297F9F1C17BBEA003F0735 /* UserMediaPermissionCheckProxy.h in Headers */ = {isa = PBXBuildFile; fileRef = 07297F9D1C17BBEA003F0735 /* UserMediaPermissionCheckProxy.h */; };
</span><span class="cx"> 07297FA21C186ADB003F0735 /* WKUserMediaPermissionCheck.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 07297FA01C186ADB003F0735 /* WKUserMediaPermissionCheck.cpp */; };
</span><span class="cx"> 07297FA31C186ADB003F0735 /* WKUserMediaPermissionCheck.h in Headers */ = {isa = PBXBuildFile; fileRef = 07297FA11C186ADB003F0735 /* WKUserMediaPermissionCheck.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><ins>+ 074E75FD1DF2211500D318EC /* UserMediaProcessManager.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 074E75FC1DF2002400D318EC /* UserMediaProcessManager.cpp */; };
+ 074E75FE1DF2211900D318EC /* UserMediaProcessManager.h in Headers */ = {isa = PBXBuildFile; fileRef = 074E75FB1DF1FD1300D318EC /* UserMediaProcessManager.h */; };
+ 074E76021DF707BE00D318EC /* MediaDeviceSandboxExtensions.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 074E76001DF7075D00D318EC /* MediaDeviceSandboxExtensions.cpp */; };
</ins><span class="cx"> 076E884E1A13CADF005E90FC /* APIContextMenuClient.h in Headers */ = {isa = PBXBuildFile; fileRef = 076E884D1A13CADF005E90FC /* APIContextMenuClient.h */; };
</span><span class="cx"> 07A5EBBB1C7BA43E00B9CA69 /* WKFrameHandleRef.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 07A5EBB91C7BA43E00B9CA69 /* WKFrameHandleRef.cpp */; };
</span><span class="cx"> 07A5EBBC1C7BA43E00B9CA69 /* WKFrameHandleRef.h in Headers */ = {isa = PBXBuildFile; fileRef = 07A5EBBA1C7BA43E00B9CA69 /* WKFrameHandleRef.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="lines">@@ -2076,6 +2079,10 @@
</span><span class="cx"> 07297F9D1C17BBEA003F0735 /* UserMediaPermissionCheckProxy.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = UserMediaPermissionCheckProxy.h; sourceTree = "<group>"; };
</span><span class="cx"> 07297FA01C186ADB003F0735 /* WKUserMediaPermissionCheck.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WKUserMediaPermissionCheck.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 07297FA11C186ADB003F0735 /* WKUserMediaPermissionCheck.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WKUserMediaPermissionCheck.h; sourceTree = "<group>"; };
</span><ins>+ 074E75FB1DF1FD1300D318EC /* UserMediaProcessManager.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = UserMediaProcessManager.h; sourceTree = "<group>"; };
+ 074E75FC1DF2002400D318EC /* UserMediaProcessManager.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = UserMediaProcessManager.cpp; sourceTree = "<group>"; };
+ 074E76001DF7075D00D318EC /* MediaDeviceSandboxExtensions.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = MediaDeviceSandboxExtensions.cpp; sourceTree = "<group>"; };
+ 074E76011DF7075D00D318EC /* MediaDeviceSandboxExtensions.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MediaDeviceSandboxExtensions.h; sourceTree = "<group>"; };
</ins><span class="cx"> 076E884D1A13CADF005E90FC /* APIContextMenuClient.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = APIContextMenuClient.h; sourceTree = "<group>"; };
</span><span class="cx"> 076E884F1A13CBC6005E90FC /* APIInjectedBundlePageContextMenuClient.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = APIInjectedBundlePageContextMenuClient.h; sourceTree = "<group>"; };
</span><span class="cx"> 07A5EBB91C7BA43E00B9CA69 /* WKFrameHandleRef.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = WKFrameHandleRef.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -5506,6 +5513,8 @@
</span><span class="cx"> 4A410F4119AF7B27002EBAB5 /* MediaStream */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><ins>+ 074E76001DF7075D00D318EC /* MediaDeviceSandboxExtensions.cpp */,
+ 074E76011DF7075D00D318EC /* MediaDeviceSandboxExtensions.h */,
</ins><span class="cx"> 4A410F4219AF7B27002EBAB5 /* UserMediaPermissionRequestManager.cpp */,
</span><span class="cx"> 4A410F4319AF7B27002EBAB5 /* UserMediaPermissionRequestManager.h */,
</span><span class="cx"> );
</span><span class="lines">@@ -6209,6 +6218,8 @@
</span><span class="cx"> BC032DC310F438260058C15A /* UIProcess */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><ins>+ 074E75FC1DF2002400D318EC /* UserMediaProcessManager.cpp */,
+ 074E75FB1DF1FD1300D318EC /* UserMediaProcessManager.h */,
</ins><span class="cx"> BC032DC410F4387C0058C15A /* API */,
</span><span class="cx"> 1AB1F7761D1B30A9007C9BD1 /* ApplePay */,
</span><span class="cx"> 512F588D12A8836F00629530 /* Authentication */,
</span><span class="lines">@@ -8364,6 +8375,7 @@
</span><span class="cx"> BC0E619812D6CD120012A72A /* WKGeolocationPosition.h in Headers */,
</span><span class="cx"> 0FCB4E4F18BBE044000FCFC9 /* WKGeolocationProviderIOS.h in Headers */,
</span><span class="cx"> BCC8B374125FB69000DE46A4 /* WKGeometry.h in Headers */,
</span><ins>+ 074E75FE1DF2211900D318EC /* UserMediaProcessManager.h in Headers */,
</ins><span class="cx"> 1A422F8B18B29B5400D8CD96 /* WKHistoryDelegatePrivate.h in Headers */,
</span><span class="cx"> B62E7312143047B00069EC35 /* WKHitTestResult.h in Headers */,
</span><span class="cx"> 5110AE0D133C16CB0072717A /* WKIconDatabase.h in Headers */,
</span><span class="lines">@@ -9434,6 +9446,7 @@
</span><span class="cx"> 0F5E200318E77051003EC3E5 /* PlatformCAAnimationRemote.mm in Sources */,
</span><span class="cx"> 2DA049B3180CCCD300AAFA9E /* PlatformCALayerRemote.cpp in Sources */,
</span><span class="cx"> 2D8710161828415D0018FA01 /* PlatformCALayerRemoteCustom.mm in Sources */,
</span><ins>+ 074E75FD1DF2211500D318EC /* UserMediaProcessManager.cpp in Sources */,
</ins><span class="cx"> 2D8949F0182044F600E898AA /* PlatformCALayerRemoteTiledBacking.cpp in Sources */,
</span><span class="cx"> BCC43ABA127B95DC00317F16 /* PlatformPopupMenuData.cpp in Sources */,
</span><span class="cx"> 1A6FB7D211E651E200DB1371 /* Plugin.cpp in Sources */,
</span><span class="lines">@@ -9895,6 +9908,7 @@
</span><span class="cx"> 1A256E3718A1A788006FB922 /* WKNavigationAction.mm in Sources */,
</span><span class="cx"> 2D3A65DA1A7C3A1F00CAC637 /* WKNavigationActionRef.cpp in Sources */,
</span><span class="cx"> 370F34A21829BE1E009027C8 /* WKNavigationData.mm in Sources */,
</span><ins>+ 074E76021DF707BE00D318EC /* MediaDeviceSandboxExtensions.cpp in Sources */,
</ins><span class="cx"> BCF69FAA1176D1CB00471A52 /* WKNavigationDataRef.cpp in Sources */,
</span><span class="cx"> 2D3A65E21A7C3A9300CAC637 /* WKNavigationRef.cpp in Sources */,
</span><span class="cx"> 1A1B0EB518A424950038481A /* WKNavigationResponse.mm in Sources */,
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessMediaStreamMediaDeviceSandboxExtensionscpp"></a>
<div class="addfile"><h4>Added: trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp (0 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp (rev 0)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -0,0 +1,80 @@
</span><ins>+/*
+ * Copyright (C) 2016 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "MediaDeviceSandboxExtensions.h"
+
+#if ENABLE(MEDIA_STREAM)
+
+#include "WebCoreArgumentCoders.h"
+
+namespace WebKit {
+
+MediaDeviceSandboxExtensions::MediaDeviceSandboxExtensions(Vector<String> ids, SandboxExtension::HandleArray&& handles)
+ : m_ids(ids)
+ , m_handles(WTFMove(handles))
+{
+ ASSERT_WITH_SECURITY_IMPLICATION(m_ids.size() == m_handles.size());
+}
+
+void MediaDeviceSandboxExtensions::encode(IPC::Encoder& encoder) const
+{
+ encoder << m_ids;
+ m_handles.encode(encoder);
+}
+
+bool MediaDeviceSandboxExtensions::decode(IPC::Decoder& decoder, MediaDeviceSandboxExtensions& result)
+{
+ if (!decoder.decode(result.m_ids))
+ return false;
+
+ if (!SandboxExtension::HandleArray::decode(decoder, result.m_handles))
+ return false;
+
+ return true;
+}
+
+std::pair<String, RefPtr<SandboxExtension>> MediaDeviceSandboxExtensions::operator[](size_t i)
+{
+ ASSERT_WITH_SECURITY_IMPLICATION(m_ids.size() == m_handles.size());
+ ASSERT_WITH_SECURITY_IMPLICATION(i < m_ids.size());
+ return { m_ids[i], SandboxExtension::create(m_handles[i]) };
+}
+
+const std::pair<String, RefPtr<SandboxExtension>> MediaDeviceSandboxExtensions::operator[](size_t i) const
+{
+ ASSERT_WITH_SECURITY_IMPLICATION(m_ids.size() == m_handles.size());
+ ASSERT_WITH_SECURITY_IMPLICATION(i < m_ids.size());
+ return { m_ids[i], SandboxExtension::create(m_handles[i]) };
+}
+
+size_t MediaDeviceSandboxExtensions::size() const
+{
+ return m_ids.size();
+}
+
+} // namespace WebKit
+
+#endif // ENABLE(MEDIA_STREAM)
</ins></span></pre></div>
<a id="trunkSourceWebKit2WebProcessMediaStreamMediaDeviceSandboxExtensionsh"></a>
<div class="addfile"><h4>Added: trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h (0 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h (rev 0)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -0,0 +1,59 @@
</span><ins>+/*
+ * Copyright (C) 2016 Apple Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY APPLE INC. AND ITS CONTRIBUTORS ``AS IS''
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+ * THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR ITS CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
+ * THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#pragma once
+
+#if ENABLE(MEDIA_STREAM)
+
+#include "SandboxExtension.h"
+#include <wtf/Noncopyable.h>
+#include <wtf/RefPtr.h>
+
+namespace WebKit {
+
+class MediaDeviceSandboxExtensions {
+ WTF_MAKE_NONCOPYABLE(MediaDeviceSandboxExtensions);
+public:
+ MediaDeviceSandboxExtensions()
+ {
+ }
+
+ MediaDeviceSandboxExtensions(Vector<String> ids, SandboxExtension::HandleArray&& handles);
+
+ std::pair<String, RefPtr<SandboxExtension>> operator[](size_t i);
+ const std::pair<String, RefPtr<SandboxExtension>> operator[](size_t i) const;
+ size_t size() const;
+
+ void encode(IPC::Encoder&) const;
+ static bool decode(IPC::Decoder&, MediaDeviceSandboxExtensions&);
+
+private:
+ Vector<String> m_ids;
+ SandboxExtension::HandleArray m_handles;
+};
+
+} // namespace WebKit
+
+#endif // ENABLE(MEDIA_STREAM)
</ins></span></pre></div>
<a id="trunkSourceWebKit2WebProcessMediaStreamUserMediaPermissionRequestManagercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -52,7 +52,7 @@
</span><span class="cx"> UserMediaPermissionRequestManager::~UserMediaPermissionRequestManager()
</span><span class="cx"> {
</span><span class="cx"> for (auto& sandboxExtension : m_userMediaDeviceSandboxExtensions)
</span><del>- sandboxExtension->revoke();
</del><ins>+ sandboxExtension.value->revoke();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void UserMediaPermissionRequestManager::startUserMediaRequest(UserMediaRequest& request)
</span><span class="lines">@@ -202,15 +202,21 @@
</span><span class="cx"> request->setDeviceInfo(deviceList, mediaDeviceIdentifierHashSalt, hasPersistentAccess);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void UserMediaPermissionRequestManager::grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray& sandboxExtensionHandles)
</del><ins>+void UserMediaPermissionRequestManager::grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions& extensions)
</ins><span class="cx"> {
</span><del>- ASSERT(m_userMediaDeviceSandboxExtensions.size() <= 2);
</del><ins>+ for (size_t i = 0; i < extensions.size(); i++) {
+ auto& extension = extensions[i];
+ extension.second->consume();
+ m_userMediaDeviceSandboxExtensions.add(extension.first, extension.second.copyRef());
+ }
+}
</ins><span class="cx">
</span><del>- for (size_t i = 0; i < sandboxExtensionHandles.size(); i++) {
- if (auto extension = SandboxExtension::create(sandboxExtensionHandles[i])) {
- extension->consume();
- m_userMediaDeviceSandboxExtensions.append(extension.release());
- }
</del><ins>+void UserMediaPermissionRequestManager::revokeUserMediaDeviceSandboxExtensions(const Vector<String>& extensionIDs)
+{
+ for (const auto& extensionID : extensionIDs) {
+ auto extension = m_userMediaDeviceSandboxExtensions.take(extensionID);
+ if (extension)
+ extension->revoke();
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessMediaStreamUserMediaPermissionRequestManagerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.h (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.h 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.h 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -22,6 +22,7 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(MEDIA_STREAM)
</span><span class="cx">
</span><ins>+#include "MediaDeviceSandboxExtensions.h"
</ins><span class="cx"> #include "SandboxExtension.h"
</span><span class="cx"> #include <WebCore/MediaCanStartListener.h>
</span><span class="cx"> #include <WebCore/MediaConstraints.h>
</span><span class="lines">@@ -51,7 +52,8 @@
</span><span class="cx"> void cancelMediaDevicesEnumeration(WebCore::MediaDevicesEnumerationRequest&);
</span><span class="cx"> void didCompleteMediaDeviceEnumeration(uint64_t, const Vector<WebCore::CaptureDevice>& deviceList, const String& deviceIdentifierHashSalt, bool originHasPersistentAccess);
</span><span class="cx">
</span><del>- void grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray&);
</del><ins>+ void grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions&);
+ void revokeUserMediaDeviceSandboxExtensions(const Vector<String>&);
</ins><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> void sendUserMediaRequest(WebCore::UserMediaRequest&);
</span><span class="lines">@@ -69,7 +71,7 @@
</span><span class="cx"> HashMap<uint64_t, RefPtr<WebCore::MediaDevicesEnumerationRequest>> m_idToMediaDevicesEnumerationRequestMap;
</span><span class="cx"> HashMap<RefPtr<WebCore::MediaDevicesEnumerationRequest>, uint64_t> m_mediaDevicesEnumerationRequestToIDMap;
</span><span class="cx">
</span><del>- Vector<RefPtr<SandboxExtension>> m_userMediaDeviceSandboxExtensions;
</del><ins>+ HashMap<String, RefPtr<SandboxExtension>> m_userMediaDeviceSandboxExtensions;
</ins><span class="cx">
</span><span class="cx"> HashMap<RefPtr<WebCore::Document>, Vector<RefPtr<WebCore::UserMediaRequest>>> m_blockedRequests;
</span><span class="cx"> };
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPageWebPagecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -3700,10 +3700,15 @@
</span><span class="cx"> m_userMediaPermissionRequestManager.didCompleteMediaDeviceEnumeration(userMediaID, devices, deviceIdentifierHashSalt, originHasPersistentAccess);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void WebPage::grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray& handles)
</del><ins>+void WebPage::grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions& extensions)
</ins><span class="cx"> {
</span><del>- m_userMediaPermissionRequestManager.grantUserMediaDevicesSandboxExtension(handles);
</del><ins>+ m_userMediaPermissionRequestManager.grantUserMediaDeviceSandboxExtensions(extensions);
</ins><span class="cx"> }
</span><ins>+
+void WebPage::revokeUserMediaDeviceSandboxExtensions(const Vector<String>& extensionIDs)
+{
+ m_userMediaPermissionRequestManager.revokeUserMediaDeviceSandboxExtensions(extensionIDs);
+}
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> #if !PLATFORM(IOS)
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPageWebPageh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -1178,7 +1178,8 @@
</span><span class="cx"> void userMediaAccessWasDenied(uint64_t userMediaID, uint64_t reason, String invalidConstraint);
</span><span class="cx">
</span><span class="cx"> void didCompleteMediaDeviceEnumeration(uint64_t userMediaID, const Vector<WebCore::CaptureDevice>& devices, const String& deviceIdentifierHashSalt, bool originHasPersistentAccess);
</span><del>- void grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray&);
</del><ins>+ void grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions&);
+ void revokeUserMediaDeviceSandboxExtensions(const Vector<String>&);
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> void advanceToNextMisspelling(bool startBeforeSelection);
</span></span></pre></div>
<a id="trunkSourceWebKit2WebProcessWebPageWebPagemessagesin"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/WebPage/WebPage.messages.in (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.messages.in 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.messages.in 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -292,7 +292,8 @@
</span><span class="cx"> UserMediaAccessWasGranted(uint64_t userMediaID, String audioDeviceUID, String videoDeviceUID)
</span><span class="cx"> UserMediaAccessWasDenied(uint64_t userMediaID, uint64_t reason, String invalidConstraint)
</span><span class="cx"> DidCompleteMediaDeviceEnumeration(uint64_t userMediaID, Vector<WebCore::CaptureDevice> devices, String mediaDeviceIdentifierHashSalt, bool hasPersistentAccess)
</span><del>- GrantUserMediaDevicesSandboxExtension(WebKit::SandboxExtension::HandleArray sandboxExtensionHandle)
</del><ins>+ GrantUserMediaDeviceSandboxExtensions(WebKit::MediaDeviceSandboxExtensions sandboxExtensions)
+ RevokeUserMediaDeviceSandboxExtensions(Vector<String> sandboxExtensionIDs)
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> # Notification
</span></span></pre></div>
<a id="trunkToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Tools/ChangeLog (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/ChangeLog 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Tools/ChangeLog 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -1,3 +1,21 @@
</span><ins>+2016-12-07 Eric Carlson <eric.carlson@apple.com>
+
+ [MediaStream][Mac] Revoke sandbox extensions when capture ends
+ https://bugs.webkit.org/show_bug.cgi?id=165476
+
+ Reviewed by Brady Eidson.
+
+ Add a test which loads a page that calls navigator.mediaDevices.enumerateDevices, kills the
+ page, and loads the same page again to ensure that the WK2 UserMediaPermissionRequestManagerProxy
+ is reset and recreated when a web page exits and is reloaded.
+
+ * TestWebKitAPI/PlatformGTK.cmake:
+ * TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
+ * TestWebKitAPI/Tests/WebKit2/EnumerateMediaDevices.cpp: Added.
+ (TestWebKitAPI::checkUserMediaPermissionCallback):
+ (TestWebKitAPI::TEST):
+ * TestWebKitAPI/Tests/WebKit2/enumerateMediaDevices.html: Added.
+
</ins><span class="cx"> 2016-12-07 Jeremy Jones <jeremyj@apple.com>
</span><span class="cx">
</span><span class="cx"> One esc to exit fullscreen and pointer lock
</span></span></pre></div>
<a id="trunkToolsTestWebKitAPIPlatformGTKcmake"></a>
<div class="modfile"><h4>Modified: trunk/Tools/TestWebKitAPI/PlatformGTK.cmake (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/PlatformGTK.cmake 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Tools/TestWebKitAPI/PlatformGTK.cmake 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -75,6 +75,7 @@
</span><span class="cx"> ${TESTWEBKITAPI_DIR}/Tests/WebKit2/DOMWindowExtensionBasic.cpp
</span><span class="cx"> ${TESTWEBKITAPI_DIR}/Tests/WebKit2/DOMWindowExtensionNoCache.cpp
</span><span class="cx"> ${TESTWEBKITAPI_DIR}/Tests/WebKit2/DownloadDecideDestinationCrash.cpp
</span><ins>+ ${TESTWEBKITAPI_DIR}/Tests/WebKit2/EnumerateMediaDevices.cpp
</ins><span class="cx"> ${TESTWEBKITAPI_DIR}/Tests/WebKit2/EvaluateJavaScript.cpp
</span><span class="cx"> ${TESTWEBKITAPI_DIR}/Tests/WebKit2/FailedLoad.cpp
</span><span class="cx"> ${TESTWEBKITAPI_DIR}/Tests/WebKit2/Find.cpp
</span></span></pre></div>
<a id="trunkToolsTestWebKitAPITestWebKitAPIxcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj (209511 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj 2016-12-08 02:18:27 UTC (rev 209511)
+++ trunk/Tools/TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -22,6 +22,8 @@
</span><span class="cx"> /* End PBXAggregateTarget section */
</span><span class="cx">
</span><span class="cx"> /* Begin PBXBuildFile section */
</span><ins>+ 07492B3B1DF8B14C00633DE1 /* EnumerateMediaDevices.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 07492B3A1DF8AE2D00633DE1 /* EnumerateMediaDevices.cpp */; };
+ 07492B3C1DF8B86600633DE1 /* enumerateMediaDevices.html in Copy Resources */ = {isa = PBXBuildFile; fileRef = 07492B391DF8ADA400633DE1 /* enumerateMediaDevices.html */; };
</ins><span class="cx"> 0F139E771A423A5B00F590F5 /* WeakObjCPtr.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0F139E751A423A5300F590F5 /* WeakObjCPtr.mm */; };
</span><span class="cx"> 0F139E781A423A6B00F590F5 /* PlatformUtilitiesCocoa.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0F139E721A423A2B00F590F5 /* PlatformUtilitiesCocoa.mm */; };
</span><span class="cx"> 0F139E791A42457000F590F5 /* PlatformUtilitiesCocoa.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0F139E721A423A2B00F590F5 /* PlatformUtilitiesCocoa.mm */; };
</span><span class="lines">@@ -583,6 +585,7 @@
</span><span class="cx"> dstPath = TestWebKitAPI.resources;
</span><span class="cx"> dstSubfolderSpec = 7;
</span><span class="cx"> files = (
</span><ins>+ 07492B3C1DF8B86600633DE1 /* enumerateMediaDevices.html in Copy Resources */,
</ins><span class="cx"> 9B270FEE1DDC2C0B002D53F3 /* closed-shadow-tree-test.html in Copy Resources */,
</span><span class="cx"> F4C2AB221DD6D95E00E06D5B /* enormous-video-with-sound.html in Copy Resources */,
</span><span class="cx"> 8349D3C41DB9728E004A9F65 /* link-with-download-attribute.html in Copy Resources */,
</span><span class="lines">@@ -734,6 +737,8 @@
</span><span class="cx"> /* Begin PBXFileReference section */
</span><span class="cx"> 00BC16851680FE810065F1E5 /* PublicSuffix.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = PublicSuffix.mm; sourceTree = "<group>"; };
</span><span class="cx"> 00CD9F6215BE312C002DA2CE /* BackForwardList.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = BackForwardList.mm; sourceTree = "<group>"; };
</span><ins>+ 07492B391DF8ADA400633DE1 /* enumerateMediaDevices.html */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text.html; path = enumerateMediaDevices.html; sourceTree = "<group>"; };
+ 07492B3A1DF8AE2D00633DE1 /* EnumerateMediaDevices.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = EnumerateMediaDevices.cpp; sourceTree = "<group>"; };
</ins><span class="cx"> 0766DD1F1A5AD5200023E3BB /* PendingAPIRequestURL.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = PendingAPIRequestURL.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0BCD833414857CE400EA2003 /* HashMap.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = HashMap.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0BCD85691485C98B00EA2003 /* SetForScope.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = SetForScope.cpp; sourceTree = "<group>"; };
</span><span class="lines">@@ -1699,6 +1704,7 @@
</span><span class="cx"> BC9096411255616000083756 /* WebKit2 */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><ins>+ 07492B3A1DF8AE2D00633DE1 /* EnumerateMediaDevices.cpp */,
</ins><span class="cx"> AD57AC1E1DA7464D00FF1BDE /* DidRemoveFrameFromHiearchyInPageCache_Bundle.cpp */,
</span><span class="cx"> AD57AC1F1DA7464D00FF1BDE /* DidRemoveFrameFromHiearchyInPageCache.cpp */,
</span><span class="cx"> 0F139E741A423A4600F590F5 /* cocoa */,
</span><span class="lines">@@ -1877,6 +1883,7 @@
</span><span class="cx"> BC90977B125571AE00083756 /* Resources */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><ins>+ 07492B391DF8ADA400633DE1 /* enumerateMediaDevices.html */,
</ins><span class="cx"> C045F9461385C2F800C0F3CD /* 18-characters.html */,
</span><span class="cx"> 1C2B81851C89252300A5529F /* Ahem.ttf */,
</span><span class="cx"> 93D3D19B17B1A7B000C7C415 /* all-content-in-one-iframe.html */,
</span><span class="lines">@@ -2587,6 +2594,7 @@
</span><span class="cx"> 7CCE7ED31A411A7E00447C4C /* TypingStyleCrash.mm in Sources */,
</span><span class="cx"> 7CCE7EDE1A411A9200447C4C /* URL.cpp in Sources */,
</span><span class="cx"> 7CCE7EB01A411A4400447C4C /* URLExtras.mm in Sources */,
</span><ins>+ 07492B3B1DF8B14C00633DE1 /* EnumerateMediaDevices.cpp in Sources */,
</ins><span class="cx"> 7A6A2C701DCCFA8C00C0D085 /* LocalStorageQuirkTest.mm in Sources */,
</span><span class="cx"> 2DFF7B6D1DA487AF00814614 /* SnapshotStore.mm in Sources */,
</span><span class="cx"> 5C6E65441D5CEFD400F7862E /* URLParser.cpp in Sources */,
</span></span></pre></div>
<a id="trunkToolsTestWebKitAPITestsWebKit2EnumerateMediaDevicescpp"></a>
<div class="addfile"><h4>Added: trunk/Tools/TestWebKitAPI/Tests/WebKit2/EnumerateMediaDevices.cpp (0 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/Tests/WebKit2/EnumerateMediaDevices.cpp (rev 0)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit2/EnumerateMediaDevices.cpp 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -0,0 +1,83 @@
</span><ins>+/*
+ * Copyright (C) 2016 Apple Inc. All rights reserved.
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#include "config.h"
+
+#if WK_HAVE_C_SPI
+
+#if ENABLE(MEDIA_STREAM)
+#include "PlatformUtilities.h"
+#include "PlatformWebView.h"
+#include "Test.h"
+#include <WebKit/WKPreferencesRef.h>
+#include <WebKit/WKPreferencesRefPrivate.h>
+#include <WebKit/WKRetainPtr.h>
+#include <WebKit/WKUserMediaPermissionCheck.h>
+#include <string.h>
+#include <vector>
+
+namespace TestWebKitAPI {
+
+static bool loadedFirstTime;
+static bool loadedSecondTime;
+
+void checkUserMediaPermissionCallback(WKPageRef, WKFrameRef, WKSecurityOriginRef, WKSecurityOriginRef, WKUserMediaPermissionCheckRef permissionRequest, const void*)
+{
+ WKUserMediaPermissionCheckSetUserMediaAccessInfo(permissionRequest, WKStringCreateWithUTF8CString("0x123456789"), true);
+ if (!loadedFirstTime) {
+ loadedFirstTime = true;
+ return;
+ }
+
+ loadedSecondTime = true;
+}
+
+TEST(WebKit2, EnumerateDevices)
+{
+ auto context = adoptWK(WKContextCreate());
+
+ WKRetainPtr<WKPageGroupRef> pageGroup(AdoptWK, WKPageGroupCreateWithIdentifier(Util::toWK("EnumerateDevices").get()));
+ WKPreferencesRef preferences = WKPageGroupGetPreferences(pageGroup.get());
+ WKPreferencesSetFileAccessFromFileURLsAllowed(preferences, true);
+ WKPreferencesSetMediaCaptureRequiresSecureConnection(preferences, false);
+
+ WKPageUIClientV6 uiClient;
+ memset(&uiClient, 0, sizeof(uiClient));
+ uiClient.base.version = 6;
+ uiClient.checkUserMediaPermissionForOrigin = checkUserMediaPermissionCallback;
+
+ PlatformWebView webView(context.get());
+ WKPageSetPageUIClient(webView.page(), &uiClient.base);
+
+ auto url = adoptWK(Util::createURLForResource("enumerateMediaDevices", "html"));
+
+ // Load and kill the page.
+ WKPageLoadURL(webView.page(), url.get());
+ Util::run(&loadedFirstTime);
+ WKPageTerminate(webView.page());
+
+ // Load it again to make sure the user media process manager doesn't assert.
+ WKPageLoadURL(webView.page(), url.get());
+ Util::run(&loadedSecondTime);
+}
+
+} // namespace TestWebKitAPI
+
+#endif // ENABLE(MEDIA_STREAM)
+
+#endif // WK_HAVE_C_SPI
</ins></span></pre></div>
<a id="trunkToolsTestWebKitAPITestsWebKit2enumerateMediaDeviceshtml"></a>
<div class="addfile"><h4>Added: trunk/Tools/TestWebKitAPI/Tests/WebKit2/enumerateMediaDevices.html (0 => 209512)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/TestWebKitAPI/Tests/WebKit2/enumerateMediaDevices.html (rev 0)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit2/enumerateMediaDevices.html 2016-12-08 02:25:57 UTC (rev 209512)
</span><span class="lines">@@ -0,0 +1,24 @@
</span><ins>+<html>
+ <head>
+ <script>
+ function enumerate(devices)
+ {
+ console.log("enumerateDevices succeeded");
+ }
+
+ function userMediaError(error)
+ {
+ console.log(error);
+ }
+
+ function test()
+ {
+ navigator.mediaDevices.enumerateDevices()
+ .then(enumerate)
+ .catch(userMediaError);
+ }
+ </script>
+ </head>
+ <body onload="test()">
+ </body>
+</html>
</ins></span></pre>
</div>
</div>
</body>
</html>