Set "Running" state only when actually running

third_party/WebKit/Source/modules/webaudio/BaseAudioContext.cpp

 /*
  * Copyright (C) 2010, Google 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 752 matching lines @@
 
 void BaseAudioContext::resolvePromisesForResumeOnMainThread() {
   DCHECK(isMainThread());
   AutoLocker locker(this);
 
   for (auto& resolver : m_resumeResolvers) {
     if (m_contextState == Closed) {
       resolver->reject(DOMException::create(
           InvalidStateError, "Cannot resume a context that has been closed"));
     } else {
+      setContextState(Running);
       resolver->resolve();
     }
   }
 
   m_resumeResolvers.clear();
   m_isResolvingResumePromises = false;
 }
 
 void BaseAudioContext::resolvePromisesForResume() {
   // This runs inside the BaseAudioContext's lock when handling pre-render
@@ skipping 81 matching lines @@
 
 const AtomicString& BaseAudioContext::interfaceName() const {
   return EventTargetNames::AudioContext;
 }
 
 ExecutionContext* BaseAudioContext::getExecutionContext() const {
   return SuspendableObject::getExecutionContext();
 }
 
 void BaseAudioContext::startRendering() {
-  // This is called for both online and offline contexts.
+  // This is called for both online and offline contexts.  The caller
+  // must set the context state appropriately. In particular, resuming
+  // a context should wait until the context has actually resumed to
+  // set the state.
   DCHECK(isMainThread());
   DCHECK(m_destinationNode);
   DCHECK(isAllowedToStart());
 
   if (m_contextState == Suspended) {
     destination()->audioDestinationHandler().startRendering();
-    setContextState(Running);
   }
 }
 
 DEFINE_TRACE(BaseAudioContext) {
   visitor->trace(m_destinationNode);
   visitor->trace(m_listener);
   visitor->trace(m_activeSourceNodes);
   visitor->trace(m_resumeResolvers);
   visitor->trace(m_decodeAudioResolvers);
 
   visitor->trace(m_periodicWaveSine);
   visitor->trace(m_periodicWaveSquare);
   visitor->trace(m_periodicWaveSawtooth);
   visitor->trace(m_periodicWaveTriangle);
   EventTargetWithInlineData::trace(visitor);
   SuspendableObject::trace(visitor);
 }
 
 SecurityOrigin* BaseAudioContext::getSecurityOrigin() const {
   if (getExecutionContext())
     return getExecutionContext()->getSecurityOrigin();
 
   return nullptr;
 }
 
 }  // namespace blink