1464934 - [wpt-sync] Sync PR 11209 - Reland "More video protocol tests"

Status: RESOLVED FIXED

(Core :: WebRTC, enhancement, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 11209 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/11209
Details from upstream follow.

Harald Alvestrand <[e-mail]> wrote:
>  Reland "More video protocol tests"
>  
>  This reverts commit 43c098364c9470c09542798210c828d6c10a562f.
>  
>  Reason for revert: Updated TestExpectations to reflect test name change
>  
>  Original change's description:
>  > Revert "More video protocol tests"
>  > 
>  > This reverts commit 89e8f4ddc3551b54158e7b177f756ae81e6bec07.
>  > 
>  > Reason for revert: failing on WebKit Linux Trusty Leak 
>  > see:
>  > https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20Leak/19437
>  > 
>  > 03:46:20.467 25977 worker/4 virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs.https.html output stderr lines:
>  > 03:46:20.467 25977   [1:17:0528/034620.208283:ERROR:adm_helpers.cc(73)] Failed to query stereo recording.
>  > 03:46:20.467 25977   [1:16:0528/034620.408117:WARNING:rtpreceiver.cc(273)] VideoRtpReceiver::Stop: No video channel exists.
>  > 03:46:20.467 25977   [1:16:0528/034620.412041:WARNING:rtpreceiver.cc(273)] VideoRtpReceiver::Stop: No video channel exists.
>  > 03:46:20.468 25977   [1:16:0528/034620.414673:WARNING:rtpreceiver.cc(273)] VideoRtpReceiver::Stop: No video channel exists.
>  > 03:46:20.469 25819 [1846/6906] virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs.https.html failed unexpectedly (asserts failed)
>  > 03:46:20.468 25977 worker/4 virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs.https.html failed:
>  > 03:46:20.468 25977 worker/4  asserts failed
>  > 
>  > Original change's description:
>  > > More video protocol tests
>  > > 
>  > > This bunch of tests uses the getParameters function to get insight into
>  > > conformance of the video requirements.
>  > > 
>  > > It tests the codecs after negotiation, and that profile-level-id is
>  > > always set.
>  > > 
>  > > Bug: chromium:839753
>  > > Change-Id: I607f1ae21320698bd1633be14ae9c33752c16a1c
>  > > Reviewed-on: https://chromium-review.googlesource.com/1072087
>  > > Reviewed-by: Guido Urdaneta <[e-mail]>
>  > > Commit-Queue: Harald Alvestrand <[e-mail]>
>  > > Cr-Commit-Position: refs/heads/master@{#562224}
>  > 
>  > TBR=[e-mail],[e-mail],[e-mail]
>  > 
>  > Change-Id: I35685e607a3ea2cabcf7971a652b93298366e1b8
>  > No-Presubmit: true
>  > No-Tree-Checks: true
>  > No-Try: true
>  > Bug: chromium:839753
>  > Reviewed-on: https://chromium-review.googlesource.com/1075067
>  > Reviewed-by: Olga Sharonova <[e-mail]>
>  > Commit-Queue: Olga Sharonova <[e-mail]>
>  > Cr-Commit-Position: refs/heads/master@{#562231}
>  
>  TBR=[e-mail],[e-mail],[e-mail],[e-mail]
>  
>  Change-Id: Id8f74407371d9b58401c9a898235b789844b2cb9
>  No-Presubmit: true
>  No-Tree-Checks: true
>  No-Try: true
>  Bug: chromium:839753
>  Reviewed-on: https://chromium-review.googlesource.com/1075068
>  Commit-Queue: Harald Alvestrand <[e-mail]>
>  Reviewed-by: Harald Alvestrand <[e-mail]>
>  Cr-Commit-Position: refs/heads/master@{#562234}
>

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=5489ce73de0655a9a8f45ff841d42e6d22d0b773

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=379108ebcc1d1e9caa2b87a22ea09bef0256e914

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Ran 1 tests and 3 subtests
OK     : 1
FAIL   : 3

New tests that have failures or other problems:
/webrtc/protocol/video-codecs.https.html
    All H.264 codecs MUST include profile-level-id: FAIL
    H.264 and VP8 should be negotiated after handshake: FAIL
    H.264 and VP8 should be supported in initial offer: FAIL

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0625631da20b
[wpt PR 11209] - Reland "More video protocol tests", a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/fdc37f5fdf83
[wpt PR 11209] - Update wpt metadata, a=testonly

Comment 5

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/0625631da20b
https://hg.mozilla.org/mozilla-central/rev/fdc37f5fdf83