Add EME Layout test for unclosed the key session

third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp

Index: third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
diff --git a/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp b/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
index 86a5f8bc181d1d84f3a6dfdadc2fb14cb614dc42..59de623df83f24062bca1b130085d850c7bb9e4b 100644
--- a/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
+++ b/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
@@ -1033,6 +1033,21 @@ bool MediaKeySession::hasPendingActivity() const {
 }
 
 void MediaKeySession::contextDestroyed() {
+  DVLOG(MEDIA_KEY_SESSION_LOG_LEVEL) << __func__ << "(" << this << ")";
+
+  // If call contextDestroyed before close session, clear session without
+  // closeTask.
+  // So call closeTask if there is a delayed pendingAction::Close.
+  if (m_actionTimer.isActive()) {

[haraken, 2016/11/01 02:18:33]

Do we need to check this?

[Dongheun Kang, 2016/11/01 03:32:56]

I think there seems to be no need.

+    while (!m_pendingActions.isEmpty()) {
+      PendingAction* action = m_pendingActions.takeFirst();
+      if (action->getType() == PendingAction::Close) {
+        closeTask(action->result());
+        break;

[haraken, 2016/11/01 02:18:33]

Isn't it possible that there are more than one pending actions?

[Dongheun Kang, 2016/11/01 03:32:56]

So I can change as like following code.

  if (!m_pendingActions.isEmpty())
    actionTimerFired(0);

Do you this about?

[Dongheun Kang, 2016/11/01 04:05:37]

Sorry. I miss-understand your comments. This patch is just for differed
closeTask. So add break.
But I will simply change the patch.
Thanks.

+      }
+    }
+  }
+
   // Stop the CDM from firing any more events for this session.
   m_session.reset();
   m_isClosed = true;