Implement unprefixed EME APIs that match the latest draft

Implement unprefixed EME APIs that match the latest draft

Update the API for WebContentDecryptionModule and
WebContentDecryptionModuleSession in order to implement
the latest EME API spec.

BUG=224786
TEST=Adding to API, no code using it. All existing EME tests pass.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183627

[jrummell, 2014-10-09 00:20:34]

jrummell@chromium.org changed reviewers:
+ ddorwin@chromium.org, sandersd@chromium.org

[jrummell, 2014-10-09 00:20:34]

PTAL. This combines all the API changes needed so far.

[ddorwin, 2014-10-09 00:25:25]

LGTM % comment. Thanks!

https://codereview.chromium.org/642583003/diff/1/public/platform/WebContentDe...
File public/platform/WebContentDecryptionModuleSession.h (right):

https://codereview.chromium.org/642583003/diff/1/public/platform/WebContentDe...
public/platform/WebContentDecryptionModuleSession.h:59: // Called when the
expiration time for the session changes. |time| is
|time|?

[jrummell, 2014-10-09 00:30:08]

jrummell@chromium.org changed reviewers:
+ dglazkov@chromium.org

[jrummell, 2014-10-09 00:30:08]

+dglazkov@ for OWNERS review of Source/platform and public/platform.

https://codereview.chromium.org/642583003/diff/1/public/platform/WebContentDe...
File public/platform/WebContentDecryptionModuleSession.h (right):

https://codereview.chromium.org/642583003/diff/1/public/platform/WebContentDe...
public/platform/WebContentDecryptionModuleSession.h:59: // Called when the
expiration time for the session changes. |time| is
On 2014/10/09 00:25:25, ddorwin wrote:
> |time|?

Fixed.

[jrummell, 2014-10-13 20:05:22]

dglazkov@: friendly ping.

[dglazkov, 2014-10-13 20:06:01]

lgtm

[jrummell, 2014-10-13 20:08:31]

Thanks for the reviews.

[jrummell, 2014-10-13 20:08:37]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-10-13 20:08:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/642583003/20001

[commit-bot: I haz the power, 2014-10-13 20:09:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-13 20:09:13]

Failed to apply patch for Source/modules/encryptedmedia/MediaKeySession.h:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/modules/encryptedmedia/MediaKeySession.h
  Hunk #1 FAILED at 108.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/modules/encryptedmedia/MediaKeySession.h.rej

Patch:       Source/modules/encryptedmedia/MediaKeySession.h
Index: Source/modules/encryptedmedia/MediaKeySession.h
diff --git a/Source/modules/encryptedmedia/MediaKeySession.h
b/Source/modules/encryptedmedia/MediaKeySession.h
index
31c1a202759f25c5051e316d90a397139bba4dcd..6a95416e182c3dadbfd93f1a17fa0374481e0763
100644
--- a/Source/modules/encryptedmedia/MediaKeySession.h
+++ b/Source/modules/encryptedmedia/MediaKeySession.h
@@ -108,6 +108,7 @@ private:
     virtual void close() OVERRIDE;
     virtual void error(MediaKeyErrorCode, unsigned long systemCode) OVERRIDE;
     virtual void error(WebContentDecryptionModuleException, unsigned long
systemCode, const WebString& errorMessage) OVERRIDE;
+    virtual void expirationChanged(double updatedExpiryTimeInMS) OVERRIDE;
 
     ScriptPromise generateRequestInternal(ScriptState*, const String&
initDataType, PassRefPtr<ArrayBuffer> initData);
     ScriptPromise updateInternal(ScriptState*, PassRefPtr<ArrayBuffer>
response);

[jrummell, 2014-10-13 20:53:15]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-10-13 20:54:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/642583003/110001

[commit-bot: I haz the power, 2014-10-13 22:27:46]

Committed patchset #3 (id:110001) as 183627