Fix WebVR requestAnimationFrame leaking

third_party/WebKit/Source/modules/vr/VRDisplay.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/vr/VRDisplay.h"
 
 #include "core/css/StylePropertySet.h"
 #include "core/dom/DOMException.h"
 #include "core/dom/DocumentUserGestureToken.h"
 #include "core/dom/FrameRequestCallback.h"
@@ skipping 732 matching lines @@
 }
 
 ExecutionContext* VRDisplay::getExecutionContext() const {
   return ContextLifecycleObserver::getExecutionContext();
 }
 
 const AtomicString& VRDisplay::interfaceName() const {
   return EventTargetNames::VRDisplay;
 }
 
 void VRDisplay::contextDestroyed() {

[mthiesse, 2017/01/13 00:34:19]

Why ::contextDestroyed() rather than ::dispose()?

   forceExitPresent();
+  m_scriptedAnimationController.clear();
 }
 
 bool VRDisplay::hasPendingActivity() const {
   // Prevent V8 from garbage collecting the wrapper object if there are
   // event listeners attached to it.
   return getExecutionContext() && hasEventListeners();
 }
 
 DEFINE_TRACE(VRDisplay) {
   EventTargetWithInlineData::trace(visitor);
   ContextLifecycleObserver::trace(visitor);
   visitor->trace(m_navigatorVR);
   visitor->trace(m_capabilities);
   visitor->trace(m_stageParameters);
   visitor->trace(m_eyeParametersLeft);
   visitor->trace(m_eyeParametersRight);
   visitor->trace(m_layer);
   visitor->trace(m_renderingContext);
   visitor->trace(m_scriptedAnimationController);
   visitor->trace(m_pendingPresentResolvers);
 }
 
 }  // namespace blink