Enable WKPreferences._lowPowerVideoAudioBufferSizeEnabled by default
authorjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Oct 2018 23:01:42 +0000 (23:01 +0000)
committerjer.noble@apple.com <jer.noble@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 18 Oct 2018 23:01:42 +0000 (23:01 +0000)
https://bugs.webkit.org/show_bug.cgi?id=190315
Source/WebKit:

Reviewed by Eric Carlson.

This preference is disabled for WebKitLegacy because it can interact poorly with clients' own use of audio.
It can be enabled for WebKit since it will only affect the WebProcess and not the client process.

* Shared/WebPreferences.yaml:

LayoutTests:

<rdar://problem/45047807>

Reviewed by Eric Carlson.

* media/audio-controls-timeline-in-media-document-expected.txt:
* media/audio-controls-timeline-in-media-document.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@237273 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/media/audio-controls-timeline-in-media-document-expected.txt
LayoutTests/media/audio-controls-timeline-in-media-document.html
Source/WebKit/ChangeLog
Source/WebKit/Shared/WebPreferences.yaml

index 6f9ae7f..1f6f4d7 100644 (file)
@@ -1,5 +1,16 @@
 2018-10-18  Jer Noble  <jer.noble@apple.com>
 
+        Enable WKPreferences._lowPowerVideoAudioBufferSizeEnabled by default
+        https://bugs.webkit.org/show_bug.cgi?id=190315
+        <rdar://problem/45047807>
+
+        Reviewed by Eric Carlson.
+
+        * media/audio-controls-timeline-in-media-document-expected.txt:
+        * media/audio-controls-timeline-in-media-document.html:
+
+2018-10-18  Jer Noble  <jer.noble@apple.com>
+
         Safari is not able to adapt between H264 streams with EditList and without EditList
         https://bugs.webkit.org/show_bug.cgi?id=190638
         <rdar://problem/45342208>
index e6f35e7..92755bb 100644 (file)
@@ -1,6 +1,6 @@
 Pass if the timeline of audio in a media document is properly updated during playback.
 
 EVENT(timeupdate)
-TEST(timeLineValue() > 0) OK
+EXPECTED (timeLineValue() > '0') OK
 END OF TEST
 
index 28718a2..1121bfc 100644 (file)
@@ -13,13 +13,12 @@ function timeLineValue()
     return timeline.value;
 }
 
-function iframeLoaded()
+async function iframeLoaded()
 {
     video = iframe.contentDocument.querySelector("video");
-    video.addEventListener("timeupdate", function() {
-        consoleWrite("EVENT(timeupdate)");
-        testAndEnd("timeLineValue() > 0");
-    });
+    await waitFor(video, 'timeupdate');
+    await testExpectedEventually("timeLineValue()", 0, ">");
+    endTest();
 }
 
 var video;
index 0e879ff..1fa0251 100644 (file)
@@ -1,3 +1,15 @@
+2018-10-18  Jer Noble  <jer.noble@apple.com>
+
+        Enable WKPreferences._lowPowerVideoAudioBufferSizeEnabled by default
+        https://bugs.webkit.org/show_bug.cgi?id=190315
+
+        Reviewed by Eric Carlson.
+
+        This preference is disabled for WebKitLegacy because it can interact poorly with clients' own use of audio.
+        It can be enabled for WebKit since it will only affect the WebProcess and not the client process.
+
+        * Shared/WebPreferences.yaml:
+
 2018-10-18  Eric Carlson  <eric.carlson@apple.com>
 
         [MediaStream] Allow ports to optionally do screen capture in the UI process
index 1458a21..3c10171 100644 (file)
@@ -452,7 +452,7 @@ HiddenPageCSSAnimationSuspensionEnabled:
 
 LowPowerVideoAudioBufferSizeEnabled:
   type: bool
-  defaultValue: false
+  defaultValue: true
   webcoreBinding: DeprecatedGlobalSettings
 
 ThreadedScrollingEnabled: