WebmMuxer-MediaRecorderHandler: thread hopping and data ownership

content/renderer/media/video_track_recorder.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/media/video_track_recorder.h"
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "base/sys_info.h"
 #include "base/threading/thread.h"
 #include "base/time/time.h"
 #include "base/trace_event/trace_event.h"
 #include "media/base/video_frame.h"
+#include "ui/gfx/geometry/size.h"
 
 extern "C" {
 // VPX_CODEC_DISABLE_COMPAT excludes parts of the libvpx API that provide
 // backwards compatibility for legacy applications using the library.
 #define VPX_CODEC_DISABLE_COMPAT 1
 #include "third_party/libvpx/source/libvpx/vpx/vp8cx.h"
 #include "third_party/libvpx/source/libvpx/vpx/vpx_encoder.h"
 }
 
 using media::VideoFrame;
@@ skipping 20 matching lines @@
 typedef scoped_ptr<vpx_codec_ctx_t, VpxCodecDeleter> ScopedVpxCodecCtxPtr;
 
 void OnFrameEncodeCompleted(
     const VideoTrackRecorder::OnEncodedVideoCB& on_encoded_video_cb,
     const scoped_refptr<VideoFrame>& frame,
     scoped_ptr<std::string> data,
     base::TimeTicks capture_timestamp,
     bool keyframe) {
   DVLOG(1) << (keyframe ? "" : "non ") << "keyframe "<< data->length() << "B, "
            << capture_timestamp << " ms";
-  on_encoded_video_cb.Run(frame, base::StringPiece(*data), capture_timestamp,
-                          keyframe);
+  on_encoded_video_cb.Run(frame, data.Pass(), capture_timestamp, keyframe);
 }
 
 }  // anonymous namespace
 
 // Inner class encapsulating all libvpx interactions and the encoding+delivery
 // of received frames. Limitation: Only VP8 is supported for the time being.
 // This class must be ref-counted because the MediaStreamVideoTrack will hold a
 // reference to it, via the callback MediaStreamVideoSink passes along, and it's
 // unknown when exactly it will release that reference. This class:
 // - is created and destroyed on its parent's thread (usually the main Render
@@ skipping 273 matching lines @@
   track_.reset();
 }
 
 void VideoTrackRecorder::OnVideoFrameForTesting(
     const scoped_refptr<media::VideoFrame>& frame,
     base::TimeTicks timestamp) {
   encoder_->StartFrameEncode(frame, timestamp);
 }
 
 }  // namespace content