Merged: Trigger MSE players UMA by user events only

Merged: Trigger MSE players UMA by user events only

This is the second fix for MediaSourcePlayer and MediaCodecPlayer UMA
statistics gathering (first was https://codereview.chromium.org/1458783002).

This CL calls the MediaStatistics::StopAndReport call from
the MediaCodecPlayer's top-level methods Pause(), SeekTo(), Release(),
when the error detected and when the end of stream is detected,
instead of calling it internally when the state machine went to
the stopped state.

By doing this we shall make MediaCodecPlayer behavior to be the
same as the MediaSourcePlayer and do not record an artificial
playback stop on internal event like starvation or resultion
change with the decoder drain.

This CL already eliminates an incorrect stop detection when the
user switches to the fullscreen mode and back while playing.

BUG=407577, 570835
TBR=timav

> Review URL: https://codereview.chromium.org/1537583002

> Cr-Commit-Position: refs/heads/master@{#365967}

Committed: https://chromium.googlesource.com/chromium/src/+/af7e7ed5278fbbef4da9130ca81bcc8e1a184579

[Tima Vaisburd, 2015-12-18 02:21:53]

Description was changed from

==========
Merged: Trigger MSE players UMA by user events only

This is the second fix for MediaSourcePlayer and MediaCodecPlayer UMA
statistics gathering (first was https://codereview.chromium.org/1458783002).

This CL calls the MediaStatistics::StopAndReport call from
the MediaCodecPlayer's top-level methods Pause(), SeekTo(), Release(),
when the error detected and when the end of stream is detected,
instead of calling it internally when the state machine went to
the stopped state.

By doing this we shall make MediaCodecPlayer behavior to be the
same as the MediaSourcePlayer and do not record an artificial
playback stop on internal event like starvation or resultion
change with the decoder drain.

This CL already eliminates an incorrect stop detection when the
user switches to the fullscreen mode and back while playing.

BUG=407577, 570835

TBR=timav

> Review URL: https://codereview.chromium.org/1537583002

> Cr-Commit-Position: refs/heads/master@{#365967}
==========

to

==========
Merged: Trigger MSE players UMA by user events only

This is the second fix for MediaSourcePlayer and MediaCodecPlayer UMA
statistics gathering (first was https://codereview.chromium.org/1458783002).

This CL calls the MediaStatistics::StopAndReport call from
the MediaCodecPlayer's top-level methods Pause(), SeekTo(), Release(),
when the error detected and when the end of stream is detected,
instead of calling it internally when the state machine went to
the stopped state.

By doing this we shall make MediaCodecPlayer behavior to be the
same as the MediaSourcePlayer and do not record an artificial
playback stop on internal event like starvation or resultion
change with the decoder drain.

This CL already eliminates an incorrect stop detection when the
user switches to the fullscreen mode and back while playing.

BUG=407577, 570835

TBR=timav

> Review URL: https://codereview.chromium.org/1537583002

> Cr-Commit-Position: refs/heads/master@{#365967}

Committed:
https://chromium.googlesource.com/chromium/src/+/af7e7ed5278fbbef4da9130ca81b...
==========

[Tima Vaisburd, 2015-12-18 02:21:54]

Committed patchset #1 (id:1) manually as
af7e7ed5278fbbef4da9130ca81bcc8e1a184579.