Attachment 372801 Details for Bug 198839 – Patch

[patch] Patch

bug-198839-20190624141441.patch (text/plain), 27.60 KB, created by youenn fablet on 2019-06-24 14:14:41 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2019-06-24 14:14:41 PDT

Size: 27.60 KB

patch

obsolete

>Subversion Revision: 246680
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 21aa1112e6df34cf799ca08762ec6bc682c7211f..7113559f3472ef9baf1b5805fe4e1f1fda71ef2a 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,50 @@
>+2019-06-24  Youenn Fablet  <youenn@apple.com>
>+
>+        [MacOS] getUserMedia called multiple times should better handle camera resolution
>+        https://bugs.webkit.org/show_bug.cgi?id=198839
>+        <rdar://problem/51718990>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Store the list of camera sources in a map of the camera manager.
>+        Whenever we create a new camera source, we look into this map for an existing source.
>+        If a source exists, we directly create a RealtimeVideoSource from it and we apply constraints on the RealtimeVideoSource.
>+        This is the same code path as clone/applyConstraints.
>+
>+        This patch does some refactoring to share that code in the camera manager and camera source factory.
>+
>+        Covered by existing tests.
>+
>+        * platform/mediastream/CaptureDeviceManager.h:
>+        * platform/mediastream/RealtimeMediaSourceFactory.cpp:
>+        (WebCore::VideoCaptureFactory::createVideoCaptureSource):
>+        * platform/mediastream/RealtimeMediaSourceFactory.h:
>+        * platform/mediastream/gstreamer/MockGStreamerVideoCaptureSource.cpp:
>+        (WebCore::MockRealtimeVideoSource::create):
>+        * platform/mediastream/mac/AVCaptureDeviceManager.h:
>+        * platform/mediastream/mac/AVVideoCaptureSource.h:
>+        * platform/mediastream/mac/AVVideoCaptureSource.mm:
>+        (WebCore::AVVideoCaptureSource::create):
>+        (WebCore::AVVideoCaptureSource::AVVideoCaptureSource):
>+        (WebCore::AVVideoCaptureSource::~AVVideoCaptureSource):
>+        * platform/mediastream/mac/MockRealtimeVideoSourceMac.mm:
>+        (WebCore::MockRealtimeVideoSource::create):
>+        * platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp:
>+        * platform/mock/MockRealtimeMediaSourceCenter.cpp:
>+        * platform/mock/MockRealtimeMediaSourceCenter.h:
>+        (WebCore::MockRealtimeMediaSourceCenter::videoCaptureDeviceManager):
>+        (WebCore::MockRealtimeMediaSourceCenter::audioCaptureDeviceManager):
>+        * platform/mock/MockRealtimeVideoSource.cpp:
>+        (WebCore::MockRealtimeVideoSource::create):
>+        (WebCore::MockRealtimeVideoSource::MockRealtimeVideoSource):
>+        (WebCore::MockRealtimeVideoSource::~MockRealtimeVideoSource):
>+        (WebCore::MockRealtimeVideoSource::deviceType const):
>+        * platform/mock/MockRealtimeVideoSource.h:
>+        * testing/Internals.cpp:
>+        (WebCore::Internals::resetToConsistentState):
>+        Now that we have sources kept in a map, we can reuse them from one test to another.
>+        We clear this state to ensure consistent init step of a test.
>+
> 2019-06-21  Youenn Fablet  <youenn@apple.com>
> 
>         ResourceLoadNotifier should check whether its frame document loader is null
>diff --git a/Source/WebCore/platform/mediastream/CaptureDeviceManager.h b/Source/WebCore/platform/mediastream/CaptureDeviceManager.h
>index 20a93973c1e3addada837ed06770803dfdbecd0e..e5c5a787a690cc139beed0677a9c205ce028f354 100644
>--- a/Source/WebCore/platform/mediastream/CaptureDeviceManager.h
>+++ b/Source/WebCore/platform/mediastream/CaptureDeviceManager.h
>@@ -28,7 +28,7 @@
> #if ENABLE(MEDIA_STREAM)
> 
> #include "CaptureDevice.h"
>-#include "RealtimeMediaSource.h"
>+#include "RealtimeVideoCaptureSource.h"
> #include <wtf/WeakPtr.h>
> 
> namespace WebCore {
>@@ -44,6 +44,34 @@ protected:
>     void deviceChanged();
> };
> 
>+class WEBCORE_EXPORT CaptureCameraManager : public CaptureDeviceManager {
>+public:
>+    RealtimeVideoCaptureSource* videoCaptureSource(const String& id)
>+    {
>+        return m_deviceSources.get(id);
>+    }
>+    void addVideoCaptureSource(RealtimeVideoCaptureSource& source)
>+    {
>+#if !ASSERT_DISABLED
>+        auto* existing = m_deviceSources.get(source.persistentID());
>+        ASSERT(!existing || existing->muted() || existing->interrupted());
>+#endif
>+        m_deviceSources.set(source.persistentID(), &source);
>+    }
>+    void removeVideoCaptureSource(RealtimeVideoCaptureSource& source)
>+    {
>+        ASSERT(m_deviceSources.contains(source.persistentID()) || m_deviceSources.isEmpty());
>+        m_deviceSources.remove(source.persistentID());
>+    }
>+    void clearSources()
>+    {
>+        m_deviceSources.clear();
>+    }
>+
>+private:
>+    HashMap<String, RealtimeVideoCaptureSource*> m_deviceSources;
>+};
>+
> } // namespace WebCore
> 
> #endif // ENABLE(MEDIA_STREAM)
>diff --git a/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.cpp b/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.cpp
>index e8e55df27f4a3f6858f13b586a48e9655afe61fd..cee9234449e23ba7cc939ee8fd761dcab27432a9 100644
>--- a/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.cpp
>+++ b/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.cpp
>@@ -32,7 +32,7 @@
> #include "CaptureDeviceManager.h"
> #include "Logging.h"
> #include "MediaStreamPrivate.h"
>-#include "RealtimeMediaSource.h"
>+#include "RealtimeVideoSource.h"
> #include <wtf/SHA1.h>
> 
> namespace WebCore {
>@@ -52,6 +52,34 @@ void SingleSourceFactory::unsetActiveSource(RealtimeMediaSource& source)
>         m_activeSource = nullptr;
> }
> 
>+CaptureSourceOrError VideoCaptureFactory::createVideoCaptureSource(const CaptureDevice& device, String&& hashSalt, const MediaConstraints* constraints)
>+{
>+    ASSERT(device.type() == CaptureDevice::DeviceType::Camera);
>+
>+    auto* deviceSource = existingSource(device.persistentId());
>+    if (deviceSource && !deviceSource->muted() && !deviceSource->interrupted() && hashSalt == deviceSource->deviceIDHashSalt()) {
>+        auto source = RealtimeVideoSource::create(*deviceSource);
>+        if (constraints) {
>+            auto result = source->applyConstraints(*constraints);
>+            if (result)
>+                return WTFMove(result.value().badConstraint);
>+        }
>+        return CaptureSourceOrError { WTFMove(source) };
>+    }
>+
>+    auto newDeviceSource = newSource(device,  WTFMove(hashSalt));
>+    if (!newDeviceSource)
>+        return { };
>+
>+    if (constraints) {
>+        auto result = newDeviceSource->applyConstraints(*constraints);
>+        if (result)
>+            return WTFMove(result.value().badConstraint);
>+    }
>+
>+    return CaptureSourceOrError { RealtimeVideoSource::create(newDeviceSource.releaseNonNull()) };
>+}
>+
> } // namespace WebCore
> 
> #endif // ENABLE(MEDIA_STREAM)
>diff --git a/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.h b/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.h
>index a3d52f0fcb773634cbf453ff94acea5a1a2fc562..b3b9d83bb51225beb2d88a2f1c2ea925c85420dd 100644
>--- a/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.h
>+++ b/Source/WebCore/platform/mediastream/RealtimeMediaSourceFactory.h
>@@ -34,6 +34,7 @@ namespace WebCore {
> class CaptureDevice;
> class CaptureDeviceManager;
> class RealtimeMediaSource;
>+class RealtimeVideoCaptureSource;
> 
> struct CaptureSourceOrError;
> struct MediaConstraints;
>@@ -74,7 +75,8 @@ class VideoCaptureFactory
> {
> public:
>     virtual ~VideoCaptureFactory() = default;
>-    virtual CaptureSourceOrError createVideoCaptureSource(const CaptureDevice&, String&&, const MediaConstraints*) = 0;
>+    virtual CaptureSourceOrError createVideoCaptureSource(const CaptureDevice&, String&&, const MediaConstraints*);
>+
>     virtual CaptureDeviceManager& videoCaptureDeviceManager() = 0;
>     virtual void setVideoCapturePageState(bool interrupted, bool pageMuted)
>     {
>@@ -84,6 +86,10 @@ public:
> 
> protected:
>     VideoCaptureFactory() = default;
>+
>+private:
>+    virtual RealtimeVideoCaptureSource* existingSource(const String&) = 0;
>+    virtual RefPtr<RealtimeVideoCaptureSource> newSource(const CaptureDevice&, String&&) = 0;
> };
> 
> class DisplayCaptureFactory {
>diff --git a/Source/WebCore/platform/mediastream/gstreamer/MockGStreamerVideoCaptureSource.cpp b/Source/WebCore/platform/mediastream/gstreamer/MockGStreamerVideoCaptureSource.cpp
>index 2c72f5b87867fa1ddddb7175370b273ecb4ab446..77f12c43e5a816a74ec77b6f491cd5d72def73f5 100644
>--- a/Source/WebCore/platform/mediastream/gstreamer/MockGStreamerVideoCaptureSource.cpp
>+++ b/Source/WebCore/platform/mediastream/gstreamer/MockGStreamerVideoCaptureSource.cpp
>@@ -61,15 +61,9 @@ public:
>     }
> };
> 
>-CaptureSourceOrError MockRealtimeVideoSource::create(String&& deviceID,
>-    String&& name, String&& hashSalt, const MediaConstraints* constraints)
>+RefPtr<MockRealtimeVideoSource> MockRealtimeVideoSource::create(String&& deviceID, String&& name, String&& hashSalt)
> {
>-    auto source = adoptRef(*new MockGStreamerVideoCaptureSource(WTFMove(deviceID), WTFMove(name), WTFMove(hashSalt)));
>-
>-    if (constraints && source->applyConstraints(*constraints))
>-        return { };
>-
>-    return CaptureSourceOrError(WTFMove(source));
>+    return adoptRef(*new MockGStreamerVideoCaptureSource(WTFMove(deviceID), WTFMove(name), WTFMove(hashSalt)));
> }
> 
> void MockGStreamerVideoCaptureSource::startProducingData()
>diff --git a/Source/WebCore/platform/mediastream/mac/AVCaptureDeviceManager.h b/Source/WebCore/platform/mediastream/mac/AVCaptureDeviceManager.h
>index 5ff4a3a3103ab2074c49f3ab7a88aee730568afd..62318e230f5d18f4fa1b8b763db2640ccf9fe195 100644
>--- a/Source/WebCore/platform/mediastream/mac/AVCaptureDeviceManager.h
>+++ b/Source/WebCore/platform/mediastream/mac/AVCaptureDeviceManager.h
>@@ -43,8 +43,9 @@ OBJC_CLASS NSString;
> OBJC_CLASS WebCoreAVCaptureDeviceManagerObserver;
> 
> namespace WebCore {
>+class AVVideoCaptureSource;
> 
>-class AVCaptureDeviceManager final : public CaptureDeviceManager {
>+class AVCaptureDeviceManager final : public CaptureCameraManager {
>     friend class NeverDestroyed<AVCaptureDeviceManager>;
> public:
>     const Vector<CaptureDevice>& captureDevices() final;
>diff --git a/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.h b/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.h
>index 2a85ad44609a9d35464cde81088895ee2a7cdb73..87fb5339cd2590cffbf603f9e83d78e03dd9f2c1 100644
>--- a/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.h
>+++ b/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.h
>@@ -53,9 +53,7 @@ class ImageTransferSessionVT;
> 
> class AVVideoCaptureSource : public RealtimeVideoCaptureSource, private OrientationNotifier::Observer {
> public:
>-    static CaptureSourceOrError create(String&& id, String&& hashSalt, const MediaConstraints*);
>-
>-    WEBCORE_EXPORT static VideoCaptureFactory& factory();
>+    static RefPtr<AVVideoCaptureSource> create(String&& id, String&& hashSalt);
> 
>     enum class InterruptionReason { None, VideoNotAllowedInBackground, AudioInUse, VideoInUse, VideoNotAllowedInSideBySide };
>     void captureSessionBeginInterruption(RetainPtr<NSNotification>);
>diff --git a/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm b/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm
>index 5c612edb323bf2e926377ed127d68781209970bf..616a557b0bb8bd5843a6941d63dac910fe60267e 100644
>--- a/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm
>+++ b/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm
>@@ -28,6 +28,7 @@
> 
> #if ENABLE(MEDIA_STREAM) && USE(AVFOUNDATION)
> 
>+#import "AVCaptureDeviceManager.h"
> #import "ImageBuffer.h"
> #import "ImageTransferSessionVT.h"
> #import "IntRect.h"
>@@ -109,20 +110,13 @@ public:
>     RetainPtr<AVCaptureDeviceFormat> format;
> };
> 
>-CaptureSourceOrError AVVideoCaptureSource::create(String&& id, String&& hashSalt, const MediaConstraints* constraints)
>+RefPtr<AVVideoCaptureSource> AVVideoCaptureSource::create(String&& id, String&& hashSalt)
> {
>     AVCaptureDevice *device = [PAL::getAVCaptureDeviceClass() deviceWithUniqueID:id];
>     if (!device)
>         return { };
> 
>-    auto source = adoptRef(*new AVVideoCaptureSource(device, WTFMove(id), WTFMove(hashSalt)));
>-    if (constraints) {
>-        auto result = source->applyConstraints(*constraints);
>-        if (result)
>-            return WTFMove(result.value().badConstraint);
>-    }
>-
>-    return CaptureSourceOrError(RealtimeVideoSource::create(WTFMove(source)));
>+    return adoptRef(*new AVVideoCaptureSource(device, WTFMove(id), WTFMove(hashSalt)));
> }
> 
> AVVideoCaptureSource::AVVideoCaptureSource(AVCaptureDevice* device, String&& id, String&& hashSalt)
>@@ -130,6 +124,9 @@ AVVideoCaptureSource::AVVideoCaptureSource(AVCaptureDevice* device, String&& id,
>     , m_objcObserver(adoptNS([[WebCoreAVVideoCaptureSourceObserver alloc] initWithCallback:this]))
>     , m_device(device)
> {
>+#if PLATFORM(MAC)
>+    AVCaptureDeviceManager::singleton().addVideoCaptureSource(*this);
>+#endif
> #if PLATFORM(IOS_FAMILY)
>     static_assert(static_cast<int>(InterruptionReason::VideoNotAllowedInBackground) == static_cast<int>(AVCaptureSessionInterruptionReasonVideoDeviceNotAvailableInBackground), "InterruptionReason::VideoNotAllowedInBackground is not AVCaptureSessionInterruptionReasonVideoDeviceNotAvailableInBackground as expected");
>     static_assert(static_cast<int>(InterruptionReason::VideoNotAllowedInSideBySide) == AVCaptureSessionInterruptionReasonVideoDeviceNotAvailableWithMultipleForegroundApps, "InterruptionReason::VideoNotAllowedInSideBySide is not AVCaptureSessionInterruptionReasonVideoDeviceNotAvailableWithMultipleForegroundApps as expected");
>@@ -142,6 +139,9 @@ AVVideoCaptureSource::AVVideoCaptureSource(AVCaptureDevice* device, String&& id,
> 
> AVVideoCaptureSource::~AVVideoCaptureSource()
> {
>+#if PLATFORM(MAC)
>+    AVCaptureDeviceManager::singleton().removeVideoCaptureSource(*this);
>+#endif
> #if PLATFORM(IOS_FAMILY)
>     RealtimeMediaSourceCenter::singleton().videoCaptureFactory().unsetActiveSource(*this);
> #endif
>diff --git a/Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm b/Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm
>index 83c63b177b6e80f0448ac1f1ae1bce347f3783f0..98ebe69404de2cf4080bde9147a9728a1d8978fc 100644
>--- a/Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm
>+++ b/Source/WebCore/platform/mediastream/mac/MockRealtimeVideoSourceMac.mm
>@@ -53,7 +53,7 @@
> namespace WebCore {
> using namespace PAL;
> 
>-CaptureSourceOrError MockRealtimeVideoSource::create(String&& deviceID, String&& name, String&& hashSalt, const MediaConstraints* constraints)
>+RefPtr<MockRealtimeVideoSource> MockRealtimeVideoSource::create(String&& deviceID, String&& name, String&& hashSalt)
> {
> #ifndef NDEBUG
>     auto device = MockRealtimeMediaSourceCenter::mockDeviceWithPersistentID(deviceID);
>@@ -61,13 +61,7 @@ CaptureSourceOrError MockRealtimeVideoSource::create(String&& deviceID, String&&
>     if (!device)
>         return { };
> #endif
>-
>-    auto source = adoptRef(*new MockRealtimeVideoSourceMac(WTFMove(deviceID), WTFMove(name), WTFMove(hashSalt)));
>-    // FIXME: We should report error messages
>-    if (constraints && source->applyConstraints(*constraints))
>-        return { };
>-
>-    return CaptureSourceOrError(RealtimeVideoSource::create(WTFMove(source)));
>+    return adoptRef(*new MockRealtimeVideoSourceMac(WTFMove(deviceID), WTFMove(name), WTFMove(hashSalt)));
> }
> 
> MockRealtimeVideoSourceMac::MockRealtimeVideoSourceMac(String&& deviceID, String&& name, String&& hashSalt)
>diff --git a/Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp b/Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp
>index a328cd3a83ecbfca049e54abf70de9d78faf331b..fe6baaf7f78176aad4ea127b9a36aab8a37336fe 100644
>--- a/Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp
>+++ b/Source/WebCore/platform/mediastream/mac/RealtimeMediaSourceCenterMac.cpp
>@@ -38,6 +38,7 @@
> #include "DisplayCaptureManagerCocoa.h"
> #include "Logging.h"
> #include "MediaStreamPrivate.h"
>+#include "RealtimeVideoSource.h"
> #include "RuntimeEnabledFeatures.h"
> #include "ScreenDisplayCaptureSourceMac.h"
> #include "WindowDisplayCaptureSourceMac.h"
>@@ -46,13 +47,6 @@
> namespace WebCore {
> 
> class VideoCaptureSourceFactoryMac final : public VideoCaptureFactory {
>-public:
>-    CaptureSourceOrError createVideoCaptureSource(const CaptureDevice& device, String&& hashSalt, const MediaConstraints* constraints) final
>-    {
>-        ASSERT(device.type() == CaptureDevice::DeviceType::Camera);
>-        return AVVideoCaptureSource::create(String { device.persistentId() }, WTFMove(hashSalt), constraints);
>-    }
>-
> private:
> #if PLATFORM(IOS_FAMILY)
>     void setVideoCapturePageState(bool interrupted, bool pageMuted)
>@@ -62,7 +56,21 @@ private:
>     }
> #endif
> 
>-    CaptureDeviceManager& videoCaptureDeviceManager() { return AVCaptureDeviceManager::singleton(); }
>+    RealtimeVideoCaptureSource* existingSource(const String& persistentID) final
>+    {
>+#if PLATFORM(IOS_FAMILY)
>+        return (activeSource() && activeSource()->persistentID() == persistentID) ? static_cast<RealtimeVideoCaptureSource*>(activeSource()) : nullptr;
>+#else
>+        return AVCaptureDeviceManager::singleton().videoCaptureSource(persistentID);
>+#endif
>+    }
>+
>+    RefPtr<RealtimeVideoCaptureSource> newSource(const CaptureDevice& device, String&& hashSalt) final
>+    {
>+        return AVVideoCaptureSource::create(String { device.persistentId() }, WTFMove(hashSalt));
>+    }
>+
>+    CaptureDeviceManager& videoCaptureDeviceManager() final { return AVCaptureDeviceManager::singleton(); }
> };
> 
> class DisplayCaptureSourceFactoryMac final : public DisplayCaptureFactory {
>diff --git a/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.cpp b/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.cpp
>index 6d83b5095b5054c43cf9e0fd246e1026d6eae54b..3637459a94db2a3e2a042a7a0b7858be9924a78c 100644
>--- a/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.cpp
>+++ b/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.cpp
>@@ -85,15 +85,6 @@ static inline Vector<MockMediaDevice> defaultDevices()
> }
> 
> class MockRealtimeVideoSourceFactory : public VideoCaptureFactory {
>-public:
>-    CaptureSourceOrError createVideoCaptureSource(const CaptureDevice& device, String&& hashSalt, const MediaConstraints* constraints) final
>-    {
>-        ASSERT(device.type() == CaptureDevice::DeviceType::Camera);
>-        ASSERT(MockRealtimeMediaSourceCenter::captureDeviceWithPersistentID(CaptureDevice::DeviceType::Camera, device.persistentId()));
>-
>-        return MockRealtimeVideoSource::create(String { device.persistentId() }, String { device.label() }, WTFMove(hashSalt), constraints);
>-    }
>-
> private:
> #if PLATFORM(IOS_FAMILY)
>     void setVideoCapturePageState(bool interrupted, bool pageMuted) final
>@@ -102,6 +93,21 @@ private:
>             activeSource()->setInterrupted(interrupted, pageMuted);
>     }
> #endif
>+
>+    RealtimeVideoCaptureSource* existingSource(const String& persistentID) final
>+    {
>+#if PLATFORM(IOS_FAMILY)
>+        return (activeSource() && activeSource()->persistentID() == persistentID) ? static_cast<RealtimeVideoCaptureSource*>(activeSource()) : nullptr;
>+#else
>+        return MockRealtimeMediaSourceCenter::singleton().videoCaptureDeviceManager().videoCaptureSource(persistentID);
>+#endif
>+    }
>+
>+    RefPtr<RealtimeVideoCaptureSource> newSource(const CaptureDevice& device, String&& hashSalt) final
>+    {
>+        return MockRealtimeVideoSource::create(String { device.persistentId() }, String { device.label() }, WTFMove(hashSalt));
>+    }
>+
>     CaptureDeviceManager& videoCaptureDeviceManager() final { return MockRealtimeMediaSourceCenter::singleton().videoCaptureDeviceManager(); }
> };
> 
>@@ -113,8 +119,15 @@ public:
> 
>         switch (device.type()) {
>         case CaptureDevice::DeviceType::Screen:
>-        case CaptureDevice::DeviceType::Window:
>-            return MockRealtimeVideoSource::create(String { device.persistentId() }, String { device.label() }, String { }, constraints);
>+        case CaptureDevice::DeviceType::Window: {
>+            auto source = MockRealtimeVideoSource::create(String { device.persistentId() }, String { device.label() }, String { });
>+            if (constraints) {
>+                auto result = source->applyConstraints(*constraints);
>+                if (result)
>+                    return WTFMove(result.value().badConstraint);
>+            }
>+            return CaptureSourceOrError { RealtimeVideoSource::create(source.releaseNonNull()) };
>+            }
>             break;
>         case CaptureDevice::DeviceType::Microphone:
>         case CaptureDevice::DeviceType::Camera:
>diff --git a/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.h b/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.h
>index 6d16fe77a6cebe3b551381b6b9aabb0eec934dd3..2cf7bb95f6ee8b2dfba1693f4d4fc16d8d094e10 100644
>--- a/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.h
>+++ b/Source/WebCore/platform/mock/MockRealtimeMediaSourceCenter.h
>@@ -59,8 +59,35 @@ public:
>     static Optional<MockMediaDevice> mockDeviceWithPersistentID(const String&);
>     static Optional<CaptureDevice> captureDeviceWithPersistentID(CaptureDevice::DeviceType, const String&);
> 
>+    class MockVideoCaptureDeviceManager final : public CaptureDeviceManager {
>+    public:
>+        MockRealtimeVideoSource* videoCaptureSource(const String& id)
>+        {
>+            return m_deviceSources.get(id);
>+        }
>+        void addVideoCaptureSource(MockRealtimeVideoSource& source)
>+        {
>+            ASSERT(!m_deviceSources.contains(source.persistentID()));
>+            m_deviceSources.add(source.persistentID(), &source);
>+        }
>+        void removeVideoCaptureSource(MockRealtimeVideoSource& source)
>+        {
>+            m_deviceSources.remove(source.persistentID());
>+        }
>+        void clearSources()
>+        {
>+            m_deviceSources.clear();
>+        }
>+
>+    private:
>+        const Vector<CaptureDevice>& captureDevices() final { return MockRealtimeMediaSourceCenter::videoDevices(); }
>+        Optional<CaptureDevice> captureDeviceWithPersistentID(CaptureDevice::DeviceType type, const String& id) final { return MockRealtimeMediaSourceCenter::captureDeviceWithPersistentID(type, id); }
>+
>+        HashMap<String, MockRealtimeVideoSource*> m_deviceSources;
>+    };
>+    MockVideoCaptureDeviceManager& videoCaptureDeviceManager() { return m_videoCaptureDeviceManager; }
>+
>     CaptureDeviceManager& audioCaptureDeviceManager() { return m_audioCaptureDeviceManager; }
>-    CaptureDeviceManager& videoCaptureDeviceManager() { return m_videoCaptureDeviceManager; }
>     CaptureDeviceManager& displayCaptureDeviceManager() { return m_displayCaptureDeviceManager; }
> 
> private:
>@@ -76,11 +103,6 @@ private:
>         const Vector<CaptureDevice>& captureDevices() final { return MockRealtimeMediaSourceCenter::audioDevices(); }
>         Optional<CaptureDevice> captureDeviceWithPersistentID(CaptureDevice::DeviceType type, const String& id) final { return MockRealtimeMediaSourceCenter::captureDeviceWithPersistentID(type, id); }
>     };
>-    class MockVideoCaptureDeviceManager final : public CaptureDeviceManager {
>-    private:
>-        const Vector<CaptureDevice>& captureDevices() final { return MockRealtimeMediaSourceCenter::videoDevices(); }
>-        Optional<CaptureDevice> captureDeviceWithPersistentID(CaptureDevice::DeviceType type, const String& id) final { return MockRealtimeMediaSourceCenter::captureDeviceWithPersistentID(type, id); }
>-    };
>     class MockDisplayCaptureDeviceManager final : public CaptureDeviceManager {
>     private:
>         const Vector<CaptureDevice>& captureDevices() final { return MockRealtimeMediaSourceCenter::displayDevices(); }
>diff --git a/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp b/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp
>index 49a48be0faa299fa4fa693d742348b3044c5e38f..c7168f5743010db58d4bb49de64b6082b37c52b4 100644
>--- a/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp
>+++ b/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp
>@@ -50,7 +50,7 @@
> namespace WebCore {
> 
> #if !PLATFORM(MAC) && !PLATFORM(IOS_FAMILY) && !(USE(GSTREAMER) && USE(LIBWEBRTC))
>-CaptureSourceOrError MockRealtimeVideoSource::create(String&& deviceID, String&& name, String&& hashSalt, const MediaConstraints* constraints)
>+RefPtr<MockRealtimeVideoSource> MockRealtimeVideoSource::create(String&& deviceID, String&& name, String&& hashSalt)
> {
> #ifndef NDEBUG
>     auto device = MockRealtimeMediaSourceCenter::mockDeviceWithPersistentID(deviceID);
>@@ -58,12 +58,7 @@ CaptureSourceOrError MockRealtimeVideoSource::create(String&& deviceID, String&&
>     if (!device)
>         return { };
> #endif
>-
>-    auto source = adoptRef(*new MockRealtimeVideoSource(WTFMove(deviceID), WTFMove(name), WTFMove(hashSalt)));
>-    if (constraints && source->applyConstraints(*constraints))
>-        return { };
>-
>-    return CaptureSourceOrError(RealtimeVideoSource::create(WTFMove(source)));
>+    return adoptRef(*new MockRealtimeVideoSource(WTFMove(deviceID), WTFMove(name), WTFMove(hashSalt)));
> }
> #endif
> 
>@@ -91,6 +86,28 @@ MockRealtimeVideoSource::MockRealtimeVideoSource(String&& deviceID, String&& nam
>     setFrameRate(properties.defaultFrameRate);
>     setFacingMode(properties.facingMode);
>     m_fillColor = properties.fillColor;
>+
>+    MockRealtimeMediaSourceCenter::singleton().videoCaptureDeviceManager().addVideoCaptureSource(*this);
>+}
>+
>+MockRealtimeVideoSource::~MockRealtimeVideoSource()
>+{
>+    if (mockDisplay())
>+        return;
>+    MockRealtimeMediaSourceCenter::singleton().videoCaptureDeviceManager().removeVideoCaptureSource(*this);
>+}
>+
>+CaptureDevice::DeviceType MockRealtimeVideoSource::deviceType() const
>+{
>+    if (mockCamera())
>+        return CaptureDevice::DeviceType::Camera;
>+    if (mockScreen())
>+        return CaptureDevice::DeviceType::Screen;
>+    if (mockWindow())
>+        return CaptureDevice::DeviceType::Window;
>+
>+    ASSERT_NOT_REACHED();
>+    return CaptureDevice::DeviceType::Unknown;
> }
> 
> bool MockRealtimeVideoSource::supportsSizeAndFrameRate(Optional<int> width, Optional<int> height, Optional<double> rate)
>diff --git a/Source/WebCore/platform/mock/MockRealtimeVideoSource.h b/Source/WebCore/platform/mock/MockRealtimeVideoSource.h
>index cbe1a004ab7f2f415a85c075cc5a5fb5aa489097..0c4f21e77884c16f6ac614ea6eb60ed67fe372e2 100644
>--- a/Source/WebCore/platform/mock/MockRealtimeVideoSource.h
>+++ b/Source/WebCore/platform/mock/MockRealtimeVideoSource.h
>@@ -47,11 +47,11 @@ class GraphicsContext;
> 
> class MockRealtimeVideoSource : public RealtimeVideoCaptureSource {
> public:
>-
>-    static CaptureSourceOrError create(String&& deviceID, String&& name, String&& hashSalt, const MediaConstraints*);
>+    static RefPtr<MockRealtimeVideoSource> create(String&& deviceID, String&& name, String&& hashSalt);
> 
> protected:
>     MockRealtimeVideoSource(String&& deviceID, String&& name, String&& hashSalt);
>+    ~MockRealtimeVideoSource();
> 
>     virtual void updateSampleBuffer() = 0;
> 
>@@ -68,7 +68,7 @@ private:
>     void startProducingData() final;
>     void stopProducingData() final;
>     bool isCaptureSource() const final { return true; }
>-    CaptureDevice::DeviceType deviceType() const final { return CaptureDevice::DeviceType::Camera; }
>+    CaptureDevice::DeviceType deviceType() const final;
>     bool supportsSizeAndFrameRate(Optional<int> width, Optional<int> height, Optional<double>) final;
>     void setSizeAndFrameRate(Optional<int> width, Optional<int> height, Optional<double>) final;
>     void setFrameRateWithPreset(double, RefPtr<VideoPreset>) final;
>diff --git a/Source/WebCore/testing/Internals.cpp b/Source/WebCore/testing/Internals.cpp
>index d25745aef1a6173cba74ecece7adbb75531df00c..a07a3f1a7b7e4ab990e432f37cc42a70da59b780 100644
>--- a/Source/WebCore/testing/Internals.cpp
>+++ b/Source/WebCore/testing/Internals.cpp
>@@ -525,6 +525,7 @@ void Internals::resetToConsistentState(Page& page)
>     rtcProvider.disableNonLocalhostConnections();
>     RuntimeEnabledFeatures::sharedFeatures().setWebRTCVP8CodecEnabled(true);
> #endif
>+    MockRealtimeMediaSourceCenter::singleton().videoCaptureDeviceManager().clearSources();
> 
>     page.settings().setStorageAccessAPIEnabled(false);
>     page.setFullscreenAutoHideDuration(0_s);

Actions: View | Formatted Diff | Diff

Attachments on bug 198839: 372801 | 372808 | 372809 | 372813 | 373058 | 373085 | 375414 | 375416