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

Unified Diff: content/browser/media/webrtc_internals.cc

Issue 428133005: Fix a time conversion bug on webrtc-internals. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 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 | « no previous file | content/browser/resources/media/peer_connection_update_table.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/media/webrtc_internals.cc
diff --git a/content/browser/media/webrtc_internals.cc b/content/browser/media/webrtc_internals.cc
index 03afc1103d3ed282d457b190b7a7016a687acd86..09f0e5a0ed42f3d5ef16820f7c2afbf3bb612297 100644
--- a/content/browser/media/webrtc_internals.cc
+++ b/content/browser/media/webrtc_internals.cc
@@ -137,8 +137,8 @@ void WebRTCInternals::OnUpdatePeerConnection(
if (!log_entry)
return;
- int64 milliseconds = (base::Time::Now() - base::Time()).InMilliseconds();
- string time = base::Int64ToString(milliseconds);
+ double epoch_time = base::Time::Now().ToJsTime();
+ string time = base::DoubleToString(epoch_time);
log_entry->SetString("time", time);
log_entry->SetString("type", type);
log_entry->SetString("value", value);
« no previous file with comments | « no previous file | content/browser/resources/media/peer_connection_update_table.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698