The change for that issue also reordered the first two steps and replaces the reject-with-NotSupportedError with resolve(false).

There was also this comment about how the steps in our implementation do not match the spec. I'm not sure whether this is observable and thus should be fixed.

>Speaking of implementations, Blink/Chromium currently implements step 2 within step 5, deferring that determination to the implementation rather than checking up front. This seems within the definitions for observable behavior except that the promise is rejected (will be resolved) asynchronously instead of synchronously as in the spec. is that difference observable? If so and this pattern is common, should we change the algorithm?

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1af40e160276b4bd756bed244c1b3709b9ba417f

commit 1af40e160276b4bd756bed244c1b3709b9ba417f
Author: jrummell <jrummell@chromium.org>
Date: Mon Jun 13 20:15:19 2016

Pass SetServerCertificate call on

Have SetServerCertificate() call the matching method on |decryptor_|
rather than resolving the promise immediately. This allows External
Clear Key to more closely match the functionality of Clear Key.

BUG= 618800 
TEST=encrypted media browser tests pass

Review-Url: https://codereview.chromium.org/2057103004
Cr-Commit-Position: refs/heads/master@{#399520}

[modify] https://crrev.com/1af40e160276b4bd756bed244c1b3709b9ba417f/media/cdm/ppapi/external_clear_key/clear_key_cdm.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e3d85c9614d5864bc3a9506b529036f2d703e0ec

commit e3d85c9614d5864bc3a9506b529036f2d703e0ec
Author: jrummell <jrummell@chromium.org>
Date: Mon Jun 13 23:26:49 2016

Change MediaKeys.setServerCertificate() to return Promise<boolean>

https://github.com/w3c/encrypted-media/issues/145 changed the EME spec
so that setServerCertificate() returns Promise<boolean>. Update the
implementation to match.

This change updates the IDL and returns true on success.
NOT_SUPPORTED_ERRORs will resolve with false.

BUG= 618800 
TEST=EME tests pass, www.netflix.com plays

Review-Url: https://codereview.chromium.org/2056053003
Cr-Commit-Position: refs/heads/master@{#399595}

[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-async-setcert-with-gc.html
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/Source/modules/encryptedmedia/ContentDecryptionModuleResultPromise.h
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/Source/modules/encryptedmedia/MediaKeys.cpp
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/Source/modules/encryptedmedia/MediaKeys.idl

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1af40e160276b4bd756bed244c1b3709b9ba417f

commit 1af40e160276b4bd756bed244c1b3709b9ba417f
Author: jrummell <jrummell@chromium.org>
Date: Mon Jun 13 20:15:19 2016

Pass SetServerCertificate call on

Have SetServerCertificate() call the matching method on |decryptor_|
rather than resolving the promise immediately. This allows External
Clear Key to more closely match the functionality of Clear Key.

BUG= 618800 
TEST=encrypted media browser tests pass

Review-Url: https://codereview.chromium.org/2057103004
Cr-Commit-Position: refs/heads/master@{#399520}

[modify] https://crrev.com/1af40e160276b4bd756bed244c1b3709b9ba417f/media/cdm/ppapi/external_clear_key/clear_key_cdm.cc

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e3d85c9614d5864bc3a9506b529036f2d703e0ec

commit e3d85c9614d5864bc3a9506b529036f2d703e0ec
Author: jrummell <jrummell@chromium.org>
Date: Mon Jun 13 23:26:49 2016

Change MediaKeys.setServerCertificate() to return Promise<boolean>

https://github.com/w3c/encrypted-media/issues/145 changed the EME spec
so that setServerCertificate() returns Promise<boolean>. Update the
implementation to match.

This change updates the IDL and returns true on success.
NOT_SUPPORTED_ERRORs will resolve with false.

BUG= 618800 
TEST=EME tests pass, www.netflix.com plays

Review-Url: https://codereview.chromium.org/2056053003
Cr-Commit-Position: refs/heads/master@{#399595}

[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-async-setcert-with-gc.html
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/LayoutTests/media/encrypted-media/encrypted-media-syntax.html
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/Source/modules/encryptedmedia/ContentDecryptionModuleResultPromise.h
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/Source/modules/encryptedmedia/MediaKeys.cpp
[modify] https://crrev.com/e3d85c9614d5864bc3a9506b529036f2d703e0ec/third_party/WebKit/Source/modules/encryptedmedia/MediaKeys.idl

It looks like the CLs landed before M53 branch cut.