Don't include abort() cost when timing MSE methods.

Don't include abort() cost when timing MSE methods.

xhr.abort() and sourceBuffer.abort() were transitively included
via onPlaybackStarted().  It's not relevant to the recorded
metrics so measure end time before the onPlaybackStarted()
calls.

BUG=367228
TEST=ran telemetry tests.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266378

[shadi, 2014-04-25 21:00:19]

LGTM 

nit: can you specify xhr.abort() in CL description? Without looking into code,
it can be confused by any video abort event.

[DaleCurtis, 2014-04-25 21:10:29]

Done! Thanks.

[tonyg, 2014-04-25 21:33:13]

lgtm2

[DaleCurtis, 2014-04-25 21:37:35]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2014-04-25 21:59:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/251593008/1

[commit-bot: I haz the power, 2014-04-26 06:29:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-26 06:29:15]

The commit queue went berserk retrying too often for a
seemingly flaky test on builder android_dbg:
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[tonyg, 2014-04-26 13:21:13]

The CQ bit was checked by tonyg@chromium.org

[commit-bot: I haz the power, 2014-04-26 13:21:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/251593008/1

[commit-bot: I haz the power, 2014-04-26 22:08:11]

Change committed as 266378