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

Unified Diff: tools/perf/page_sets/media_cns_cases.json

Issue 217033007: Add seek tests to media constrained network benchmark. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 6 years, 9 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 | tools/perf/page_sets/tough_video_cases/video.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/page_sets/media_cns_cases.json
diff --git a/tools/perf/page_sets/media_cns_cases.json b/tools/perf/page_sets/media_cns_cases.json
index 7a54d2cae6cdf3d88371d90879c5a8203436fd2a..1cf20a7bfc257f17b146fa1d029dc1f1e36e3acc 100644
--- a/tools/perf/page_sets/media_cns_cases.json
+++ b/tools/perf/page_sets/media_cns_cases.json
@@ -8,6 +8,27 @@
"wait_for_ended": true
}
],
+ "seek_before_and_after_playhead": [
+ { "action": "play",
+ "wait_for_playing": true,
+ "wait_for_ended": false
+ },
+ { "_comment": "Wait for 1 second so that we know the play-head is at ~1s.",
+ "action": "wait", "seconds": 1
+ },
+ { "_comment": "Seek to before the play-head location.",
+ "action": "seek",
+ "wait_for_seeked": true,
+ "seek_time": "0.5",
+ "seek_label": "seek_warm"
+ },
+ { "_comment": "Seek to after the play-head location.",
+ "action": "seek",
+ "wait_for_seeked": true,
+ "seek_time": 15,
+ "seek_label": "seek_cold"
+ }
+ ],
"pages": [
{
"url": "file://tough_video_cases/video.html?id=no_constraints_webm&src=tulip2.webm&net=none"
@@ -71,6 +92,48 @@
},
{
"url": "file://tough_video_cases/video.html?id=wifi_m4a&src=tulip2.m4a&type=audio&net=wifi"
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_mp3&src=tulip2.mp3&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_m4a&src=tulip2.m4a&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_ogg&src=tulip2.ogg&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_wav&src=tulip2.wav&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_mp4&src=tulip2.mp4&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_ogv&src=tulip2.ogv&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
+ },
+ {
+ "url": "file://tough_video_cases/video.html?id=wifi_webm&src=tulip2.webm&type=audio&net=wifi",
+ "skip_basic_metrics": true,
+ "add_browser_metrics": false,
+ "media_metrics": {"action": "seek_before_and_after_playhead"}
}
]
}
« no previous file with comments | « no previous file | tools/perf/page_sets/tough_video_cases/video.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698