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

Unified Diff: tools/metrics/histograms/histograms.xml

Issue 16189009: Reduce wait time for Windows in AudioOutputController:WaitTillDataReady() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove new histogram as asked Created 7 years, 7 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
Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index fdb429b3ad5bbc019babb89278af80c79c6cbfb5..267874de413dd63fa7c69b0d1cedad5101b7b86c 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -2186,13 +2186,6 @@ other types of suffix sets.
<summary>Measures the time taken for AudioOutputController::</summary>
</histogram>
-<histogram name="Media.AudioOutputControllerDataNotReady" units="ms">
Alexei Svitkine (slow) 2013/05/29 21:25:07 We don't usually remove histograms from this file,
- <summary>
- Time spent waiting in AudioOutputController::WaitTillDataReady() if the data
- was not initially available.
- </summary>
-</histogram>
-
<histogram name="Media.AudioRendererEvents" enum="AudioRendererEvents">
<summary>Captures statistics for various AudioRendererImpl events.</summary>
</histogram>
@@ -14762,6 +14755,9 @@ other types of suffix sets.
<group name="PlayTime"
label="Measures the time taken for DoPlay(). Technically only the
worker method AudioOutputController::PollAndStartIfDataReady()."/>
+ <group name="WaitForDataTime"
+ label="Measures the time waited in WaitTillDataReady() if data was not
+ initially available"/>
<affected-histogram name="Media.AudioOutputController"/>
</fieldtrial>
« media/audio/audio_output_controller.cc ('K') | « media/audio/audio_output_controller.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698