media : Return after rejecting promise in MediaDrmBridge

media : Return after rejecting promise in MediaDrmBridge

This fixes a bug where we continue to call Java after rejecting a
promise, which could trigger rejecting the same promise again.

BUG=597514
TEST=No double promise rejection in manual testing.

Committed: https://crrev.com/950d2d3ba6e0aee4bbae0c67eb6c77feaae55314
Cr-Commit-Position: refs/heads/master@{#383110}

[xhwang, 2016-03-24 06:12:20]

xhwang@chromium.org changed reviewers:
+ ddorwin@chromium.org

[xhwang, 2016-03-24 06:12:21]

PTAL

[xhwang, 2016-03-24 06:26:45]

More details: This bug wasn't exposed because in most cases the initdata is
valid so we seldom fail here. Recently this is exposed because of another bug.
See https://chromiumcodereview.appspot.com/1828153002/

[ddorwin, 2016-03-24 16:43:38]

lgtm

[xhwang, 2016-03-24 16:44:42]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-03-24 16:45:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823413003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823413003/1

[commit-bot: I haz the power, 2016-03-24 18:57:14]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-24 18:58:49]

Description was changed from

==========
media : Return after rejecting promise in MediaDrmBridge

This fixes a bug where we continue to call Java after rejecting a
promise, which could trigger rejecting the same promise again.

BUG=597514
TEST=No double promise rejection in manual testing.
==========

to

==========
media : Return after rejecting promise in MediaDrmBridge

This fixes a bug where we continue to call Java after rejecting a
promise, which could trigger rejecting the same promise again.

BUG=597514
TEST=No double promise rejection in manual testing.

Committed: https://crrev.com/950d2d3ba6e0aee4bbae0c67eb6c77feaae55314
Cr-Commit-Position: refs/heads/master@{#383110}
==========

[commit-bot: I haz the power, 2016-03-24 18:58:50]

Patchset 1 (id:??) landed as
https://crrev.com/950d2d3ba6e0aee4bbae0c67eb6c77feaae55314
Cr-Commit-Position: refs/heads/master@{#383110}