Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1055)

Unified Diff: content/renderer/media/webrtc/media_stream_remote_video_source.cc

Issue 2855783003: Fix dependencies on deprecated webrtc videoframe features. (Closed)
Patch Set: Add missing gn dependency on video_frame_api. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/media/webrtc/media_stream_remote_video_source.cc
diff --git a/content/renderer/media/webrtc/media_stream_remote_video_source.cc b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
index a3aa59e0fa00f287f052055cec5c05fcbc2b6626..d433204ac62f11007cf4c716c2484ea51004f082 100644
--- a/content/renderer/media/webrtc/media_stream_remote_video_source.cc
+++ b/content/renderer/media/webrtc/media_stream_remote_video_source.cc
@@ -16,6 +16,7 @@
#include "media/base/timestamp_constants.h"
#include "media/base/video_frame.h"
#include "media/base/video_util.h"
+#include "third_party/webrtc/api/video/i420_buffer.h"
#include "third_party/webrtc/media/base/videosinkinterface.h"
namespace content {
@@ -123,7 +124,7 @@ void MediaStreamRemoteVideoSource::RemoteVideoSourceDelegate::OnFrame(
WebRTCToMediaVideoRotation(incoming_frame.rotation()));
}
} else {
- buffer = webrtc::I420Buffer::Rotate(incoming_frame.video_frame_buffer(),
+ buffer = webrtc::I420Buffer::Rotate(*incoming_frame.video_frame_buffer(),
incoming_frame.rotation());
gfx::Size size(buffer->width(), buffer->height());

Powered by Google App Engine
This is Rietveld 408576698