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

Unified Diff: trunk/src/content/renderer/media/rtc_video_capturer.cc

Issue 13882004: Revert 192670 "Use NTP time for video frame timestamp." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 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
« no previous file with comments | « trunk/src/content/renderer/media/rtc_video_capturer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/content/renderer/media/rtc_video_capturer.cc
===================================================================
--- trunk/src/content/renderer/media/rtc_video_capturer.cc (revision 193357)
+++ trunk/src/content/renderer/media/rtc_video_capturer.cc (working copy)
@@ -5,7 +5,6 @@
#include "content/renderer/media/rtc_video_capturer.h"
#include "base/bind.h"
-#include "base/debug/trace_event.h"
namespace content {
@@ -16,17 +15,6 @@
: is_screencast_(is_screencast),
delegate_(new RtcVideoCaptureDelegate(id, vc_manager)),
state_(VIDEO_CAPTURE_STATE_STOPPED) {
- base::Time::Exploded exploded = {};
- exploded.year = 1900;
- exploded.month = 1;
- exploded.day_of_week = 0;
- exploded.day_of_month = 1;
- exploded.hour = 0;
- exploded.minute = 0;
- exploded.second = 0;
- exploded.millisecond = 0;
- DCHECK(exploded.HasValidValues());
- ntp_epoch_ = base::Time::FromUTCExploded(exploded);
}
RtcVideoCapturer::~RtcVideoCapturer() {
@@ -112,20 +100,11 @@
// cricket::CapturedFrame time is in nanoseconds.
frame.elapsed_time = (buf.timestamp - start_time_).InMicroseconds() *
base::Time::kNanosecondsPerMicrosecond;
- // Timestamp in NTP time (seconds since 0:00 UTC 1 January 1900) in ms.
- frame.time_stamp = (buf.timestamp - ntp_epoch_).InMilliseconds();
+ frame.time_stamp = frame.elapsed_time;
frame.data = buf.memory_pointer;
frame.pixel_height = 1;
frame.pixel_width = 1;
- TRACE_EVENT_INSTANT2("rtc_video_capturer",
- "OnFrameCaptured",
- TRACE_EVENT_SCOPE_THREAD,
- "elapsed time",
- frame.elapsed_time,
- "timestamp",
- frame.time_stamp);
-
// This signals to libJingle that a new VideoFrame is available.
// libJingle have no assumptions on what thread this signal come from.
SignalFrameCaptured(this, &frame);
« no previous file with comments | « trunk/src/content/renderer/media/rtc_video_capturer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698