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

Issue 183063011: Revert 255166 "Cast: Using a min filter to compute time offset" (Closed)

Created:
6 years, 9 months ago by dvh
Modified:
6 years, 9 months ago
Reviewers:
mikhal1
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 255166 "Cast: Using a min filter to compute time offset" > Cast: Using a min filter to compute time offset > > Modifying the time offset filter to take to minimum of the first 10 values. > Once ten offset values were seen, the value will be constant for the entire session. > This may be justified by the fact we are seeking the offset between sender and > receiver without accounting the network jitter. > > Review URL: https://codereview.chromium.org/186043003 TBR=mikhal@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255173

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -17 lines) Patch
M trunk/src/media/cast/video_receiver/video_receiver.h View 1 chunk +0 lines, -1 line 0 comments Download
M trunk/src/media/cast/video_receiver/video_receiver.cc View 5 chunks +6 lines, -16 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
dvh
6 years, 9 months ago (2014-03-05 23:25:59 UTC) #1
dvh
6 years, 9 months ago (2014-03-05 23:26:32 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r255173 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698