RTCMediaStreamTrackStats.framesDecoded collected.

RTCMediaStreamTrackStats.framesDecoded collected.

According to spec:
https://w3c.github.io/webrtc-stats/#dom-rtcmediastreamtrackstats-framesdecoded

BUG=webrtc:6757, chromium:659137, chromium:627816
Review-Url: https://codereview.webrtc.org/2642713004
Cr-Commit-Position: refs/heads/master@{#16192}
Committed: https://chromium.googlesource.com/external/webrtc/+/f64941f1a5a18cf47152c5b938d5b4264768c6de

[hbos, 2017-01-19 15:04:38]

The CQ bit was checked by hbos@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-19 15:04:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[hbos, 2017-01-19 15:05:34]

hbos@webrtc.org changed reviewers:
+ hta@webrtc.org

[hbos, 2017-01-19 15:05:35]

Please take a look, hta.

[commit-bot: I haz the power, 2017-01-19 17:09:03]

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

[commit-bot: I haz the power, 2017-01-19 17:09:05]

Dry run: Try jobs failed on following builders:
  linux_msan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/16878)

[hta-webrtc, 2017-01-20 09:41:36]

lgtm

https://codereview.webrtc.org/2642713004/diff/1/webrtc/api/rtcstatscollector_...
File webrtc/api/rtcstatscollector_unittest.cc (right):

https://codereview.webrtc.org/2642713004/diff/1/webrtc/api/rtcstatscollector_...
webrtc/api/rtcstatscollector_unittest.cc:1637:
expected_remote_video_track_ssrc4.frames_decoded = 0;
If we expect in the future that two SSRCs may be associated with an incoming
track, we should maybe add a test for it now - see that it behaves as expected
(I don't care if it's a DCHECK, a sum or a number that is less than the max - as
long as it's the behavior you expect).

If we can't do it now, just leave it.

[hbos, 2017-01-20 14:42:54]

The CQ bit was checked by hbos@webrtc.org

[hbos, 2017-01-20 14:42:54]

The patchset sent to the CQ was uploaded after l-g-t-m from hta@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2642713004/#ps20001 (title:
"Rebase with master")

[commit-bot: I haz the power, 2017-01-20 14:43:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[hbos, 2017-01-20 14:45:48]

https://codereview.webrtc.org/2642713004/diff/1/webrtc/api/rtcstatscollector_...
File webrtc/api/rtcstatscollector_unittest.cc (right):

https://codereview.webrtc.org/2642713004/diff/1/webrtc/api/rtcstatscollector_...
webrtc/api/rtcstatscollector_unittest.cc:1637:
expected_remote_video_track_ssrc4.frames_decoded = 0;
On 2017/01/20 09:41:36, hta-webrtc wrote:
> If we expect in the future that two SSRCs may be associated with an incoming
> track, we should maybe add a test for it now - see that it behaves as expected
> (I don't care if it's a DCHECK, a sum or a number that is less than the max -
as
> long as it's the behavior you expect).
> 
> If we can't do it now, just leave it.

Can't do it now, the assumption about only having one receiver info per remote
track is DCHECKed, testing that would crash.

Unless you mean multiple SSRCs for the same receiver, that is possible, but only
the first one is used as an ID, the rest don't matter. (I don't fully understand
what multiple SSRCs in the same "group" like that means but it doesn't affect
this AFAIK.)

[commit-bot: I haz the power, 2017-01-20 15:39:10]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484923374209420,
"parent_rev": "aea1a017edcad97c63c80d21973c7bf8522f5247", "commit_rev":
"f64941f1a5a18cf47152c5b938d5b4264768c6de"}

[commit-bot: I haz the power, 2017-01-20 15:39:12]

Description was changed from

==========
RTCMediaStreamTrackStats.framesDecoded collected.

According to spec:
https://w3c.github.io/webrtc-stats/#dom-rtcmediastreamtrackstats-framesdecoded

BUG=webrtc:6757, chromium:659137, chromium:627816
==========

to

==========
RTCMediaStreamTrackStats.framesDecoded collected.

According to spec:
https://w3c.github.io/webrtc-stats/#dom-rtcmediastreamtrackstats-framesdecoded

BUG=webrtc:6757, chromium:659137, chromium:627816

Review-Url: https://codereview.webrtc.org/2642713004
Cr-Commit-Position: refs/heads/master@{#16192}
Committed:
https://chromium.googlesource.com/external/webrtc/+/f64941f1a5a18cf47152c5b93...
==========

[commit-bot: I haz the power, 2017-01-20 15:39:13]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/f64941f1a5a18cf47152c5b93...