Add EME Layout test for unclosed the key session

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

 /*
  * Copyright (C) 2013 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1015 matching lines @@
       << (m_asyncEventQueue->hasPendingEvents()
               ? " m_asyncEventQueue->hasPendingEvents()"
               : "")
       << ((m_mediaKeys && !m_isClosed) ? " m_mediaKeys && !m_isClosed" : "");
 
   return !m_pendingActions.isEmpty() || m_asyncEventQueue->hasPendingEvents() ||
          (m_mediaKeys && !m_isClosed);
 }
 
 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;
   m_actionTimer.stop();
   m_pendingActions.clear();
   m_asyncEventQueue->close();
 }
 
 DEFINE_TRACE(MediaKeySession) {
   visitor->trace(m_asyncEventQueue);
   visitor->trace(m_pendingActions);
   visitor->trace(m_mediaKeys);
   visitor->trace(m_keyStatusesMap);
   visitor->trace(m_closedPromise);
   EventTargetWithInlineData::trace(visitor);
   ActiveDOMObject::trace(visitor);
 }
 
 }  // namespace blink