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

Side by Side Diff: content/renderer/pepper/pepper_media_stream_video_track_host.cc

Issue 1476523005: Verify returned frames from media::VideoFrame::Wrap*() methods (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/pepper/pepper_media_stream_video_track_host.h" 5 #include "content/renderer/pepper/pepper_media_stream_video_track_host.h"
6 6
7 #include "base/base64.h" 7 #include "base/base64.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/rand_util.h" 9 #include "base/rand_util.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 337 matching lines...) Expand 10 before | Expand all | Expand 10 after
348 plugin_frame_size_, 348 plugin_frame_size_,
349 gfx::Rect(plugin_frame_size_), 349 gfx::Rect(plugin_frame_size_),
350 plugin_frame_size_, 350 plugin_frame_size_,
351 y_stride, 351 y_stride,
352 uv_stride, 352 uv_stride,
353 uv_stride, 353 uv_stride,
354 y_data, 354 y_data,
355 u_data, 355 u_data,
356 v_data, 356 v_data,
357 base::TimeDelta::FromMilliseconds(ts_ms)); 357 base::TimeDelta::FromMilliseconds(ts_ms));
358 358 if (!frame)
359 return PP_ERROR_FAILED;
359 frame_deliverer_->DeliverVideoFrame(frame); 360 frame_deliverer_->DeliverVideoFrame(frame);
Pawel Osciak 2015/11/26 01:19:03 Please empty line above.
emircan 2015/12/04 04:20:09 Done.
360 } 361 }
361 362
362 // Makes the frame available again for plugin. 363 // Makes the frame available again for plugin.
363 SendEnqueueBufferMessageToPlugin(index); 364 SendEnqueueBufferMessageToPlugin(index);
364 return PP_OK; 365 return PP_OK;
365 } 366 }
366 367
367 void PepperMediaStreamVideoTrackHost::OnVideoFrame( 368 void PepperMediaStreamVideoTrackHost::OnVideoFrame(
368 const scoped_refptr<VideoFrame>& frame, 369 const scoped_refptr<VideoFrame>& frame,
369 base::TimeTicks estimated_capture_time) { 370 base::TimeTicks estimated_capture_time) {
(...skipping 152 matching lines...) Expand 10 before | Expand all | Expand 10 after
522 } 523 }
523 524
524 void PepperMediaStreamVideoTrackHost::OnTrackStarted( 525 void PepperMediaStreamVideoTrackHost::OnTrackStarted(
525 MediaStreamSource* source, 526 MediaStreamSource* source,
526 MediaStreamRequestResult result, 527 MediaStreamRequestResult result,
527 const blink::WebString& result_name) { 528 const blink::WebString& result_name) {
528 DVLOG(3) << "OnTrackStarted result: " << result; 529 DVLOG(3) << "OnTrackStarted result: " << result;
529 } 530 }
530 531
531 } // namespace content 532 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698