Attachment 350025 Details for Bug 177501 – Patch

[patch] Patch

bug-177501-20180918094833.patch (text/plain), 4.66 KB, created by youenn fablet on 2018-09-18 09:48:34 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2018-09-18 09:48:34 PDT

Size: 4.66 KB

patch

obsolete

>Subversion Revision: 236098
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 715675b0ff32617bcb4325885854bc24c667987f..14a6dc5470d4b147a673b4a0bb1510f616bb925e 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-09-18  Youenn Fablet  <youenn@apple.com>
>+
>+        Layout Test webrtc/video-mute.html is flaky.
>+        https://bugs.webkit.org/show_bug.cgi?id=177501
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Covered by updated test expectation.
>+
>+        * platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm:
>+        (WebCore::createBlackPixelBuffer): Make sure the whole buffer is properly initialized.
>+
> 2018-09-17  Jer Noble  <jer.noble@apple.com>
> 
>         Add support for HEVC codec types in Media Capabilities
>diff --git a/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm b/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm
>index d86a59f8c617d0770c5dff360bae26d25332890e..4e4a0f8676d61093e67d91a8520cd5f4aa8f977b 100644
>--- a/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm
>+++ b/Source/WebCore/platform/mediastream/mac/RealtimeIncomingVideoSourceCocoa.mm
>@@ -73,9 +73,12 @@ RetainPtr<CVPixelBufferRef> createBlackPixelBuffer(size_t width, size_t height)
>     status = CVPixelBufferLockBaseAddress(pixelBuffer, 0);
>     ASSERT(status == noErr);
>     void* data = CVPixelBufferGetBaseAddress(pixelBuffer);
>+
>     size_t yLength = width * height;
>     memset(data, 0, yLength);
>-    memset(static_cast<uint8_t*>(data) + yLength, 128, yLength / 2);
>+
>+    auto totalSize = CVPixelBufferGetDataSize(pixelBuffer);
>+    memset(static_cast<uint8_t*>(data) + yLength, 128, totalSize - yLength);
> 
>     status = CVPixelBufferUnlockBaseAddress(pixelBuffer, 0);
>     ASSERT(!status);
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index d60033a4bce1a5818c601524061fa165a3aa057f..4c2272a0ca4598d49b80c8719dcc56067f0d6a04 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,14 @@
>+2018-09-18  Youenn Fablet  <youenn@apple.com>
>+
>+        Layout Test webrtc/video-mute.html is flaky.
>+        https://bugs.webkit.org/show_bug.cgi?id=177501
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * platform/ios-wk2/TestExpectations:
>+        * platform/mac-wk2/TestExpectations:
>+        * webrtc/routines.js: Increase counter to make test more robust.
>+
> 2018-09-17  Ryan Haddad  <ryanhaddad@apple.com>
> 
>         Layout Test legacy-animation-engine/animations/play-state-in-shorthand.html is a flaky failure
>diff --git a/LayoutTests/platform/ios-wk2/TestExpectations b/LayoutTests/platform/ios-wk2/TestExpectations
>index 0b27c86017c7244272bce0f4bc2a27645cf4af8d..9c0b1ca179864d45f2a7f9251b0a63e112d85521 100644
>--- a/LayoutTests/platform/ios-wk2/TestExpectations
>+++ b/LayoutTests/platform/ios-wk2/TestExpectations
>@@ -1234,8 +1234,6 @@ webkit.org/b/176240 http/tests/websocket/tests/hybi/handshake-ok-with-legacy-web
> 
> webkit.org/b/173608 webrtc/video-replace-muted-track.html [ Skip ]
> 
>-webkit.org/b/177501 webrtc/video-mute.html [ Pass Timeout ]
>-
> webkit.org/b/187388 animations/needs-layout.html [ Pass ImageOnlyFailure ]
> webkit.org/b/172397 legacy-animation-engine/animations/needs-layout.html [ Pass ImageOnlyFailure ]
> 
>diff --git a/LayoutTests/platform/mac-wk2/TestExpectations b/LayoutTests/platform/mac-wk2/TestExpectations
>index a74d475d684988fc748a8a88e22d9cd8d0f10c03..cf048f4b75d6c7deb73be695e71cb512c67b072e 100644
>--- a/LayoutTests/platform/mac-wk2/TestExpectations
>+++ b/LayoutTests/platform/mac-wk2/TestExpectations
>@@ -783,8 +783,6 @@ webkit.org/b/176881 plugins/js-from-destroy.html [ Pass Failure ]
> 
> webkit.org/b/178553 [ HighSierra Release ] imported/w3c/web-platform-tests/html/dom/dynamic-markup-insertion/opening-the-input-stream/011.html [ Skip ]
> 
>-webkit.org/b/177501 webrtc/video-mute.html [ Pass Timeout ]
>-
> webkit.org/b/165620 [ Sierra ] http/tests/cache/disk-cache/disk-cache-request-headers.html [ Pass Failure ]
> 
> webkit.org/b/179773 imported/w3c/web-platform-tests/fetch/api/abort/serviceworker-intercepted.https.html [ Pass Failure ]
>diff --git a/LayoutTests/webrtc/routines.js b/LayoutTests/webrtc/routines.js
>index 11499aefc03663257b0fa032ac2022db5de91fe3..78c0b01c29735aee35ff9cbe334c6dcdff3b8ef3 100644
>--- a/LayoutTests/webrtc/routines.js
>+++ b/LayoutTests/webrtc/routines.js
>@@ -195,7 +195,7 @@ async function checkVideoBlack(expected, canvas, video, errorMessage, counter)
> 
>     if (counter === undefined)
>         counter = 0;
>-    if (counter > 50) {
>+    if (counter > 200) {
>         if (!errorMessage)
>             errorMessage = "checkVideoBlack timed out expecting " + expected;
>         return Promise.reject(errorMessage);

Actions: View | Formatted Diff | Diff

Attachments on bug 177501: 350025 | 350035 | 350065 | 350067