Diff
Modified: trunk/LayoutTests/ChangeLog (211911 => 211912)
--- trunk/LayoutTests/ChangeLog 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/ChangeLog 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,3 +1,26 @@
+2017-02-08 Eric Carlson <eric.carl...@apple.com>
+
+ [MediaStream] Remove legacy Navigator.webkitGetUserMedia
+ https://bugs.webkit.org/show_bug.cgi?id=168016
+
+ Reviewed by Youenn Fablet.
+
+ Update valid tests to use navigator.mediaDevices.getUserMedia. Remove tests
+ specific to Navigator.webkitGetUserMedia.
+
+ * fast/mediastream/RTCPeerConnection-AddRemoveStream.html:
+ * fast/mediastream/argument-types.html: Removed.
+ * fast/mediastream/delayed-permission-allowed.html:
+ * fast/mediastream/delayed-permission-denied.html:
+ * fast/mediastream/enabled-expected.txt:
+ * fast/mediastream/error.html:
+ * fast/mediastream/getusermedia.html: Removed.
+ * fast/mediastream/script-tests/argument-types.js: Removed..
+ * fast/mediastream/script-tests/enabled.js:
+ (hasGetUserMediaProperty):
+ * fast/mediastream/success.html:
+ * fast/mediastream/webkitGetUserMedia-shadowing-then.html: Removed.
+
2017-02-08 Chris Dumez <cdu...@apple.com>
Use smaller tiles in windows that are not active to facilitate App Napping
Modified: trunk/LayoutTests/fast/mediastream/RTCPeerConnection-AddRemoveStream.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/RTCPeerConnection-AddRemoveStream.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/RTCPeerConnection-AddRemoveStream.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -2,6 +2,7 @@
<html>
<head>
<script src=""
+ <script src=""
</head>
<body>
<script>
@@ -20,7 +21,9 @@
function getUserMedia(dictionary, callback)
{
try {
- navigator.webkitGetUserMedia(dictionary, callback, error);
+ navigator.mediaDevices.getUserMedia(dictionary)
+ .then(callback)
+ .catch(error);
} catch (e) {
testFailed('webkitGetUserMedia threw exception :' + e);
finishJSTest();
Deleted: trunk/LayoutTests/fast/mediastream/argument-types.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/argument-types.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/argument-types.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,10 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
- <head>
- <script src=""
- </head>
- <body>
- <script src=""
- <script src=""
- </body>
-</html>
Modified: trunk/LayoutTests/fast/mediastream/delayed-permission-allowed.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/delayed-permission-allowed.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/delayed-permission-allowed.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -15,17 +15,19 @@
}
var options = {audio: true, video: true};
-navigator.webkitGetUserMedia(options, function(stream) {
- if (permissionSet) {
- testPassed('Success callback invoked');
+navigator.mediaDevices.getUserMedia(options)
+ .then(stream => {
+ if (permissionSet) {
+ testPassed('Success callback invoked');
+ finishJSTest();
+ return;
+ }
+ testFailed('Success callback invoked unexpectedly');
+ })
+ .catch(err => {
+ testFailed('Error callback invoked unexpectedly');
finishJSTest();
- return;
- }
- testFailed('Success callback invoked unexpectedly');
-}, function(e) {
- testFailed('Error callback invoked unexpectedly');
- finishJSTest();
-});
+ });
window.setTimeout(allowPermission, 100);
</script>
Modified: trunk/LayoutTests/fast/mediastream/delayed-permission-denied.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/delayed-permission-denied.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/delayed-permission-denied.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -16,19 +16,19 @@
var error;
var options = {audio: true, video: true};
-navigator.webkitGetUserMedia(options, function(stream) {
- testFailed('Success callback invoked unexpectedly');
- finishJSTest();
-}, function(e) {
- if (permissionSet) {
- error = e;
- shouldBeEqualToString('error.name', 'NotAllowedError');
+navigator.mediaDevices.getUserMedia(options)
+ .then(stream => {
+ testFailed('Success callback invoked unexpectedly');
finishJSTest();
- return;
- }
- testFailed('Error callback invoked unexpectedly');
- finishJSTest();
-});
+ })
+ .catch(err => {
+ if (permissionSet) {
+ error = err;
+ shouldBeEqualToString('error.name', 'NotAllowedError');
+ } else
+ testFailed('Error callback invoked unexpectedly');
+ finishJSTest();
+ });
window.setTimeout(denyPermission, 100);
</script>
Modified: trunk/LayoutTests/fast/mediastream/enabled-expected.txt (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/enabled-expected.txt 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/enabled-expected.txt 2017-02-09 01:30:56 UTC (rev 211912)
@@ -3,10 +3,12 @@
On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
-PASS typeof navigator.webkitGetUserMedia == 'function' is true
+PASS typeof navigator.mediaDevices == 'object' is true
+PASS typeof navigator.mediaDevices.getUserMedia == 'function' is true
PASS hasGetUserMediaProperty() is true
-PASS 'webkitGetUserMedia' in navigator is true
-PASS navigator.hasOwnProperty('webkitGetUserMedia') is false
+PASS 'mediaDevices' in navigator is true
+PASS 'getUserMedia' in navigator.mediaDevices is true
+PASS navigator.hasOwnProperty('mediaDevices') is false
PASS successfullyParsed is true
TEST COMPLETE
Modified: trunk/LayoutTests/fast/mediastream/error.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/error.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/error.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -15,14 +15,16 @@
var error;
var options = {audio:false, video:true};
-navigator.webkitGetUserMedia(options, function(stream) {
- testFailed('Success callback invoked unexpectedly');
- finishJSTest();
-}, function(e) {
- error = e;
- shouldBeEqualToString('error.name', 'NotAllowedError');
- finishJSTest();
-});
+navigator.mediaDevices.getUserMedia(options)
+ .then(stream => {
+ testFailed('Success callback invoked unexpectedly');
+ finishJSTest();
+ })
+ .catch(err => {
+ error = err;
+ shouldBeEqualToString('error.name', 'NotAllowedError');
+ finishJSTest();
+ });
</script>
<script src=""
</body>
Deleted: trunk/LayoutTests/fast/mediastream/getusermedia.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/getusermedia.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/getusermedia.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,105 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML//EN">
-<html>
- <head>
- <script src=""
- </head>
- <body>
- <p id="description"></p>
- <div id="console"></div>
- <script>
- description("Tests webkitGetUserMedia.");
-
- function setUserMediaPermission(isGranted) {
- if (window.testRunner)
- testRunner.setUserMediaPermission(isGranted);
- else
- debug("This test can not be run without the testRunner");
- }
-
- var stream;
- var errorArg;
-
- function error() {
- testFailed('Error callback called.');
- finishJSTest();
- }
-
- function expectedError() {
- testPassed('Error callback called.');
- }
-
- function gotStreamInError(s) {
- testFailed('Stream generated.');
- finishJSTest();
- }
-
- function gotStream5(s) {
- stream = s;
- testPassed('Stream generated.');
- shouldBe('stream.getAudioTracks().length', '1');
- shouldBe('stream.getVideoTracks().length', '1');
- finishJSTest();
- }
-
- function error1(e) {
- errorArg = e;
- testPassed('Error callback called.');
- shouldBeEqualToString('errorArg.name', 'OverconstrainedError');
- shouldBeEqualToString('errorArg.message', 'Invalid constraint');
-
- shouldThrow("navigator.webkitGetUserMedia({audio:{volume:{exact:1}}, video:true}, gotStream5, 0);");
- shouldNotThrow("navigator.webkitGetUserMedia({audio:{volume:{exact:1}}, video:true}, gotStream5, error);");
- }
-
- function gotStream4(s) {
- stream = s;
- testPassed('Stream generated.');
- shouldBe('stream.getAudioTracks().length', '1');
- shouldBe('stream.getVideoTracks().length', '1');
-
- shouldThrow("navigator.webkitGetUserMedia({audio:true, video:{width:{exact:11}}}, gotStreamInError, null);");
- shouldNotThrow("navigator.webkitGetUserMedia({audio:true, video:{width:{exact:11}}}, gotStreamInError, error1);");
- }
-
- function gotStream3(s) {
- stream = s;
- testPassed('Stream generated.');
- shouldBe('stream.getAudioTracks().length', '1');
- shouldBe('stream.getVideoTracks().length', '1');
-
- shouldThrow("navigator.webkitGetUserMedia({audio:{}, video:true}, gotStream4, null);");
- shouldNotThrow("navigator.webkitGetUserMedia({audio:{}, video:true}, gotStream4, error);");
- }
-
- function gotStream2(s) {
- stream = s;
- testPassed('Stream generated.');
- shouldBe('stream.getAudioTracks().length', '0');
- shouldBe('stream.getVideoTracks().length', '1');
-
- shouldThrow("navigator.webkitGetUserMedia({audio:true, video:true}, gotStream3, null);");
- shouldNotThrow("navigator.webkitGetUserMedia({audio:true, video:true}, gotStream3, error);");
- }
-
- function gotStream1(s) {
- stream = s;
- testPassed('Stream generated.');
- shouldBe('stream.getAudioTracks().length', '1');
- shouldBe('stream.getVideoTracks().length', '0');
-
- shouldThrow("navigator.webkitGetUserMedia({video:true}, gotStream2, null);")
- shouldNotThrow("navigator.webkitGetUserMedia({video:true}, gotStream2, error);")
- }
-
- shouldNotThrow("navigator.webkitGetUserMedia({audio:false, video:false}, error, expectedError);")
- shouldThrow("navigator.webkitGetUserMedia({audio:false, video:false}, error, null);")
-
- setUserMediaPermission(true);
- shouldNotThrow("navigator.webkitGetUserMedia({audio:true}, gotStream1, error);");
-
- window.jsTestIsAsync = true;
- window.successfullyParsed = true;
- </script>
- <script src=""
- </body>
-</html>
Deleted: trunk/LayoutTests/fast/mediastream/script-tests/argument-types.js (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/script-tests/argument-types.js 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/script-tests/argument-types.js 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,70 +0,0 @@
-description("Tests the acceptable types for arguments to navigator.getUserMedia methods.");
-
-function test(_expression_, expressionShouldThrow, expectedException) {
- if (expressionShouldThrow) {
- if (expectedException)
- shouldThrow(_expression_, '"' + expectedException + '"');
- else
- shouldThrow(_expression_, '"TypeError: Not enough arguments"');
- } else {
- shouldNotThrow(_expression_);
- }
-}
-
-var errorCallbackError = new TypeError("Argument 3 ('errorCallback') to Navigator.webkitGetUserMedia must be a function")
-var invalidDictionaryError = new TypeError('First argument of webkitGetUserMedia must be a valid Dictionary')
-var notSupportedError = new Error('NotSupportedError: DOM Exception 9');
-var successCallbackError = new TypeError("Argument 2 ('successCallback') to Navigator.webkitGetUserMedia must be a function")
-var typeError = new TypeError('Type error');
-var typeNotAnObjectError = new TypeError('Not an object.');
-
-var emptyFunction = function() {};
-
-// No arguments
-test('navigator.webkitGetUserMedia()', true);
-
-// 1 Argument (Navigtor.webkitGetUserMedia requires at least 3 arguments).
-test('navigator.webkitGetUserMedia(undefined)', true);
-test('navigator.webkitGetUserMedia(null)', true);
-test('navigator.webkitGetUserMedia({ })', true);
-test('navigator.webkitGetUserMedia({video: true})', true);
-test('navigator.webkitGetUserMedia(true)', true);
-test('navigator.webkitGetUserMedia(42)', true);
-test('navigator.webkitGetUserMedia(Infinity)', true);
-test('navigator.webkitGetUserMedia(-Infinity)', true);
-test('navigator.webkitGetUserMedia(emptyFunction)', true);
-
-// 2 Arguments.
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(undefined, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(null, emptyFunction)', true);
-test('navigator.webkitGetUserMedia({ }, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(true, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(42, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(Infinity, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(-Infinity, emptyFunction)', true);
-test('navigator.webkitGetUserMedia(emptyFunction, emptyFunction)', true);
-test('navigator.webkitGetUserMedia({video: true}, "foobar")', true);
-test('navigator.webkitGetUserMedia({video: true}, undefined)', true);
-test('navigator.webkitGetUserMedia({video: true}, null)', true);
-test('navigator.webkitGetUserMedia({video: true}, {})', true);
-test('navigator.webkitGetUserMedia({video: true}, true)', true);
-test('navigator.webkitGetUserMedia({video: true}, 42)', true);
-test('navigator.webkitGetUserMedia({video: true}, Infinity)', true);
-test('navigator.webkitGetUserMedia({video: true}, -Infinity)', true);
-
-// 3 Arguments.
-test('navigator.webkitGetUserMedia({ }, emptyFunction, emptyFunction)', false);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, emptyFunction)', false);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, undefined)', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({audio:true, video:true}, emptyFunction, undefined)', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({audio:true}, emptyFunction, undefined)', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, "video")', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, null)', true, errorCallbackError );
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, {})', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, true)', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, 42)', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, Infinity)', true, errorCallbackError);
-test('navigator.webkitGetUserMedia({video: true}, emptyFunction, -Infinity)', true, errorCallbackError);
-
-window.jsTestIsAsync = false;
Modified: trunk/LayoutTests/fast/mediastream/script-tests/enabled.js (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/script-tests/enabled.js 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/script-tests/enabled.js 2017-02-09 01:30:56 UTC (rev 211912)
@@ -3,15 +3,22 @@
function hasGetUserMediaProperty()
{
for (var property in navigator) {
- if (property == "webkitGetUserMedia")
- return true;
+ if (property == "mediaDevices") {
+ var devices = navigator[property];
+ for (property in devices) {
+ if (property == "getUserMedia")
+ return true;
+ }
+ }
}
return false;
}
-shouldBeTrue("typeof navigator.webkitGetUserMedia == 'function'");
+shouldBeTrue("typeof navigator.mediaDevices == 'object'");
+shouldBeTrue("typeof navigator.mediaDevices.getUserMedia == 'function'");
shouldBeTrue("hasGetUserMediaProperty()");
-shouldBeTrue("'webkitGetUserMedia' in navigator");
-shouldBeFalse("navigator.hasOwnProperty('webkitGetUserMedia')");
+shouldBeTrue("'mediaDevices' in navigator");
+shouldBeTrue("'getUserMedia' in navigator.mediaDevices");
+shouldBeFalse("navigator.hasOwnProperty('mediaDevices')");
window.jsTestIsAsync = false;
Modified: trunk/LayoutTests/fast/mediastream/success.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/success.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/success.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -14,12 +14,14 @@
debug('This test can not be run without the testRunner');
var options = {audio: true, video: true};
-navigator.webkitGetUserMedia(options, function(stream) {
- finishJSTest();
-}, function(e) {
- testFailed('Error callback invoked unexpectedly');
- finishJSTest();
-});
+navigator.mediaDevices.getUserMedia(options)
+ .then(stream => {
+ finishJSTest();
+ })
+ .catch(err => {
+ testFailed('Error callback invoked unexpectedly');
+ finishJSTest();
+ });
</script>
<script src=""
</body>
Deleted: trunk/LayoutTests/fast/mediastream/webkitGetUserMedia-shadowing-then.html (211911 => 211912)
--- trunk/LayoutTests/fast/mediastream/webkitGetUserMedia-shadowing-then.html 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/LayoutTests/fast/mediastream/webkitGetUserMedia-shadowing-then.html 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,44 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-<head>
- <meta charset="UTF-8">
- <title>Shadowing then()</title>
-</head>
-<body>
-<!--
-This test verifies that the implementation of navigator.webkitGetUserMedia does not use public promise APIs.
--->
-<p id="output"></p>
-<script>
- if (window.testRunner) {
- testRunner.dumpAsText();
- testRunner.waitUntilDone();
- }
-
- function endTest(testStatus) {
- document.getElementById("output").textContent = testStatus;
- testRunner.notifyDone();
- }
-
- // Shadow Promise.prototype.then
- Promise.prototype.then = (function () {
- const realThen = Promise.prototype.then;
- return function () {
- endTest("FAIL: Executed shadowed, public .then()");
- }
- })();
-
- function gotUserMedia(mediaStream) {
- endTest("FAIL: Got user media");
- }
-
- function userMediaError(error) {
- endTest("PASS");
- }
-
- var options = { audio: false, video: false };
- navigator.webkitGetUserMedia(options, gotUserMedia, userMediaError);
-
-</script>
-</body>
-</html>
Modified: trunk/Source/WebCore/CMakeLists.txt (211911 => 211912)
--- trunk/Source/WebCore/CMakeLists.txt 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/CMakeLists.txt 2017-02-09 01:30:56 UTC (rev 211912)
@@ -222,7 +222,6 @@
Modules/mediastream/MediaTrackConstraints.idl
Modules/mediastream/MediaTrackSupportedConstraints.idl
Modules/mediastream/NavigatorMediaDevices.idl
- Modules/mediastream/NavigatorUserMedia.idl
Modules/mediastream/OverconstrainedError.idl
Modules/mediastream/OverconstrainedErrorEvent.idl
Modules/mediastream/RTCConfiguration.idl
@@ -3672,7 +3671,6 @@
${WEBCORE_DIR}/Modules/fetch/FetchRequest.js
${WEBCORE_DIR}/Modules/fetch/FetchResponse.js
${WEBCORE_DIR}/Modules/fetch/WorkerGlobalScopeFetch.js
- ${WEBCORE_DIR}/Modules/mediastream/NavigatorUserMedia.js
${WEBCORE_DIR}/Modules/mediastream/RTCPeerConnection.js
${WEBCORE_DIR}/Modules/mediastream/RTCPeerConnectionInternals.js
${WEBCORE_DIR}/Modules/streams/ByteLengthQueuingStrategy.js
Modified: trunk/Source/WebCore/ChangeLog (211911 => 211912)
--- trunk/Source/WebCore/ChangeLog 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/ChangeLog 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,3 +1,20 @@
+2017-02-08 Eric Carlson <eric.carl...@apple.com>
+
+ [MediaStream] Remove legacy Navigator.webkitGetUserMedia
+ https://bugs.webkit.org/show_bug.cgi?id=168016
+
+ Reviewed by Youenn Fablet.
+
+ No new tests, existing tests updated.
+
+ * WebCore.xcodeproj/project.pbxproj: NavigatorUserMedia.
+ * DerivedSources.make: Ditto.
+ * CMakeLists.txt: Ditto.
+ * Modules/mediastream/NavigatorUserMedia.idl: Removed.
+ * Modules/mediastream/NavigatorUserMedia.js: Removed.
+ * WebCore.xcodeproj/project.pbxproj:
+ * page/Navigator.idl: Add some whitespace to force derived sources to be rebuild correctly.
+
2017-02-08 Chris Dumez <cdu...@apple.com>
Use smaller tiles in windows that are not active to facilitate App Napping
Modified: trunk/Source/WebCore/DerivedSources.cpp (211911 => 211912)
--- trunk/Source/WebCore/DerivedSources.cpp 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/DerivedSources.cpp 2017-02-09 01:30:56 UTC (rev 211912)
@@ -350,7 +350,6 @@
#include "JSNavigatorContentUtils.cpp"
#include "JSNavigatorGeolocation.cpp"
#include "JSNavigatorMediaDevices.cpp"
-#include "JSNavigatorUserMedia.cpp"
#include "JSNavigatorVibration.cpp"
#include "JSNode.cpp"
#include "JSNodeFilter.cpp"
Modified: trunk/Source/WebCore/DerivedSources.make (211911 => 211912)
--- trunk/Source/WebCore/DerivedSources.make 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/DerivedSources.make 2017-02-09 01:30:56 UTC (rev 211912)
@@ -163,7 +163,6 @@
$(WebCore)/Modules/mediastream/MediaTrackConstraints.idl \
$(WebCore)/Modules/mediastream/MediaTrackSupportedConstraints.idl \
$(WebCore)/Modules/mediastream/NavigatorMediaDevices.idl \
- $(WebCore)/Modules/mediastream/NavigatorUserMedia.idl \
$(WebCore)/Modules/mediastream/OverconstrainedError.idl \
$(WebCore)/Modules/mediastream/OverconstrainedErrorEvent.idl \
$(WebCore)/Modules/mediastream/RTCConfiguration.idl \
@@ -1344,7 +1343,6 @@
$(WebCore)/Modules/fetch/FetchRequest.js \
$(WebCore)/Modules/fetch/FetchResponse.js \
${WebCore}/Modules/fetch/WorkerGlobalScopeFetch.js \
- $(WebCore)/Modules/mediastream/NavigatorUserMedia.js \
$(WebCore)/Modules/mediastream/RTCPeerConnection.js \
$(WebCore)/Modules/mediastream/RTCPeerConnectionInternals.js \
$(WebCore)/Modules/streams/ByteLengthQueuingStrategy.js \
Deleted: trunk/Source/WebCore/Modules/mediastream/NavigatorUserMedia.idl (211911 => 211912)
--- trunk/Source/WebCore/Modules/mediastream/NavigatorUserMedia.idl 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/Modules/mediastream/NavigatorUserMedia.idl 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,25 +0,0 @@
-/*
- * Copyright (C) 2012 Google Inc. All rights reserved.
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Library 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
- * Library General Public License for more details.
- *
- * You should have received a copy of the GNU Library General Public License
- * along with this library; see the file COPYING.LIB. If not, write to
- * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
- * Boston, MA 02110-1301, USA.
- */
-
-[
- Conditional=MEDIA_STREAM,
- EnabledAtRuntime=MediaStream,
-] partial interface Navigator {
- [JSBuiltin] void webkitGetUserMedia(MediaStreamConstraints constraints, NavigatorUserMediaSuccessCallback successCallback, NavigatorUserMediaErrorCallback errorCallback);
-};
Deleted: trunk/Source/WebCore/Modules/mediastream/NavigatorUserMedia.js (211911 => 211912)
--- trunk/Source/WebCore/Modules/mediastream/NavigatorUserMedia.js 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/Modules/mediastream/NavigatorUserMedia.js 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,48 +0,0 @@
-/*
- * Copyright (C) 2015 Canon Inc.
- *
- * 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. ``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
- * 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.
- */
-
-// @conditional=ENABLE(MEDIA_STREAM)
-
-function webkitGetUserMedia(options, successCallback, errorCallback)
-{
- "use strict";
-
- // FIXME: We should raise a DOM unsupported exception if there is no navigator and properly detect whether method is not called on a Navigator object.
- if (!(this.mediaDevices && this.mediaDevices.@getUserMedia))
- throw @makeThisTypeError("Navigator", "webkitGetUserMedia");
-
- if (arguments.length < 3)
- @throwTypeError("Not enough arguments");
-
- if (options !== @Object(options))
- @throwTypeError("Argument 1 (options) to Navigator.webkitGetUserMedia must be an object");
-
- if (typeof successCallback !== "function")
- @throwTypeError("Argument 2 ('successCallback') to Navigator.webkitGetUserMedia must be a function");
- if (typeof errorCallback !== "function")
- @throwTypeError("Argument 3 ('errorCallback') to Navigator.webkitGetUserMedia must be a function");
-
- this.mediaDevices.@getUserMedia(options).@then(successCallback, errorCallback);
-}
Modified: trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj (211911 => 211912)
--- trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2017-02-09 01:30:56 UTC (rev 211912)
@@ -146,8 +146,6 @@
073B87671E4385AC0071C0EC /* AudioSampleBufferList.h in Headers */ = {isa = PBXBuildFile; fileRef = 073B87631E43859D0071C0EC /* AudioSampleBufferList.h */; };
073B87681E4385AC0071C0EC /* AudioSampleDataSource.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 073B87641E43859D0071C0EC /* AudioSampleDataSource.cpp */; };
073B87691E4385AC0071C0EC /* AudioSampleDataSource.h in Headers */ = {isa = PBXBuildFile; fileRef = 073B87651E43859D0071C0EC /* AudioSampleDataSource.h */; };
- 073BE34017D17E01002BD431 /* JSNavigatorUserMedia.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 073BE33E17D17E01002BD431 /* JSNavigatorUserMedia.cpp */; };
- 073BE34117D17E01002BD431 /* JSNavigatorUserMedia.h in Headers */ = {isa = PBXBuildFile; fileRef = 073BE33F17D17E01002BD431 /* JSNavigatorUserMedia.h */; settings = {ATTRIBUTES = (Private, ); }; };
073BE34E17D180B2002BD431 /* RTCSessionDescriptionDescriptor.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 07221BAB17CF0AD400848E51 /* RTCSessionDescriptionDescriptor.cpp */; };
073BE34F17D18183002BD431 /* RTCIceCandidateDescriptor.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 07221BA617CF0AD400848E51 /* RTCIceCandidateDescriptor.cpp */; };
0744ECED1E0C4E30000D0944 /* MockRealtimeAudioSourceMac.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0744ECEC1E0C4AE5000D0944 /* MockRealtimeAudioSourceMac.mm */; };
@@ -7185,7 +7183,6 @@
07221B5717CEC32700848E51 /* MediaStreamTrackEvent.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = MediaStreamTrackEvent.cpp; sourceTree = "<group>"; };
07221B5817CEC32700848E51 /* MediaStreamTrackEvent.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MediaStreamTrackEvent.h; sourceTree = "<group>"; };
07221B5917CEC32700848E51 /* MediaStreamTrackEvent.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = MediaStreamTrackEvent.idl; sourceTree = "<group>"; };
- 07221B5C17CEC32700848E51 /* NavigatorUserMedia.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = NavigatorUserMedia.idl; sourceTree = "<group>"; };
07221B6317CEC32700848E51 /* RTCDataChannel.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = RTCDataChannel.cpp; sourceTree = "<group>"; };
07221B6417CEC32700848E51 /* RTCDataChannel.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RTCDataChannel.h; sourceTree = "<group>"; };
07221B6517CEC32700848E51 /* RTCDataChannel.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = RTCDataChannel.idl; sourceTree = "<group>"; };
@@ -7278,8 +7275,6 @@
073B87631E43859D0071C0EC /* AudioSampleBufferList.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AudioSampleBufferList.h; sourceTree = "<group>"; };
073B87641E43859D0071C0EC /* AudioSampleDataSource.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = AudioSampleDataSource.cpp; sourceTree = "<group>"; };
073B87651E43859D0071C0EC /* AudioSampleDataSource.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = AudioSampleDataSource.h; sourceTree = "<group>"; };
- 073BE33E17D17E01002BD431 /* JSNavigatorUserMedia.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = JSNavigatorUserMedia.cpp; sourceTree = "<group>"; };
- 073BE33F17D17E01002BD431 /* JSNavigatorUserMedia.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = JSNavigatorUserMedia.h; sourceTree = "<group>"; };
0744ECEB1E0C4AE5000D0944 /* MockRealtimeAudioSourceMac.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = MockRealtimeAudioSourceMac.h; sourceTree = "<group>"; };
0744ECEC1E0C4AE5000D0944 /* MockRealtimeAudioSourceMac.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = MockRealtimeAudioSourceMac.mm; sourceTree = "<group>"; };
074E82B818A69F0E007EF54C /* PlatformTimeRanges.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = PlatformTimeRanges.cpp; sourceTree = "<group>"; };
@@ -8862,7 +8857,6 @@
419BC2DC1685329900D64D6D /* VisitedLinkState.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = VisitedLinkState.cpp; sourceTree = "<group>"; };
419BC2DD1685329900D64D6D /* VisitedLinkState.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = VisitedLinkState.h; sourceTree = "<group>"; };
419BE7521BC7F3DB00E1C85B /* WebCoreBuiltinNames.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = WebCoreBuiltinNames.h; sourceTree = "<group>"; };
- 419BE75A1BCBD3ED00E1C85B /* NavigatorUserMedia.js */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode._javascript_; path = NavigatorUserMedia.js; sourceTree = "<group>"; };
419FAFAD1ABABCD5005B828B /* ReadableStreamDefaultReader.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = ReadableStreamDefaultReader.idl; sourceTree = "<group>"; };
41A023ED1A39DB7900F722CF /* ReadableStream.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = ReadableStream.idl; sourceTree = "<group>"; };
41A023ED1A39DB7900F722DF /* WritableStream.idl */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = text; path = WritableStream.idl; sourceTree = "<group>"; };
@@ -15349,8 +15343,6 @@
5EA725CA1ACABCB500EAD17B /* NavigatorMediaDevices.cpp */,
5EA725CB1ACABCB500EAD17B /* NavigatorMediaDevices.h */,
5EA725CC1ACABCB500EAD17B /* NavigatorMediaDevices.idl */,
- 07221B5C17CEC32700848E51 /* NavigatorUserMedia.idl */,
- 419BE75A1BCBD3ED00E1C85B /* NavigatorUserMedia.js */,
0704A4051D6DE9F10086DCDB /* OverconstrainedError.h */,
0704A4031D6DE9F10086DCDB /* OverconstrainedError.idl */,
072A703E1D6E8F6200DF0AFC /* OverconstrainedErrorEvent.h */,
@@ -15578,8 +15570,6 @@
932CC0D11DFFD667004C0F9F /* JSMediaTrackConstraints.h */,
0787C4671BFBDF6F006DCD7F /* JSMediaTrackSupportedConstraints.cpp */,
0787C4681BFBDF6F006DCD7F /* JSMediaTrackSupportedConstraints.h */,
- 073BE33E17D17E01002BD431 /* JSNavigatorUserMedia.cpp */,
- 073BE33F17D17E01002BD431 /* JSNavigatorUserMedia.h */,
0704A4091D6DFC690086DCDB /* JSOverconstrainedError.cpp */,
0704A40A1D6DFC690086DCDB /* JSOverconstrainedError.h */,
0704A4131D6F39FB0086DCDB /* JSOverconstrainedErrorEvent.cpp */,
@@ -26806,7 +26796,6 @@
C6F08FCA1431000D00685849 /* JSMutationRecord.h in Headers */,
BCD9C2C10C17B69E005C90A2 /* JSNamedNodeMap.h in Headers */,
A9D247F80D757E3400FDF959 /* JSNavigator.h in Headers */,
- 073BE34117D17E01002BD431 /* JSNavigatorUserMedia.h in Headers */,
BC9439C3116CF4940048C750 /* JSNodeCustom.h in Headers */,
14115B7309F84CD600CA4FC1 /* JSNodeFilter.h in Headers */,
1A750D8E0A90E521000FF215 /* JSNodeIterator.h in Headers */,
@@ -30648,7 +30637,6 @@
C6F08FC91431000D00685849 /* JSMutationRecord.cpp in Sources */,
BCD9C2C00C17B69E005C90A2 /* JSNamedNodeMap.cpp in Sources */,
A9D247F70D757E3400FDF959 /* JSNavigator.cpp in Sources */,
- 073BE34017D17E01002BD431 /* JSNavigatorUserMedia.cpp in Sources */,
14DC0D3709FED073007B0235 /* JSNode.cpp in Sources */,
BCD9C2640C17AA67005C90A2 /* JSNodeCustom.cpp in Sources */,
E3150EA61DA7219000194012 /* JSNodeDOMJIT.cpp in Sources */,
Modified: trunk/Source/WebCore/page/Navigator.idl (211911 => 211912)
--- trunk/Source/WebCore/page/Navigator.idl 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebCore/page/Navigator.idl 2017-02-09 01:30:56 UTC (rev 211912)
@@ -24,7 +24,7 @@
readonly attribute DOMMimeTypeArray mimeTypes;
readonly attribute boolean cookieEnabled;
boolean javaEnabled();
-
+
// FIXME: Convert this #if'def to an IDL conditional attribute.
#if defined(WTF_PLATFORM_IOS) && WTF_PLATFORM_IOS
readonly attribute boolean standalone;
Modified: trunk/Source/WebInspectorUI/ChangeLog (211911 => 211912)
--- trunk/Source/WebInspectorUI/ChangeLog 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebInspectorUI/ChangeLog 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1,3 +1,12 @@
+2017-02-08 Eric Carlson <eric.carl...@apple.com>
+
+ [MediaStream] Remove legacy Navigator.webkitGetUserMedia
+ https://bugs.webkit.org/show_bug.cgi?id=168016
+
+ Reviewed by Youenn Fablet.
+
+ * UserInterface/Models/NativeFunctionParameters.js:
+
2017-02-08 Chris Dumez <cdu...@apple.com>
Unreviewed, rolling out r211907.
Modified: trunk/Source/WebInspectorUI/UserInterface/Models/NativeFunctionParameters.js (211911 => 211912)
--- trunk/Source/WebInspectorUI/UserInterface/Models/NativeFunctionParameters.js 2017-02-09 01:24:36 UTC (rev 211911)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/NativeFunctionParameters.js 2017-02-09 01:30:56 UTC (rev 211912)
@@ -1161,21 +1161,6 @@
__proto__: null,
},
- Navigator: {
- webkitGetUserMedia: "options, successCallback, errorCallback",
- __proto__: null,
- },
-
- NavigatorUserMediaErrorCallback: {
- handleEvent: "error",
- __proto__: null,
- },
-
- NavigatorUserMediaSuccessCallback: {
- handleEvent: "stream",
- __proto__: null,
- },
-
NodeFilter: {
acceptNode: "[n]",
__proto__: null,