Set callback interface on WebCDMSessionImpl after creation.

Set callback interface on WebCDMSessionImpl after creation.

In order to allow MediaKeySession to be created only when resolving
the promise, add the ability to set the client callback interface on
WebContentDecryptionModuleSessionImpl after the object is created.

This will be used in a subsequent CL to update blink. Requires a third
change to remove the old methods once blink is updated.

BUG=358271
TEST=EME layouts tests pass

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283854

[jrummell, 2014-07-09 23:53:27]

PTAL. blink changes in https://codereview.chromium.org/379343003/.

[ddorwin, 2014-07-10 23:22:41]

LG, but let's explore the Blink side more before landing.

https://codereview.chromium.org/380953002/diff/1/content/renderer/media/webco...
File content/renderer/media/webcontentdecryptionmodulesession_impl.cc (right):

https://codereview.chromium.org/380953002/diff/1/content/renderer/media/webco...
content/renderer/media/webcontentdecryptionmodulesession_impl.cc:214:
client_->message(
This is the interesting point, especially after the refactoring. We might add:
DCHECK(client_) << "Client not set before message event";

[jrummell, 2014-07-17 00:56:55]

Updated.

https://codereview.chromium.org/380953002/diff/1/content/renderer/media/webco...
File content/renderer/media/webcontentdecryptionmodulesession_impl.cc (right):

https://codereview.chromium.org/380953002/diff/1/content/renderer/media/webco...
content/renderer/media/webcontentdecryptionmodulesession_impl.cc:214:
client_->message(
On 2014/07/10 23:22:41, ddorwin wrote:
> This is the interesting point, especially after the refactoring. We might add:
> DCHECK(client_) << "Client not set before message event";

Done.

[ddorwin, 2014-07-17 01:02:33]

LGTM, but let's get the public APIs reviewed before landing this.

[jrummell, 2014-07-17 17:39:43]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2014-07-17 17:41:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jrummell@chromium.org/380953002/20001

[commit-bot: I haz the power, 2014-07-17 20:24:14]

Change committed as 283854