Chromium Code Reviews Chromium Code Reviews
Issue 1584433002:
Delete MockVideoRenderer::SetSize. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc |
| diff --git a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc |
| index 1e1d749c2d37a92375c6a94d90c2ca9ca6602ff3..d100b6c0793e9a5d84a0af2df6d6dfe1a000f207 100644 |
| --- a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc |
| +++ b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc |
| @@ -164,12 +164,6 @@ MockVideoRenderer::MockVideoRenderer() |
| MockVideoRenderer::~MockVideoRenderer() {} |
| -bool MockVideoRenderer::SetSize(int width, int height, int reserved) { |
| - width_ = width; |
|
perkj_chrome
2016/01/12 17:02:33
please also remove width_ and height and their acc
nisse-chromium (ooo August 14)
2016/01/13 07:47:52
Done. I thought they were used, but I mixed it up
|
| - height_ = height; |
| - return true; |
| -} |
| - |
| bool MockVideoRenderer::RenderFrame(const cricket::VideoFrame* frame) { |
| ++num_; |
| return true; |
