Only call finishLoad() if session actually loaded.

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 257 matching lines @@
     {
     }
 
     // ContentDecryptionModuleResult implementation.
     virtual void completeWithSession(WebContentDecryptionModuleResult::SessionStatus status) override
     {
         bool result = false;
         switch (status) {
         case WebContentDecryptionModuleResult::NewSession:
             result = true;
+            m_session->finishLoad();
             break;
 
         case WebContentDecryptionModuleResult::SessionNotFound:
             result = false;

[ddorwin, 2015/07/07 21:05:11]

I think we can clean this up a little now to make the behavior super clear. For
example:
 resolve(false); return;
 eliminate |result|
 replace l291 with ASSERT_NOT_REACHED();
WDYT?

[jrummell, 2015/07/08 18:43:35]

Done.

             break;
 
         case WebContentDecryptionModuleResult::SessionAlreadyExists:
             ASSERT_NOT_REACHED();
             reject(InvalidStateError, "Unexpected completion.");
             return;
         }
 
-        m_session->finishLoad();
         resolve(result);
     }
 
     DEFINE_INLINE_TRACE()
     {
         visitor->trace(m_session);
         ContentDecryptionModuleResultPromise::trace(visitor);
     }
 
 private:
@@ skipping 623 matching lines @@
     visitor->trace(m_asyncEventQueue);
     visitor->trace(m_pendingActions);
     visitor->trace(m_mediaKeys);
     visitor->trace(m_keyStatusesMap);
     visitor->trace(m_closedPromise);
     RefCountedGarbageCollectedEventTargetWithInlineData<MediaKeySession>::trace(visitor);
     ActiveDOMObject::trace(visitor);
 }
 
 } // namespace blink