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..224e1f13e9a8f9c8bae21237fd7b3bf26733e8b6 100644
--- a/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
+++ b/third_party/WebKit/Source/modules/encryptedmedia/MediaKeySession.cpp
@@ -1033,6 +1033,14 @@ 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 actionTimerFired(0) if there is a delayed pendingAction.
+  if (!m_pendingActions.isEmpty())
+    actionTimerFired(0);

[haraken, 2016/11/01 06:28:25]

If there is a pending task with PendingAction::GenerateRequest, you'll end up
with initiating a new request in contextDestroyed(). Is it really what you want
to do here?

[Dongheun Kang, 2016/11/01 06:54:44]

No. I'd better rollback to a previous patch.

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