Implement supports for MediaCapabilities object

RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
2 years ago
9 months ago

People

(Reporter: jya, Assigned: jya)

Tracking

(Blocks 1 bug, {dev-doc-complete})

55 Branch
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

(URL)

Attachments

(28 attachments)

46 bytes, text/x-phabricator-request
bzbarsky
: review+
Details | Review
46 bytes, text/x-phabricator-request
bzbarsky
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
gerald
: review+
Details | Review
46 bytes, text/x-phabricator-request
gerald
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
padenot
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
baku
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
bzbarsky
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
gerald
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
46 bytes, text/x-phabricator-request
bryce
: review+
Details | Review
(Assignee)

Description

2 years ago
This bug is to track support for Media Capabilities as proposed by Google.

Do we want to support this object, do we have fingerprinting concerns and so on.
Ethan,
Are there any concerns from the security perspective, like anti-fingerprinting?
Flags: needinfo?(ettseng)
In the abstract of the spec [1]:
"The APIs will expose information about the decoding and encoding capabilities for a given format but also output capabilities to find the best match based on the device’s display."

I imagine this will be a fingerprinting attribute, similar to the MediaDevices API [2], which we are adding fingerprinting protection in bug 1372073.

Since this is a new API, we should let the Tor Browser be aware of it.
Once we confirmed it's a fingerprinting threat, we will file a new bug for the protection work.


[1] https://wicg.github.io/media-capabilities/
[2] https://developer.mozilla.org/en-US/docs/Web/API/MediaDevices
Flags: needinfo?(ettseng)
IMHO, now that media decoding/encoding stack for playback and WebRTC will be unified (or is it already so?), we should also strive to have only one capability decision implementation for both if deciding to support this.
(Assignee)

Updated

a year ago
Summary: Implement supports for Media Capabilities object → Implement supports for MediaCapabilities object
(Assignee)

Updated

a year ago
Priority: P3 → P2
(Assignee)

Comment 5

a year ago
This appears to be the most wanted feature by all the partners we've met...
Assignee: nobody → jyavenard
As per https://wicg.github.io/media-capabilities/#idl-index

Placed behind user pref media.mediacapabilities.enabled that is disabled by default

MozReview-Commit-ID: 9zg4aRdGolw
MozReview-Commit-ID: 5qq2VxSM58z

Depends on D1614
The information returned is identical to
1- canPlayType() for file
2- MediaSource.isTypeSupported for media-source
3- MediaRecorder.isTypeRecorder() for recordings.

MozReview-Commit-ID: 2JeGx9o8ozv

Depends on D1616
MozReview-Commit-ID: 4FaLAZl7Ca3

Depends on D1617
Let's always enable it.

MozReview-Commit-ID: 1cwI8LHRynu

Depends on D1618
MozReview-Commit-ID: HlGQjUtocvU

Depends on D1619
Can be used in place of a TaskQueue, but without the need to explicitly shut it down. It will automatically be shutdown once it's no longer referenced.

MozReview-Commit-ID: 1lJcTqtk2r6

Depends on D1620
It is used outside MSE, ultimately this should move to xpcom inside TaskQueue.h

MozReview-Commit-ID: GpoGnhVA0X5

Depends on D1621
Addtionally, change the framerate to be of type double and allow to create a MediaExtendedMIMEType based on the new dom VideoConfiguration and AudioConfiguration object.

MozReview-Commit-ID: B1VkOhmzsdU

Depends on D1622
Allows to build an array ot TrackInfo built from the mimetype provided. This will allow to create dummy decoder to check that if they are supported and how well the decoder will perform.

MozReview-Commit-ID: I4OH9EY1Ldl

Depends on D1623
We know those sampling rate aren't supported and cause initialization errors later.

MozReview-Commit-ID: Ds1yLd0h6rI

