Commit 5335df06 authored by Jose Dapena Paz's avatar Jose Dapena Paz Committed by Commit Bot

GCC: to be able to pass through cross thread copier webRTC mocks they cannot be abstract

MockWebRtcVideoTrack and MockWebRtcVideoTrackSource are abstract because they do
not implement the rtc::RefCounted interface. To fix hte problem, solution is
inheriting from rtc::RefCountedObject.

Bug: 819294
Change-Id: Ia0975711cff5755f0608e801a5cb60d175bd9b42
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2310414Reviewed-by: default avatarGuido Urdaneta <guidou@chromium.org>
Commit-Queue: José Dapena Paz <jdapena@igalia.com>
Cr-Commit-Position: refs/heads/master@{#791288}
parent 2ebb347f
......@@ -12,6 +12,7 @@
#include "base/macros.h"
#include "third_party/blink/renderer/modules/peerconnection/peer_connection_dependency_factory.h"
#include "third_party/webrtc/api/media_stream_interface.h"
#include "third_party/webrtc/rtc_base/ref_counted_object.h"
namespace base {
class SingleThreadTaskRunner;
......@@ -65,7 +66,8 @@ class MockWebRtcAudioTrack : public webrtc::AudioTrackInterface {
ObserverSet observers_;
};
class MockWebRtcVideoTrackSource : public webrtc::VideoTrackSourceInterface {
class MockWebRtcVideoTrackSource
: public rtc::RefCountedObject<webrtc::VideoTrackSourceInterface> {
public:
static scoped_refptr<MockWebRtcVideoTrackSource> Create(
bool supports_encoded_output);
......@@ -91,7 +93,8 @@ class MockWebRtcVideoTrackSource : public webrtc::VideoTrackSourceInterface {
bool supports_encoded_output_;
};
class MockWebRtcVideoTrack : public webrtc::VideoTrackInterface {
class MockWebRtcVideoTrack
: public rtc::RefCountedObject<webrtc::VideoTrackInterface> {
public:
static scoped_refptr<MockWebRtcVideoTrack> Create(
const std::string& id,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment