Remove HasMediaCaptureDevice

https://bugs.webkit.org/show_bug.cgi?id=164767
<rdar://problem/29263696>

Reviewed by Eric Carlson.

Source/WebCore:

* Modules/mediastream/MediaStream.cpp:
(WebCore::MediaStream::mediaState): Remove HasMediaCaptureDevice in state.
* page/MediaProducer.h:
* testing/Internals.cpp:
(WebCore::Internals::pageMediaState): Remove it in the media string.

Source/WebKit2:

* UIProcess/API/C/WKPage.cpp: Remove kWKMediaHasCaptureDevice.
(WKPageGetMediaState):
* UIProcess/API/C/WKPagePrivate.h:

LayoutTests:

* fast/mediastream/MediaStream-page-muted-expected.txt: Remove check for HasMediaCaptureDevice.
* fast/mediastream/MediaStream-page-muted.html:


Canonical link: https://commits.webkit.org/182449@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@208735 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog

@@ -1,3 +1,14 @@
+2016-11-15  Jon Lee  <jonlee@apple.com>
+
+        Remove HasMediaCaptureDevice
+        https://bugs.webkit.org/show_bug.cgi?id=164767
+        <rdar://problem/29263696>
+
+        Reviewed by Eric Carlson.
+
+        * fast/mediastream/MediaStream-page-muted-expected.txt: Remove check for HasMediaCaptureDevice.
+        * fast/mediastream/MediaStream-page-muted.html:
+
 2016-11-14  Dave Hyatt  <hyatt@apple.com>
 
         [CSS Parser] Fix font-synthesis and text-decoration-skip parsing

LayoutTests/fast/mediastream/MediaStream-page-muted-expected.txt

@@ -12,20 +12,16 @@ PASS window.internals.pageMediaState().includes('HasActiveMediaCaptureDevice') b
 *** Muting capture devices
 EVENT: mute
 PASS muteChangedEvent.target.muted is true
-PASS window.internals.pageMediaState().includes('HasMediaCaptureDevice') is true
 EVENT: mute
 PASS muteChangedEvent.target.muted is true
-PASS window.internals.pageMediaState().includes('HasMediaCaptureDevice') is true
 
 PASS window.internals.pageMediaState().includes('HasActiveMediaCaptureDevice') became false
 
 *** Unmuting capture devices
 EVENT: unmute
 PASS muteChangedEvent.target.muted is false
-PASS window.internals.pageMediaState().includes('HasMediaCaptureDevice') is true
 EVENT: unmute
 PASS muteChangedEvent.target.muted is false
-PASS window.internals.pageMediaState().includes('HasMediaCaptureDevice') is true
 
 PASS window.internals.pageMediaState().includes('HasActiveMediaCaptureDevice') became true
 PASS successfullyParsed is true

LayoutTests/fast/mediastream/MediaStream-page-muted.html

@@ -24,7 +24,6 @@
                 muteChangedEvent = ev;
                 debug(`EVENT: ${muteChangedEvent.type}`);
                 shouldBe('muteChangedEvent.target.muted', (muteChangedEvent.type == "mute").toString());
-                shouldBe("window.internals.pageMediaState().includes('HasMediaCaptureDevice')", "true");
 
                 if (++eventCount == 2) {
                     debug("");

Source/WebCore/ChangeLog

@@ -1,3 +1,17 @@
+2016-11-15  Jon Lee  <jonlee@apple.com>
+
+        Remove HasMediaCaptureDevice
+        https://bugs.webkit.org/show_bug.cgi?id=164767
+        <rdar://problem/29263696>
+
+        Reviewed by Eric Carlson.
+
+        * Modules/mediastream/MediaStream.cpp:
+        (WebCore::MediaStream::mediaState): Remove HasMediaCaptureDevice in state.
+        * page/MediaProducer.h:
+        * testing/Internals.cpp:
+        (WebCore::Internals::pageMediaState): Remove it in the media string.
+
 2016-11-14  Dave Hyatt  <hyatt@apple.com>
 
         [CSS Parser] Fix font-synthesis and text-decoration-skip parsing

Source/WebCore/Modules/mediastream/MediaStream.cpp

@@ -312,7 +312,6 @@ MediaProducer::MediaStateFlags MediaStream::mediaState() const
     if (!m_isActive)
         return state;
 
-    state |= HasMediaCaptureDevice;
     if (m_private->isProducingData())
         state |= HasActiveMediaCaptureDevice;
 

Source/WebCore/page/MediaProducer.h

@@ -43,7 +43,6 @@ class MediaProducer {
         HasPlaybackTargetAvailabilityListener = 1 << 9,
         HasAudioOrVideo = 1 << 10,
         HasActiveMediaCaptureDevice = 1 << 11,
-        HasMediaCaptureDevice = 1 << 12,
     };
     typedef unsigned MediaStateFlags;
 

Source/WebCore/testing/Internals.cpp

@@ -3012,8 +3012,6 @@ String Internals::pageMediaState()
         string.append("HasAudioOrVideo,");
     if (state & MediaProducer::HasActiveMediaCaptureDevice)
         string.append("HasActiveMediaCaptureDevice,");
-    if (state & MediaProducer::HasMediaCaptureDevice)
-        string.append("HasMediaCaptureDevice,");
 
     if (string.isEmpty())
         string.append("IsNotPlaying");

Source/WebKit2/ChangeLog

@@ -1,3 +1,15 @@
+2016-11-15  Jon Lee  <jonlee@apple.com>
+
+        Remove HasMediaCaptureDevice
+        https://bugs.webkit.org/show_bug.cgi?id=164767
+        <rdar://problem/29263696>
+
+        Reviewed by Eric Carlson.
+
+        * UIProcess/API/C/WKPage.cpp: Remove kWKMediaHasCaptureDevice.
+        (WKPageGetMediaState):
+        * UIProcess/API/C/WKPagePrivate.h:
+
 2016-11-15  Daniel Bates  <dabates@apple.com>
 
         Disallow loads using HTTP 0.9 at the ResourceHandle/NetworkDataTask level

Source/WebKit2/UIProcess/API/C/WKPage.cpp

@@ -2748,8 +2748,6 @@ WKMediaState WKPageGetMediaState(WKPageRef page)
         state |= kWKMediaIsPlayingAudio;
     if (coreState & WebCore::MediaProducer::IsPlayingVideo)
         state |= kWKMediaIsPlayingVideo;
-    if (coreState & WebCore::MediaProducer::HasMediaCaptureDevice)
-        state |= kWKMediaHasCaptureDevice;
     if (coreState & WebCore::MediaProducer::HasActiveMediaCaptureDevice)
         state |= kWKMediaHasActiveCaptureDevice;
 

Source/WebKit2/UIProcess/API/C/WKPagePrivate.h

@@ -134,7 +134,6 @@ enum {
     kWKMediaIsPlayingAudio = 1 << 0,
     kWKMediaIsPlayingVideo = 1 << 1,
     kWKMediaHasActiveCaptureDevice = 1 << 2,
-    kWKMediaHasCaptureDevice = 1 << 3,
 };
 typedef uint32_t WKMediaState;