Depends on D1624
We can't create a H264 VT decoder until we have all SPS/PPS NALs, which makes it tricky to generate when we only want to check if H264 is supported.
On mac, we can reasonable assume that hardware acceleration is always supported (though on a mac pro 2013 that isn't the case or hackintosh with nvidia cards).

MozReview-Commit-ID: nS2VVLYN39

Depends on D1625
We now provides more detailed information for audio (check sampling rate and channels if provided).

And check for the power efficient attribute. We directly correlate this information with the decoder being hardware accelerated or not. All audio codecs are deemed to be power efficient.

MozReview-Commit-ID: HcZlbMDSADb

Depends on D1626
Most tests will pass now.
The remaining tests failing are related to incompatibility with our bindings generator, and will be addressed in a later change.

MozReview-Commit-ID: HGrJ5gT9aHd

Depends on D1627
For flac, mp3 and adts, if a codec was provided but wasn't supported in the container, it would have reported Maybe instead of No

MozReview-Commit-ID: ECMNX1R8hTz

Depends on D1628
MozReview-Commit-ID: 1Yamljittrh

Depends on D1629
Silence warnings.

Depends on D1630

MozReview-Commit-ID: GTzH2UtEBXz
To properly determine if a decoder is hardware accelerated, we must pass information about the compositor to the decoder.

MozReview-Commit-ID: J9AkfbIBIM0

Depends on D1631
By default, when creating a H264 decoder it is wrapped into a H264Converter which will only create the actual decoder once a valid SPS/PPS has been seen.
As creating valid SPS/PPS NALs isn't trivial, when all we care about are capabilities of such decoder, we do not wrap the decoder so that it will be immediately created.
We can then test its capabilities.

We only enable this on windows, as on mac we need to generate a SPS/PPS, otherwise the mac decoder always report that HW decoding is not enabled.

Depends on D1632
MozReview-Commit-ID: Iir5yvFRyYr

Updated

11 months ago
Attachment #8984990 - Attachment description: Bug 1409664 - P19. Proper pass KnowsCompositor object to decoder. r?bryce → Bug 1409664 - P19. Properly pass KnowsCompositor object to decoder. r?bryce, r?bz

Updated

11 months ago
Attachment #8984970 - Attachment description: Bug 1409664 - P3. Allow audio/webm container type in MediaRecorder. r?bryce → Bug 1409664 - P3. Allow audio/webm container type in MediaRecorder.

Updated

11 months ago
Attachment #8984971 - Attachment description: Bug 1409664 - P4. Provide basis MediaCapabilities support. r?bryce → Bug 1409664 - P4. Provide basis MediaCapabilities support

Updated

11 months ago
Attachment #8984972 - Attachment description: Bug 1409664 - P5. Fix comment. r?bryce → Bug 1409664 - P5. Fix comment

Updated

11 months ago
Attachment #8984973 - Attachment description: Bug 1409664 - P6. Remove flac in ogg preference. r?bryce → Bug 1409664 - P6. Remove flac in ogg preference

Updated

11 months ago
Attachment #8984978 - Attachment description: Bug 1409664 - P9. Move AutoTaskQueue.h to dom/media. r?bryce → Bug 1409664 - P9. Move AutoTaskQueue.h to dom/media

Updated

11 months ago
Attachment #8984979 - Attachment description: Bug 1409664 - P10. Extend MediaExtendedMIMEType to know about audio configuration. r?bryce → Bug 1409664 - P10. Extend MediaExtendedMIMEType to know about audio configuration

Updated

11 months ago
Attachment #8984980 - Attachment description: Bug 1409664 - P11. Add GetTracksInfo method to all MediaDecoder instances. r?bryce → Bug 1409664 - P11. Add GetTracksInfo method to all MediaDecoder instances

Updated

11 months ago
Attachment #8984981 - Attachment description: Bug 1409664 - P12. Don't attempt to play non-supported AAC sampling rate. r?padenot → Bug 1409664 - P12. Don't attempt to play non-supported AAC sampling rate

Updated

11 months ago
Attachment #8984982 - Attachment description: Bug 1409664 - P13. Report the Apple's H264 decoder is hardware accelerated when not yet initialized. r?bryce → Bug 1409664 - P13. Report the Apple's H264 decoder is hardware accelerated when not yet initialized

Updated

11 months ago
Attachment #8984983 - Attachment description: Bug 1409664 - P14. Provide more details for audio and video configuration. r?bryce → Bug 1409664 - P14. Provide more details for audio and video configuration

Updated

11 months ago
Attachment #8984985 - Attachment description: Bug 1409664 - P16. Fix canPlayType so that it checks codecs if provided. r?bryce → Bug 1409664 - P16. Fix canPlayType so that it checks codecs if provided

Updated

11 months ago
Attachment #8984987 - Attachment description: Bug 1409664 - P17. Fix typo in variable name. r?bryce → Bug 1409664 - P17. Fix typo in variable name

Updated

11 months ago
Attachment #8984988 - Attachment description: Bug 1409664 - P18. Remove unused variables. r?bryce → Bug 1409664 - P18. Remove unused variables

Updated

11 months ago
Attachment #8984990 - Attachment description: Bug 1409664 - P19. Properly pass KnowsCompositor object to decoder. r?bryce, r?bz → Bug 1409664 - P19. Properly pass KnowsCompositor object to decoder

Updated

11 months ago
Attachment #8984991 - Attachment description: Bug 1409664 - P20. Don't wrap H264 decoder in H264 converter. r?bryce → Bug 1409664 - P20. Don't wrap H264 decoder in H264 converter

Updated

11 months ago
Attachment #8984969 - Attachment description: Bug 1409664 - P2. Add ScreenLuminance objects and friends to Screen. r?bz → Bug 1409664 - P2. Add ScreenLuminance objects and friends to Screen
Comment on attachment 8984981 [details]
Bug 1409664 - P12. Don't attempt to play non-supported AAC sampling rate

Paul Adenot (:padenot) has approved the revision.

https://phabricator.services.mozilla.com/D1625
Attachment #8984981 - Flags: review+
Comment on attachment 8984970 [details]
Bug 1409664 - P3. Allow audio/webm container type in MediaRecorder.

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1616
Attachment #8984970 - Flags: review+
MozReview-Commit-ID: lTBhFzO38l

Updated

10 months ago
Attachment #8984971 - Attachment description: Bug 1409664 - P4. Provide basis MediaCapabilities support → Bug 1409664 - P4. Provide basic MediaCapabilities support

Updated

10 months ago
Attachment #8984984 - Attachment description: Bug 1409664 - P15. Update wpt expected results. r?bryce → Bug 1409664 - P15. Update wpt expected results

Updated

10 months ago
Attachment #8985757 - Attachment description: Bug 1409664 - P21. Add BitWriter class. r?gerald Depends on D1633 → Bug 1409664 - P21. Add BitWriter class. r?gerald

Updated

10 months ago
Attachment #8984974 - Attachment description: Bug 1409664 - P7. Remove unused TaskQueue method. r?gerald → Bug 1409664 - P7. Remove unused TaskQueue method

Updated

10 months ago
Attachment #8984977 - Attachment description: Bug 1409664 - P8. Add TaskQueueShutdownWrapper class. r?gerald → Bug 1409664 - P8. Add TaskQueueShutdownWrapper class

Updated

10 months ago
Attachment #8985757 - Attachment description: Bug 1409664 - P21. Add BitWriter class. r?gerald → Bug 1409664 - P21. Add BitWriter class

Updated

10 months ago
Attachment #8984977 - Attachment description: Bug 1409664 - P8. Add TaskQueueShutdownWrapper class → Bug 1409664 - P8. Make explicit shutdown of TaskQueue optional

Updated

10 months ago
Attachment #8984978 - Attachment description: Bug 1409664 - P9. Move AutoTaskQueue.h to dom/media → Bug 1409664 - P9. Remove AutoTaskQueue and its use
Comment on attachment 8984971 [details]
Bug 1409664 - P4. Provide basic MediaCapabilities support

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1617
Attachment #8984971 - Flags: review+
Comment on attachment 8984972 [details]
Bug 1409664 - P5. Fix comment

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1618
Attachment #8984972 - Flags: review+
Comment on attachment 8984973 [details]
Bug 1409664 - P6. Remove flac in ogg preference

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1619
Attachment #8984973 - Flags: review+
Comment on attachment 8984974 [details]
Bug 1409664 - P7. Remove unused TaskQueue method

Gerald Squelart [:gerald] has approved the revision.

https://phabricator.services.mozilla.com/D1620
Attachment #8984974 - Flags: review+
Comment on attachment 8985757 [details]
Bug 1409664 - P21. Add BitWriter class

Gerald Squelart [:gerald] has approved the revision.

https://phabricator.services.mozilla.com/D1678
Attachment #8985757 - Flags: review+
We'll need it to properly build a SPS/PPS extradata later. Also, change the types used. The original data is stored on two bytes ASCII, it will always fit in a uint8_t. Additionally, this is how those values are stored in a SPS.

MozReview-Commit-ID: C1UBU7OqhQM

Depends on D1678
The Apple VT decoder requires SPS+PPS at construction time. If not provided, in earlier macOS it used to give an error. In the current 10.13 it appears to work, however the decoder always report to be software only.

To properly determine the decoder capabilities, we construct a SPS NAL from the codec mimetype provided.

Details on the structure of the mimetype can be found in https://tools.ietf.org/html/rfc6381#section-3.3 and is a 1:1 match with the data found in the SPS.

MozReview-Commit-ID: I0WeTux14K4

Depends on D1718
Comment on attachment 8984977 [details]
Bug 1409664 - P8. Make explicit shutdown of TaskQueue optional

Gerald Squelart [:gerald] has approved the revision.

https://phabricator.services.mozilla.com/D1621
Attachment #8984977 - Flags: review+
Comment on attachment 8984978 [details]
Bug 1409664 - P9. Remove AutoTaskQueue and its use

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1622
Attachment #8984978 - Flags: review+
Comment on attachment 8984979 [details]
Bug 1409664 - P10. Extend MediaExtendedMIMEType to know about audio configuration

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1623
Attachment #8984979 - Flags: review+
Comment on attachment 8984980 [details]
Bug 1409664 - P11. Add GetTracksInfo method to all MediaDecoder instances

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1624
Attachment #8984980 - Flags: review+
Comment on attachment 8984982 [details]
Bug 1409664 - P13. Report the Apple's H264 decoder is hardware accelerated when not yet initialized

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1626
Attachment #8984982 - Flags: review+
Comment on attachment 8984983 [details]
Bug 1409664 - P14. Provide more details for audio and video configuration

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1627
Attachment #8984983 - Flags: review+
Comment on attachment 8984984 [details]
Bug 1409664 - P15. Update wpt expected results

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1628
Attachment #8984984 - Flags: review+
Comment on attachment 8984985 [details]
Bug 1409664 - P16. Fix canPlayType so that it checks codecs if provided

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1629
Attachment #8984985 - Flags: review+
Comment on attachment 8984987 [details]
Bug 1409664 - P17. Fix typo in variable name

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1630
Attachment #8984987 - Flags: review+
Comment on attachment 8984988 [details]
Bug 1409664 - P18. Remove unused variables

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1631
Attachment #8984988 - Flags: review+
Comment on attachment 8984990 [details]
Bug 1409664 - P19. Properly pass KnowsCompositor object to decoder

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1632
Attachment #8984990 - Flags: review+
(Assignee)

Updated

10 months ago
Blocks: 1470281
Comment on attachment 8984991 [details]
Bug 1409664 - P20. Don't wrap H264 decoder in H264 converter

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1633
Attachment #8984991 - Flags: review+
Comment on attachment 8986332 [details]
Bug 1409664 - P22. Retrieve H264 constraint flags with ExtractH264CodecDetails

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1718
Attachment #8986332 - Flags: review+
Comment on attachment 8986333 [details]
Bug 1409664 - P23. Create SPS/PPS extradata to better determine decoder capabilities

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1719
Attachment #8986333 - Flags: review+
Instead have it use its own task queue.

Depends on D1767
If the benchmark task hasn't run yet, we will assume smoothness for now.

Depends on D1771
This will allow to be called from the MediaCapabilities taskqueue if we find that a decoder won't be hardware accelerated.

It is still assumed that Benchmark::Init() was called at least once on the main thread.

Depends on D1628
Comment on attachment 8987125 [details]
Bug 1409664 - P24. Make IsVP9DecodeFast usable on any thread. r?bryce

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1767
Attachment #8987125 - Flags: review+
(Assignee)

Updated

10 months ago
See Also: → 1470754
MediaCapabilities provide a finer detail on VP9 being supported or not. YouTube will use that information to determine which resolutions to support when using VP9

Depends on D1772
Additionally, consider all videos <= 480p to be smooth and power efficient as:
1- Hardware decoding it typically not used for those
2- We can't do any better
3- Any machines should be able to do 480p

Depends on D1794
Comment on attachment 8987122 [details]
Bug 1409664 - P25. Don't use main thread with Benchmark . r?bryce

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1771
Attachment #8987122 - Flags: review+
Comment on attachment 8987123 [details]
Bug 1409664 - P26. Use VP9 benchmark results to determine smoothness. r?bryce

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1772
Attachment #8987123 - Flags: review+
Comment on attachment 8987376 [details]
Bug 1409664 - P27. Don't disable webm container for MSE::isTypeSupported when MediaCapabilities is enabled. r?bryce

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1794
Attachment #8987376 - Flags: review+
Comment on attachment 8987458 [details]
Bug 1409664 - P28. Extrapolate VP9 benchmark results for requested vp9 configuration. r?bryce

Bryce Van Dyk (:bryce) has approved the revision.

https://phabricator.services.mozilla.com/D1796
Attachment #8987458 - Flags: review+
(Assignee)

Updated

10 months ago
Depends on: 1368949
Comment on attachment 8984990 [details]
Bug 1409664 - P19. Properly pass KnowsCompositor object to decoder

Boris Zbarsky [:bz] (no decent commit message means r-) has approved the revision.

https://phabricator.services.mozilla.com/D1632
Attachment #8984990 - Flags: review+
Comment on attachment 8984968 [details]
Bug 1409664 - P1. Add MediaCapabilities skeleton IDL.

Boris Zbarsky [:bz] (no decent commit message means r-) has approved the revision.

https://phabricator.services.mozilla.com/D1613
Attachment #8984968 - Flags: review+
Comment on attachment 8984969 [details]
Bug 1409664 - P2. Add ScreenLuminance objects and friends to Screen

Boris Zbarsky [:bz] (no decent commit message means r-) has approved the revision.

https://phabricator.services.mozilla.com/D1614
Attachment #8984969 - Flags: review+
Comment on attachment 8984983 [details]
Bug 1409664 - P14. Provide more details for audio and video configuration

Andrea Marchesini [:baku] has approved the revision.

https://phabricator.services.mozilla.com/D1627
Attachment #8984983 - Flags: review+

Comment 64

10 months ago
Pushed by jyavenard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/56cfc9b388ed
P1. Add MediaCapabilities skeleton IDL. r=bz
https://hg.mozilla.org/integration/mozilla-inbound/rev/c27874c06658
P2. Add ScreenLuminance objects and friends to Screen. r=bz
https://hg.mozilla.org/integration/mozilla-inbound/rev/89d390fbbfc2
P3. Allow audio/webm container type in MediaRecorder. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/58be92b1a730
P4. Provide basic MediaCapabilities support. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/f465c541d485
P5. Fix comment. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c4f6e1376b8
P6. Remove flac in ogg preference. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/779fdf8781bd
P7. Remove unused TaskQueue method. r=gerald
https://hg.mozilla.org/integration/mozilla-inbound/rev/75ea1b10a437
P8. Make explicit shutdown of TaskQueue optional. r=gerald
https://hg.mozilla.org/integration/mozilla-inbound/rev/dcb3d64ed761
P9. Remove AutoTaskQueue and its use r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/c9ebc35c018f
P10. Extend MediaExtendedMIMEType to know about audio configuration. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/cc96ea98ea5f
P11. Add GetTracksInfo method to all MediaDecoder instances. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/ab4b6e060b8f
P12. Don't attempt to play non-supported AAC sampling rate. r=padenot
https://hg.mozilla.org/integration/mozilla-inbound/rev/6c6d943c1aa4
P13. Report the Apple's H264 decoder is hardware accelerated when not yet initialized. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/191b6cabf55b
P14. Provide more details for audio and video configuration. r=bryce, r=baku
https://hg.mozilla.org/integration/mozilla-inbound/rev/c98c9828c2dc
P15. Fix canPlayType so that it checks codecs if provided. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/82eccef03093
P16. Fix typo in variable name. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/db78b712d5ec
P17. Remove unused variables. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/25ff69daabca
P18. Properly pass KnowsCompositor object to decoder. r=bryce, r=bz
https://hg.mozilla.org/integration/mozilla-inbound/rev/6fe4e0609af6
P19. Don't wrap H264 decoder in H264 converter. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/b77050ae2d43
P20. Add BitWriter class. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/128bb8d855d4
P21. Retrieve H264 constraint flags with ExtractH264CodecDetails. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/95f7bce33797
P22. Create SPS/PPS extradata to better determine decoder capabilities. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/58e74a5ac2c2
P23. Update wpt expected results. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/14717f5f8f8f
P24. Make IsVP9DecodeFast usable on any thread. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/cd93530bc66e
P25. Don't use main thread with Benchmark . r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/b9081b427eae
P26. Use VP9 benchmark results to determine smoothness. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/d3c49681033c
P27. Don't disable webm container for MSE::isTypeSupported when MediaCapabilities is enabled. r=bryce
https://hg.mozilla.org/integration/mozilla-inbound/rev/994fa896ebaf
P28. Extrapolate VP9 benchmark results for requested vp9 configuration. r=bryce

Comment 65

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/56cfc9b388ed
https://hg.mozilla.org/mozilla-central/rev/c27874c06658
https://hg.mozilla.org/mozilla-central/rev/89d390fbbfc2
https://hg.mozilla.org/mozilla-central/rev/58be92b1a730
https://hg.mozilla.org/mozilla-central/rev/f465c541d485
https://hg.mozilla.org/mozilla-central/rev/9c4f6e1376b8
https://hg.mozilla.org/mozilla-central/rev/779fdf8781bd
https://hg.mozilla.org/mozilla-central/rev/75ea1b10a437
https://hg.mozilla.org/mozilla-central/rev/dcb3d64ed761
https://hg.mozilla.org/mozilla-central/rev/c9ebc35c018f
https://hg.mozilla.org/mozilla-central/rev/cc96ea98ea5f
https://hg.mozilla.org/mozilla-central/rev/ab4b6e060b8f
https://hg.mozilla.org/mozilla-central/rev/6c6d943c1aa4
https://hg.mozilla.org/mozilla-central/rev/191b6cabf55b
https://hg.mozilla.org/mozilla-central/rev/c98c9828c2dc
https://hg.mozilla.org/mozilla-central/rev/82eccef03093
https://hg.mozilla.org/mozilla-central/rev/db78b712d5ec
https://hg.mozilla.org/mozilla-central/rev/25ff69daabca
https://hg.mozilla.org/mozilla-central/rev/6fe4e0609af6
https://hg.mozilla.org/mozilla-central/rev/b77050ae2d43
https://hg.mozilla.org/mozilla-central/rev/128bb8d855d4
https://hg.mozilla.org/mozilla-central/rev/95f7bce33797
https://hg.mozilla.org/mozilla-central/rev/58e74a5ac2c2
https://hg.mozilla.org/mozilla-central/rev/14717f5f8f8f
https://hg.mozilla.org/mozilla-central/rev/cd93530bc66e
https://hg.mozilla.org/mozilla-central/rev/b9081b427eae
https://hg.mozilla.org/mozilla-central/rev/d3c49681033c
https://hg.mozilla.org/mozilla-central/rev/994fa896ebaf
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Blocks: 1480190

Updated

9 months ago
You need to log in before you can comment on or make changes to this bug.