[Bug 20991] MediaKeys constructor: make CDM loading asynchronous and fix the load failure algorithm

https://www.w3.org/Bugs/Public/show_bug.cgi?id=20991

David Dorwin <ddorwin@google.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #7 from David Dorwin <ddorwin@google.com> ---
The above change added the following synchronous step to createSession()'s
algorithm.
"If there is a MediaKeyError stored with the MediaKeys object that occurred
because of an error loading the cdm then queue a task to fire a simple event
named error at the MediaKeySession object and abort these steps."

However, CDM loading may not may not have completed yet, which might mean we
miss such an error and/or continue with steps before the CDM is ready.

I recommend:
a) Move this step to the asynchronous steps.
b) Before that step, insert a new step to wait for CDM initialization
(initiated in the MediaKeys constructor) has not completed, wait for it to
complete.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

Received on Tuesday, 5 November 2013 21:24:52 UTC