From 457554d5c2ca104e22c6ecb34734ba2be9b1059c Mon Sep 17 00:00:00 2001 From: xhwang Date: Tue, 17 Nov 2015 12:38:03 -0800 Subject: [PATCH] media: Post task in MediaDrmBridge::OnSessionExpirationUpdate(). This was missed in https://codereview.chromium.org/1414923004/ BUG=546108 Review URL: https://codereview.chromium.org/1455723002 Cr-Commit-Position: refs/heads/master@{#360152} --- media/base/android/media_drm_bridge.cc | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/media/base/android/media_drm_bridge.cc b/media/base/android/media_drm_bridge.cc index fee16c7d62e8a..f3ffc0f976e09 100644 --- a/media/base/android/media_drm_bridge.cc +++ b/media/base/android/media_drm_bridge.cc @@ -529,6 +529,7 @@ void MediaDrmBridge::SetMediaCryptoReadyCB( media_crypto_ready_cb_ = media_crypto_ready_cb; } +//------------------------------------------------------------------------------ // The following OnXxx functions are called from Java. The implementation must // only do minimal work and then post tasks to avoid reentrancy issues. @@ -672,9 +673,10 @@ void MediaDrmBridge::OnSessionExpirationUpdate(JNIEnv* env, jbyteArray j_session_id, jlong expiry_time_ms) { DVLOG(2) << __FUNCTION__ << ": " << expiry_time_ms << " ms"; - session_expiration_update_cb_.Run( - AsString(env, j_session_id), - base::Time::FromDoubleT(expiry_time_ms / 1000.0)); + task_runner_->PostTask( + FROM_HERE, + base::Bind(session_expiration_update_cb_, AsString(env, j_session_id), + base::Time::FromDoubleT(expiry_time_ms / 1000.0))); } void MediaDrmBridge::OnLegacySessionError(JNIEnv* env, @@ -701,6 +703,7 @@ void MediaDrmBridge::OnResetDeviceCredentialsCompleted(JNIEnv* env, base::Bind(base::ResetAndReturn(&reset_credentials_cb_), success)); } +//------------------------------------------------------------------------------ // The following are private methods. MediaDrmBridge::MediaDrmBridge(