<!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>[209449] trunk/Source/WebKit2</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/209449">209449</a></dd>
<dt>Author</dt> <dd>ryanhaddad@apple.com</dd>
<dt>Date</dt> <dd>2016-12-06 21:31:57 -0800 (Tue, 06 Dec 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>Unreviewed, rolling out <a href="http://trac.webkit.org/projects/webkit/changeset/209422">r209422</a>.
This change caused assertion failures during API tests.
Reverted changeset:
"[MediaStream][Mac] Revoke sandbox extensions when capture
ends"
https://bugs.webkit.org/show_bug.cgi?id=165476
http://trac.webkit.org/changeset/209422</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>
</ul>
<h3>Removed 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>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkSourceWebKit2CMakeListstxt"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/CMakeLists.txt (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/CMakeLists.txt 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/CMakeLists.txt 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -369,7 +369,6 @@
</span><span class="cx"> UIProcess/UserMediaPermissionCheckProxy.cpp
</span><span class="cx"> UIProcess/UserMediaPermissionRequestManagerProxy.cpp
</span><span class="cx"> UIProcess/UserMediaPermissionRequestProxy.cpp
</span><del>- UIProcess/UserMediaProcessManager.cpp
</del><span class="cx"> UIProcess/VisitedLinkStore.cpp
</span><span class="cx"> UIProcess/WebBackForwardList.cpp
</span><span class="cx"> UIProcess/WebColorPicker.cpp
</span><span class="lines">@@ -582,7 +581,6 @@
</span><span class="cx"> WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp
</span><span class="cx"> WebProcess/InjectedBundle/DOM/InjectedBundleRangeHandle.cpp
</span><span class="cx">
</span><del>- WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp
</del><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 (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/ChangeLog 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/ChangeLog 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2016-12-06 Ryan Haddad <ryanhaddad@apple.com>
+
+ Unreviewed, rolling out r209422.
+
+ This change caused assertion failures during API tests.
+
+ Reverted changeset:
+
+ "[MediaStream][Mac] Revoke sandbox extensions when capture
+ ends"
+ https://bugs.webkit.org/show_bug.cgi?id=165476
+ http://trac.webkit.org/changeset/209422
+
</ins><span class="cx"> 2016-12-06 Dean Jackson <dino@apple.com>
</span><span class="cx">
</span><span class="cx"> Include default value in Experimental Feature information
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaPermissionRequestManagerProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.cpp 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -22,7 +22,6 @@
</span><span class="cx">
</span><span class="cx"> #include "APISecurityOrigin.h"
</span><span class="cx"> #include "APIUIClient.h"
</span><del>-#include "UserMediaProcessManager.h"
</del><span class="cx"> #include "WebPageMessages.h"
</span><span class="cx"> #include "WebPageProxy.h"
</span><span class="cx"> #include "WebProcessProxy.h"
</span><span class="lines">@@ -93,14 +92,8 @@
</span><span class="cx"> UserMediaPermissionRequestManagerProxy::UserMediaPermissionRequestManagerProxy(WebPageProxy& page)
</span><span class="cx"> : m_page(page)
</span><span class="cx"> {
</span><del>- UserMediaProcessManager::singleton().addUserMediaPermissionRequestManagerProxy(*this);
</del><span class="cx"> }
</span><span class="cx">
</span><del>-UserMediaPermissionRequestManagerProxy::~UserMediaPermissionRequestManagerProxy()
-{
- UserMediaProcessManager::singleton().removeUserMediaPermissionRequestManagerProxy(*this);
-}
-
</del><span class="cx"> void UserMediaPermissionRequestManagerProxy::invalidateRequests()
</span><span class="cx"> {
</span><span class="cx"> for (auto& request : m_pendingUserMediaRequests.values())
</span><span class="lines">@@ -204,8 +197,30 @@
</span><span class="cx"> fameState.setHasPermissionToUseCaptureDevice(audioDeviceUID, true);
</span><span class="cx"> fameState.setHasPermissionToUseCaptureDevice(videoDeviceUID, true);
</span><span class="cx">
</span><del>- UserMediaProcessManager::singleton().willCreateMediaStream(*this, !audioDeviceUID.isEmpty(), !videoDeviceUID.isEmpty());
</del><ins>+ size_t extensionCount = 0;
+ unsigned requiredExtensions = SandboxExtensionsGranted::None;
+ if (!audioDeviceUID.isEmpty()) {
+ requiredExtensions |= SandboxExtensionsGranted::Audio;
+ extensionCount++;
+ }
+ if (!videoDeviceUID.isEmpty()) {
+ requiredExtensions |= SandboxExtensionsGranted::Video;
+ extensionCount++;
+ }
</ins><span class="cx">
</span><ins>+ 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());
+ }
+
</ins><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">@@ -327,34 +342,4 @@
</span><span class="cx"> #endif
</span><span class="cx"> }
</span><span class="cx">
</span><del>-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
-}
-
</del><span class="cx"> } // namespace WebKit
</span></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaPermissionRequestManagerProxyh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/UIProcess/UserMediaPermissionRequestManagerProxy.h 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -53,10 +53,7 @@
</span><span class="cx"> class UserMediaPermissionRequestManagerProxy {
</span><span class="cx"> public:
</span><span class="cx"> explicit UserMediaPermissionRequestManagerProxy(WebPageProxy&);
</span><del>- ~UserMediaPermissionRequestManagerProxy();
</del><span class="cx">
</span><del>- WebPageProxy& page() const { return m_page; }
-
</del><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">@@ -71,10 +68,6 @@
</span><span class="cx">
</span><span class="cx"> void clearCachedState();
</span><span class="cx">
</span><del>- void startedCaptureSession();
- void endedCaptureSession();
- void stopCapture();
-
</del><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">@@ -83,6 +76,14 @@
</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><ins>+
+ enum SandboxExtensionsGranted {
+ None = 0,
+ Video = 1 << 0,
+ Audio = 1 << 1
+ };
+ unsigned m_pageSandboxExtensionsGranted;
+
</ins><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="delfile"><h4>Deleted: trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.cpp 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1,198 +0,0 @@
</span><del>-/*
- * 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
</del></span></pre></div>
<a id="trunkSourceWebKit2UIProcessUserMediaProcessManagerh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/UIProcess/UserMediaProcessManager.h 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1,45 +0,0 @@
</span><del>-/*
- * 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
</del></span></pre></div>
<a id="trunkSourceWebKit2UIProcessWebPageProxycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.cpp 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -6409,21 +6409,15 @@
</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><ins>+ if (!oldMediaStateHasActiveCapture && newMediaStateHasActiveCapture)
+ m_uiClient->didBeginCaptureSession();
+ if (oldMediaStateHasActiveCapture && !newMediaStateHasActiveCapture)
+ m_uiClient->didEndCaptureSession();
</ins><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><del>-#if ENABLE(MEDIA_STREAM)
- if (!oldMediaStateHasActiveCapture && newMediaStateHasActiveCapture) {
- m_uiClient->didBeginCaptureSession();
- m_userMediaPermissionRequestManager.startedCaptureSession();
- } else if (oldMediaStateHasActiveCapture && !newMediaStateHasActiveCapture) {
- m_uiClient->didEndCaptureSession();
- m_userMediaPermissionRequestManager.endedCaptureSession();
- }
-#endif
-
</del><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 (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/UIProcess/WebPageProxy.h 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/UIProcess/WebPageProxy.h 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1049,8 +1049,6 @@
</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><del>- bool hasActiveAudioStream() const { return m_mediaState & WebCore::MediaProducer::HasActiveAudioCaptureDevice; }
- bool hasActiveVideoStream() const { return m_mediaState & WebCore::MediaProducer::HasActiveVideoCaptureDevice; }
</del><span class="cx"> WebCore::MediaProducer::MediaStateFlags mediaStateFlags() const { return m_mediaState; }
</span><span class="cx">
</span><span class="cx"> #if PLATFORM(MAC)
</span></span></pre></div>
<a id="trunkSourceWebKit2WebKit2xcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebKit2.xcodeproj/project.pbxproj 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -58,9 +58,6 @@
</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><del>- 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 */; };
</del><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">@@ -2064,10 +2061,6 @@
</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><del>- 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>"; };
</del><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">@@ -5483,8 +5476,6 @@
</span><span class="cx"> 4A410F4119AF7B27002EBAB5 /* MediaStream */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><del>- 074E76001DF7075D00D318EC /* MediaDeviceSandboxExtensions.cpp */,
- 074E76011DF7075D00D318EC /* MediaDeviceSandboxExtensions.h */,
</del><span class="cx"> 4A410F4219AF7B27002EBAB5 /* UserMediaPermissionRequestManager.cpp */,
</span><span class="cx"> 4A410F4319AF7B27002EBAB5 /* UserMediaPermissionRequestManager.h */,
</span><span class="cx"> );
</span><span class="lines">@@ -6165,8 +6156,6 @@
</span><span class="cx"> BC032DC310F438260058C15A /* UIProcess */ = {
</span><span class="cx"> isa = PBXGroup;
</span><span class="cx"> children = (
</span><del>- 074E75FC1DF2002400D318EC /* UserMediaProcessManager.cpp */,
- 074E75FB1DF1FD1300D318EC /* UserMediaProcessManager.h */,
</del><span class="cx"> BC032DC410F4387C0058C15A /* API */,
</span><span class="cx"> 1AB1F7761D1B30A9007C9BD1 /* ApplePay */,
</span><span class="cx"> 512F588D12A8836F00629530 /* Authentication */,
</span><span class="lines">@@ -8311,7 +8300,6 @@
</span><span class="cx"> 532159551DBAE7290054AA3C /* NetworkSessionCocoa.h in Headers */,
</span><span class="cx"> 0FCB4E4F18BBE044000FCFC9 /* WKGeolocationProviderIOS.h in Headers */,
</span><span class="cx"> BCC8B374125FB69000DE46A4 /* WKGeometry.h in Headers */,
</span><del>- 074E75FE1DF2211900D318EC /* UserMediaProcessManager.h in Headers */,
</del><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">@@ -9375,7 +9363,6 @@
</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><del>- 074E75FD1DF2211500D318EC /* UserMediaProcessManager.cpp in Sources */,
</del><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">@@ -9840,7 +9827,6 @@
</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><del>- 074E76021DF707BE00D318EC /* MediaDeviceSandboxExtensions.cpp in Sources */,
</del><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="delfile"><h4>Deleted: trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.cpp 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1,80 +0,0 @@
</span><del>-/*
- * 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)
</del></span></pre></div>
<a id="trunkSourceWebKit2WebProcessMediaStreamMediaDeviceSandboxExtensionsh"></a>
<div class="delfile"><h4>Deleted: trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/MediaDeviceSandboxExtensions.h 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1,59 +0,0 @@
</span><del>-/*
- * 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)
</del></span></pre></div>
<a id="trunkSourceWebKit2WebProcessMediaStreamUserMediaPermissionRequestManagercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.cpp 2016-12-07 05:31:57 UTC (rev 209449)
</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.value->revoke();
</del><ins>+ sandboxExtension->revoke();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void UserMediaPermissionRequestManager::startUserMediaRequest(UserMediaRequest& request)
</span><span class="lines">@@ -202,21 +202,15 @@
</span><span class="cx"> request->setDeviceInfo(deviceList, mediaDeviceIdentifierHashSalt, hasPersistentAccess);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void UserMediaPermissionRequestManager::grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions& extensions)
</del><ins>+void UserMediaPermissionRequestManager::grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray& sandboxExtensionHandles)
</ins><span class="cx"> {
</span><del>- for (size_t i = 0; i < extensions.size(); i++) {
- auto& extension = extensions[i];
- extension.second->consume();
- m_userMediaDeviceSandboxExtensions.add(extension.first, extension.second.copyRef());
- }
-}
</del><ins>+ ASSERT(m_userMediaDeviceSandboxExtensions.size() <= 2);
</ins><span class="cx">
</span><del>-void UserMediaPermissionRequestManager::revokeUserMediaDeviceSandboxExtensions(const Vector<String>& extensionIDs)
-{
- for (const auto& extensionID : extensionIDs) {
- auto extension = m_userMediaDeviceSandboxExtensions.take(extensionID);
- if (extension)
- extension->revoke();
</del><ins>+ for (size_t i = 0; i < sandboxExtensionHandles.size(); i++) {
+ if (auto extension = SandboxExtension::create(sandboxExtensionHandles[i])) {
+ extension->consume();
+ m_userMediaDeviceSandboxExtensions.append(extension.release());
+ }
</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 (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.h 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/MediaStream/UserMediaPermissionRequestManager.h 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -22,7 +22,6 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(MEDIA_STREAM)
</span><span class="cx">
</span><del>-#include "MediaDeviceSandboxExtensions.h"
</del><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">@@ -52,8 +51,7 @@
</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 grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions&);
- void revokeUserMediaDeviceSandboxExtensions(const Vector<String>&);
</del><ins>+ void grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray&);
</ins><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> void sendUserMediaRequest(WebCore::UserMediaRequest&);
</span><span class="lines">@@ -71,7 +69,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>- HashMap<String, RefPtr<SandboxExtension>> m_userMediaDeviceSandboxExtensions;
</del><ins>+ Vector<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 (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.cpp 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -3700,15 +3700,10 @@
</span><span class="cx"> m_userMediaPermissionRequestManager.didCompleteMediaDeviceEnumeration(userMediaID, devices, deviceIdentifierHashSalt, originHasPersistentAccess);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void WebPage::grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions& extensions)
</del><ins>+void WebPage::grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray& handles)
</ins><span class="cx"> {
</span><del>- m_userMediaPermissionRequestManager.grantUserMediaDeviceSandboxExtensions(extensions);
</del><ins>+ m_userMediaPermissionRequestManager.grantUserMediaDevicesSandboxExtension(handles);
</ins><span class="cx"> }
</span><del>-
-void WebPage::revokeUserMediaDeviceSandboxExtensions(const Vector<String>& extensionIDs)
-{
- m_userMediaPermissionRequestManager.revokeUserMediaDeviceSandboxExtensions(extensionIDs);
-}
</del><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 (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.h 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -1178,8 +1178,7 @@
</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 grantUserMediaDeviceSandboxExtensions(const MediaDeviceSandboxExtensions&);
- void revokeUserMediaDeviceSandboxExtensions(const Vector<String>&);
</del><ins>+ void grantUserMediaDevicesSandboxExtension(const SandboxExtension::HandleArray&);
</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 (209448 => 209449)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebKit2/WebProcess/WebPage/WebPage.messages.in 2016-12-07 05:18:56 UTC (rev 209448)
+++ trunk/Source/WebKit2/WebProcess/WebPage/WebPage.messages.in 2016-12-07 05:31:57 UTC (rev 209449)
</span><span class="lines">@@ -292,8 +292,7 @@
</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>- GrantUserMediaDeviceSandboxExtensions(WebKit::MediaDeviceSandboxExtensions sandboxExtensions)
- RevokeUserMediaDeviceSandboxExtensions(Vector<String> sandboxExtensionIDs)
</del><ins>+ GrantUserMediaDevicesSandboxExtension(WebKit::SandboxExtension::HandleArray sandboxExtensionHandle)
</ins><span class="cx"> #endif
</span><span class="cx">
</span><span class="cx"> # Notification
</span></span></pre>
</div>
</div>
</body>
</html>