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

Unified Diff: media/base/pipeline.cc

Issue 10832334: use type std::string instead of integer for MediaLogEvent::TOTAL_BYTES_SET (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 4 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 | « media/base/media_log.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/pipeline.cc
diff --git a/media/base/pipeline.cc b/media/base/pipeline.cc
index 55f8a408f3d392aca5e1065a93954fad5fd49c53..8e9257035cd13e0736e7475d64c754d6a9861987 100644
--- a/media/base/pipeline.cc
+++ b/media/base/pipeline.cc
@@ -13,6 +13,7 @@
#include "base/metrics/histogram.h"
#include "base/message_loop.h"
#include "base/stl_util.h"
+#include "base/string_number_conversions.h"
#include "base/string_util.h"
#include "base/synchronization/condition_variable.h"
#include "media/base/audio_decoder.h"
@@ -390,8 +391,9 @@ void Pipeline::SetDuration(TimeDelta duration) {
void Pipeline::SetTotalBytes(int64 total_bytes) {
DCHECK(IsRunning());
media_log_->AddEvent(
- media_log_->CreateIntegerEvent(
- MediaLogEvent::TOTAL_BYTES_SET, "total_bytes", total_bytes));
+ media_log_->CreateStringEvent(
+ MediaLogEvent::TOTAL_BYTES_SET, "total_bytes",
+ base::Int64ToString(total_bytes)));
int64 total_mbytes = total_bytes >> 20;
if (total_mbytes > kint32max)
total_mbytes = kint32max;
« no previous file with comments | « media/base/media_log.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698