Title: [256346] trunk
Revision
256346
Author
eric.carl...@apple.com
Date
2020-02-11 13:14:44 -0800 (Tue, 11 Feb 2020)

Log Message

video.currentTime is not being updated in iOS 13.4 Beta
https://bugs.webkit.org/show_bug.cgi?id=207489
<rdar://problem/59322640>

Reviewed by Youenn Fablet.

Source/WebKit:

Allow the Web process and the GPU process to communicate with
'com.apple.coremedia.audiodeviceclock.xpc' because it is necessary to use a
CMTimeBase, which is used by MediaPlayerPrivateMediaStreamAVFObjC for currentTime.

Test: fast/mediastream/media-element-current-time.html

* Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb:
* Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:

LayoutTests:

* fast/mediastream/media-element-current-time.html: Added.

Modified Paths

Added Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (256345 => 256346)


--- trunk/LayoutTests/ChangeLog	2020-02-11 21:09:21 UTC (rev 256345)
+++ trunk/LayoutTests/ChangeLog	2020-02-11 21:14:44 UTC (rev 256346)
@@ -1,3 +1,13 @@
+2020-02-11  Eric Carlson  <eric.carl...@apple.com>
+
+        video.currentTime is not being updated in iOS 13.4 Beta
+        https://bugs.webkit.org/show_bug.cgi?id=207489
+        <rdar://problem/59322640>
+
+        Reviewed by Youenn Fablet.
+
+        * fast/mediastream/media-element-current-time.html: Added.
+
 2020-02-11  Jason Lawrence  <lawrenc...@apple.com>
 
         [ Mac wk2 ] http/tests/ssl/applepay/ApplePayBillingAddress.html is flaky failing.

Added: trunk/LayoutTests/fast/mediastream/media-element-current-time-expected.txt (0 => 256346)


--- trunk/LayoutTests/fast/mediastream/media-element-current-time-expected.txt	                        (rev 0)
+++ trunk/LayoutTests/fast/mediastream/media-element-current-time-expected.txt	2020-02-11 21:14:44 UTC (rev 256346)
@@ -0,0 +1,4 @@
+
+
+PASS Check video.currentTime behavior 
+

Added: trunk/LayoutTests/fast/mediastream/media-element-current-time.html (0 => 256346)


--- trunk/LayoutTests/fast/mediastream/media-element-current-time.html	                        (rev 0)
+++ trunk/LayoutTests/fast/mediastream/media-element-current-time.html	2020-02-11 21:14:44 UTC (rev 256346)
@@ -0,0 +1,31 @@
+<body>
+<video id="video" autoplay playsInline></video>
+<script src=""
+<script src=""
+<script>
+
+promise_test(async() => {
+
+    let stream = await navigator.mediaDevices.getUserMedia({video : true});
+
+    video.srcObject = stream;
+    assert_equals(video.currentTime, 0);
+
+    await video.play();
+    let currentTime = video.currentTime;
+    assert_not_equals(currentTime, 0, "Playback has started, currentTime must not be zero");
+
+    await new Promise(resolve => setTimeout(resolve, 10));
+    assert_greater_than(video.currentTime, currentTime, "video is playing, time should advance");
+
+    video.pause();
+    currentTime = video.currentTime;
+    await new Promise(resolve => setTimeout(resolve, 10));
+    assert_equals(video.currentTime, currentTime, "video is paused, currentTime must not advance");
+
+    await video.play();
+    assert_greater_than(video.currentTime, currentTime, "Playback has started, currentTime should advance");
+
+}, "Check video.currentTime behavior");
+</script>
+</body>

Modified: trunk/Source/WebKit/ChangeLog (256345 => 256346)


--- trunk/Source/WebKit/ChangeLog	2020-02-11 21:09:21 UTC (rev 256345)
+++ trunk/Source/WebKit/ChangeLog	2020-02-11 21:14:44 UTC (rev 256346)
@@ -1,3 +1,20 @@
+2020-02-11  Eric Carlson  <eric.carl...@apple.com>
+
+        video.currentTime is not being updated in iOS 13.4 Beta
+        https://bugs.webkit.org/show_bug.cgi?id=207489
+        <rdar://problem/59322640>
+
+        Reviewed by Youenn Fablet.
+
+        Allow the Web process and the GPU process to communicate with
+        'com.apple.coremedia.audiodeviceclock.xpc' because it is necessary to use a 
+        CMTimeBase, which is used by MediaPlayerPrivateMediaStreamAVFObjC for currentTime.
+
+        Test: fast/mediastream/media-element-current-time.html
+
+        * Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb:
+        * Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:
+
 2020-02-11  Alex Christensen  <achristen...@webkit.org>
 
         Deprecate _WKUserContentWorld replaced by WKContentWorld

Modified: trunk/Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb (256345 => 256346)


--- trunk/Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb	2020-02-11 21:09:21 UTC (rev 256345)
+++ trunk/Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.GPU.sb	2020-02-11 21:14:44 UTC (rev 256346)
@@ -140,6 +140,7 @@
            (global-name "com.apple.coremedia.admin")
            (global-name "com.apple.coremedia.asset.xpc")
            (global-name "com.apple.coremedia.assetimagegenerator.xpc")
+           (global-name "com.apple.coremedia.audiodeviceclock.xpc") ; Needed for CMTimeBase
            (global-name "com.apple.coremedia.audioprocessingtap.xpc")
            (global-name "com.apple.coremedia.capturesession")      ; Actually for video capture
            (global-name "com.apple.coremedia.capturesource")       ; Also for video capture (<rdar://problem/15794291>).

Modified: trunk/Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb (256345 => 256346)


--- trunk/Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb	2020-02-11 21:09:21 UTC (rev 256345)
+++ trunk/Source/WebKit/Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb	2020-02-11 21:14:44 UTC (rev 256346)
@@ -154,6 +154,7 @@
            (global-name "com.apple.coremedia.admin")
            (global-name "com.apple.coremedia.asset.xpc")
            (global-name "com.apple.coremedia.assetimagegenerator.xpc")
+           (global-name "com.apple.coremedia.audiodeviceclock.xpc") ; Needed for CMTimeBase
            (global-name "com.apple.coremedia.audioprocessingtap.xpc")
            (global-name "com.apple.coremedia.capturesession")      ; Actually for video capture
            (global-name "com.apple.coremedia.capturesource")       ; Also for video capture (<rdar://problem/15794291>).
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to