Add TimedCanvasDrawListener

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

Index: third_party/WebKit/Source/modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.cpp
diff --git a/third_party/WebKit/Source/modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.cpp b/third_party/WebKit/Source/modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.cpp
index a8a12892e584146fc2c1a7917173909d933991e4..ac3b5d1edcbf70b9d08ccf451f4bd26cc383a5f8 100644
--- a/third_party/WebKit/Source/modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.cpp
+++ b/third_party/WebKit/Source/modules/mediacapturefromelement/CanvasCaptureMediaStreamTrack.cpp
@@ -7,14 +7,15 @@
 
 #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
@@ -52,12 +53,16 @@ CanvasCaptureMediaStreamTrack::CanvasCaptureMediaStreamTrack(const CanvasCapture
     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());
 }