Add TimedCanvasDrawListener

third_party/WebKit/Source/modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.cpp

 // Copyright 2015 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 "config.h"
 #include "modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.h"
 
 #include "core/html/HTMLCanvasElement.h"
 #include "modules/mediacapturefromelement/AutoCanvasDrawListener.h"
+#include "modules/mediacapturefromelement/TimedCanvasDrawListener.h"
 #include "platform/NotImplemented.h"
 #include "platform/mediastream/MediaStreamCenter.h"
 
 namespace blink {
 
-CanvasCaptureMediaStreamTrack* CanvasCaptureMediaStreamTrack::create(MediaStreamComponent* component, PassRefPtrWillBeRawPtr<HTMLCanvasElement> element, const PassOwnPtr<WebCanvasCaptureHandler> handler)
+CanvasCaptureMediaStreamTrack* CanvasCaptureMediaStreamTrack::create(MediaStreamComponent* component, PassRefPtrWillBeRawPtr<HTMLCanvasElement> element, const PassOwnPtr<WebCanvasCaptureHandler> handler, bool givenFrameRate, double frameRate)
 {
-    return new CanvasCaptureMediaStreamTrack(component, element, handler);
+    return new CanvasCaptureMediaStreamTrack(component, element, handler, givenFrameRate, frameRate);
 }
 
 HTMLCanvasElement* CanvasCaptureMediaStreamTrack::canvas() const
 {
     return m_canvasElement.get();
 }
 
 void CanvasCaptureMediaStreamTrack::requestFrame()
 {
     notImplemented();
@@ skipping 17 matching lines @@
 
 CanvasCaptureMediaStreamTrack::CanvasCaptureMediaStreamTrack(const CanvasCaptureMediaStreamTrack& track, MediaStreamComponent* component)
     :MediaStreamTrack(track.m_canvasElement->executionContext(), component)
     , m_canvasElement(track.m_canvasElement)
     , m_drawListener(track.m_drawListener)
 {
     suspendIfNeeded();
     m_canvasElement->addListener(m_drawListener.get());
 }
 
-CanvasCaptureMediaStreamTrack::CanvasCaptureMediaStreamTrack(MediaStreamComponent* component, PassRefPtrWillBeRawPtr<HTMLCanvasElement> element, const PassOwnPtr<WebCanvasCaptureHandler> handler)
+CanvasCaptureMediaStreamTrack::CanvasCaptureMediaStreamTrack(MediaStreamComponent* component, PassRefPtrWillBeRawPtr<HTMLCanvasElement> element, const PassOwnPtr<WebCanvasCaptureHandler> handler, bool givenFrameRate, double frameRate)

[esprehn, 2015/12/15 23:58:01]

this is very strange, lets just add two constructors, one with a frameFrame, and
one without. Having the bool where it controls if we respect or ignore that
other arg is weird.

[emircan, 2015/12/16 15:36:21]

Done.

     : MediaStreamTrack(element->executionContext(), component)
     , m_canvasElement(element)
 {
     suspendIfNeeded();
-    m_drawListener = AutoCanvasDrawListener::create(handler);
+    if (givenFrameRate) {
+        m_drawListener = TimedCanvasDrawListener::create(handler, frameRate);
+    } else {
+        m_drawListener = AutoCanvasDrawListener::create(handler);
+    }
     m_canvasElement->addListener(m_drawListener.get());
 }
 
 } // namespace blink