The cricket::VideoCapturer on longer makes a copy of the frame from the camera. As a result we can'…

content/renderer/media/video_source_handler.cc

 // Copyright (c) 2013 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_source_handler.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "content/renderer/media/media_stream_dependency_factory.h"
@@ skipping 18 matching lines @@
   PpFrameReceiver() : reader_(NULL) {}
   virtual ~PpFrameReceiver() {}
 
   // Implements VideoRenderer.
   virtual bool SetSize(int width, int height, int reserved) OVERRIDE {
     return true;
   }
   virtual bool RenderFrame(const cricket::VideoFrame* frame) OVERRIDE {
     base::AutoLock auto_lock(lock_);
     if (reader_) {
-      // Make a shallow copy of the frame as the |reader_| may need to queue it.
-      // Both frames will share a single reference-counted frame buffer.
-      reader_->GotFrame(frame->Copy());
+      // |frame| will be invalid after this function is returned. So keep a copy
+      // before return.
+      cricket::VideoFrame* copied_frame = frame->Copy();
+      copied_frame->MakeExclusive();
+      reader_->GotFrame(copied_frame);
     }
     return true;
   }
 
   void SetReader(FrameReaderInterface* reader) {
     base::AutoLock auto_lock(lock_);
     reader_ = reader;
   }
 
  private:
@@ skipping 91 matching lines @@
   receiver_->SetReader(reader);
 }
 
 VideoSourceHandler::SourceInfo::~SourceInfo() {
   source_->RemoveSink(receiver_.get());
   receiver_->SetReader(NULL);
 }
 
 }  // namespace content