Replace media::VideoCapture::VideoFrameBuffer with media::VideoFrame.

content/renderer/media/rtc_video_capturer.h

 // Copyright (c) 2012 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.
 
 #ifndef CONTENT_RENDERER_MEDIA_RTC_VIDEO_CAPTURER_H_
 #define CONTENT_RENDERER_MEDIA_RTC_VIDEO_CAPTURER_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 23 matching lines @@
       const cricket::VideoFormat& capture_format) OVERRIDE;
   virtual void Stop() OVERRIDE;
   virtual bool IsRunning() OVERRIDE;
   virtual bool GetPreferredFourccs(std::vector<uint32>* fourccs) OVERRIDE;
   virtual bool GetBestCaptureFormat(const cricket::VideoFormat& desired,
                                     cricket::VideoFormat* best_format) OVERRIDE;
   virtual bool IsScreencast() const OVERRIDE;
 
  private:
   // Frame captured callback method.
-  virtual void OnFrameCaptured(
-      const media::VideoCapture::VideoFrameBuffer& frame);
+  virtual void OnFrameCaptured(const scoped_refptr<media::VideoFrame>& frame);
 
   // State change callback, must be called on same thread as Start is called.
   void OnStateChange(RtcVideoCaptureDelegate::CaptureState state);
 
   const bool is_screencast_;
   scoped_refptr<RtcVideoCaptureDelegate> delegate_;
   VideoCaptureState state_;
-  base::Time start_time_;
+  bool first_frame_timestamp_valid_;

[Ami GONE FROM CHROMIUM, 2013/09/12 22:40:41]

FWIW, media/base/buffers.h:kNoTimestamp() is a thing that exists, which you
could use as a sentinel value instead of tracking this extra variable.  I don't
really care which you do.

[sheu, 2013/09/13 00:15:19]

That actually makes more sense.  Done.

+  base::TimeDelta first_frame_timestamp_;
 
   DISALLOW_COPY_AND_ASSIGN(RtcVideoCapturer);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_RTC_VIDEO_CAPTURER_H_