W3C home > Mailing lists > Public > public-html-media@w3.org > September 2016

Re: HME WG meeting minutes, Tue Aug 30

From: Matt Wolenetz <wolenetz@google.com>
Date: Wed, 7 Sep 2016 14:22:50 -0700
Message-ID: <CAADho6O5nB3C3-mw7OVymRUrmYYwvF+u63G0gcZCjR78eAuQcA@mail.gmail.com>
To: Philippe Le Hégaret <plh@w3.org>
Cc: Paul Cotton <Paul.Cotton@microsoft.com>, "public-html-media@w3.org" <public-html-media@w3.org>
Thanks Philippe. Help with the IDL failures would be appreciated -- it
looks like it's not specific to just Chrome. As mentioned in narrower
thread, I've updated and landed #3655, and will rerun Chrome's JSON (for
builds with, and without, ISO-BMFF support claimed by isTypeSupported())
once I land https://codereview.chromium.org/2315113002/ (which should fix a
bunch of the TypeError-related Chrome failures). Landing that Chrome patch
may not come before EOD today, though.

On Wed, Sep 7, 2016 at 12:03 PM, Philippe Le Hégaret <plh@w3.org> wrote:

>
>
> On 9/2/2016 7:24 PM, Matt Wolenetz wrote:
>
> I've made some progress, not as much as I had hoped. 2 of my PRs are
> currently pending co-editor review. 5 pending PRs are still awaiting my
> full review.
>
>
> Thank you Matt for making progress!
>
> I merged one of your PRs, still looking at #3655.
>
> I regenerated test results and added a filter to eliminate
> VideoPlaybackQuality and TrackDefault related:
>   http://w3c.github.io/test-results/media-source/less-than-2.html
>
> We get a lot of errors on the idlharness tests which I'll look at as well.
>
> Philippe
>
>
> Detail:
>
>    - Prepped PR to cover 4 gaps from #823 : https://github.com/w3c/web-p
>    latform-tests/pull/3635
>       - *Awaiting @jdsmith3000's review to land it*
>       - crbug 623781 <http://crbug/623781> tracks fixing the Chrome
>       compliance issue behind at least the failure of 2 subtests (TypeError) in
>       mediasource-addsourcebuffer.html
>       - Filed crbug 643788 <http://crbug/643788> to fix debug build
>       assertion in w-p-t mediasource-removesourcebuffers.html and landed
>       my fix for it <https://codereview.chromium.org/2305023002>
>
>
>    - Reviewed and closed ancient https://github.com/
>    w3c/web-platform-tests/pull/939
>    <https://github.com/w3c/web-platform-tests/pull/939>
>       - Found only 1 trivial test coverage gap, prepped and merged
>       trivial PR to close that gap: https://github.com/w3c/
>       web-platform-tests/pull/3636
>    - Deferred to VNext ancient appendStream coverage test PR
>    https://github.com/w3c/web-platform-tests/pull/954
>    <https://github.com/w3c/web-platform-tests/pull/954>
>    - Deferred to VNext more strict isTypeSupported() coverage test PR
>    https://github.com/w3c/web-platform-tests/pull/1405
>    <https://github.com/w3c/web-platform-tests/pull/1405>
>    -
>
>    Commented on mechanical PR https://github.com/w3c/web-
>    platform-tests/pull/1816 (doesn't block MSE v1 PR, it's a
>    cross-cutting infra PR)
>    -
>
>    In-progress: Fixing https://github.com/w3c/web-platform-tests/pull/3082
>    - Filed, and in-progress fixing (short-term Chrome MSE piece of) crbug
>       643846 <http://crbug/643846>
>    -
>
>    TODO 5 more pending PRs...
>
>
> On Tue, Aug 30, 2016 at 6:42 PM, Matt Wolenetz <wolenetz@google.com>
> wrote:
>
>>
>> On Tue, Aug 30, 2016 at 11:20 AM, Paul Cotton <Paul.Cotton@microsoft.com>
>> wrote:
>>
>>> *MSE test suite results*
>>>
>>> <*plh*> Paul: still some outstanding pull requests
>>>
>>> https://lists.w3.org/Archives/Public/public-html-media/2016Aug/0102.html
>>> is Matt's most recent update
>>>
>>> <*plh*> Matt: going through them...
>>>
>>> Outstanding PRs: https://github.com/w3c/web-platform-tests/pulls?q=is%
>>> 3Apr+is%3Aopen+label%3Amedia-source
>>>
>>> <*plh*> ... resolving the merge conflict
>>>
>>> <*plh*> ... I'll send out an update later today
>>>
>>
>> I've worked through about 5 of the 15 pending PRs today, and will
>> continue tomorrow. Current status:
>>
>>    -
>>    - Merged @tidoust's https://github.com/w3c/web-platform-tests/pull/32
>>    39
>>    - Prepped and landed my https://github.com/w3c/web-
>>    platform-tests/pull/3612
>>    - Needed Chromium+W3C license stamp, so this replaces/extends
>>    @tidoust's https://github.com/w3c/web-platform-tests/pull/3264
>>    - Updated and landed my https://github.com/w3c/web-
>>    platform-tests/pull/3296
>>    - Landed @tidoust's https://github.com/w3c/web-platform-tests/pull/32
>>    33
>>    - Prepped PR to fix the test introduced in #3233 to comply with
>>    w3c/media-source#154: https://github.com/w3c/web-platform-te
>>    sts/pull/3613
>>       - *Awaiting @jdsmith3000's review to land it*
>>       - crbug 639144 <http://crbug/639144> tracks fixing the Chrome
>>       compliance issue behind failure of subtest 1/3 in this PR
>>       - crbug 373039 <http://crbug/373039> is one of a group of bugs
>>       tracking fixing the Chrome compliance issue behind failure of subtest 3/3
>>       in this PR
>>    - Reviewed ancient https://github.com/w3c/web-platform-tests/pull/823
>>       - Found 4 gaps, but otherwise closed this PR since most of the
>>       coverage it includes is already covered elsewhere in our current suite
>>    -
>>
>>    In-progress: put together PR to cover 4 gaps from #823
>>    - crbug 623781 <http://crbug/623781> tracks fixing the Chrome
>>       compliance issue behind at least the failure of 2 subtests (TypeError) in
>>       mediasource-addsourcebuffer.html
>>       - *In-progress: Investigating Chromium issue with PR I'm building
>>       to cover the 4 test gaps.*
>>    - TODO 10 more pending PRs, followed by call to regenerate reports
>>    and lists of known user agent bugs tracking test failures.
>>
>>
>
>
Received on Wednesday, 7 September 2016 21:24:04 UTC

This archive was generated by hypermail 2.3.1 : Wednesday, 7 September 2016 21:24:05 UTC