Only update codec type histogram if lifetime is long enough (10 sec).

Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=

Committed: https://crrev.com/4374a09f9bfb24e21ba1510fd6dbe40defada6c3
Cr-Commit-Position: refs/heads/master@{#13537}

[åsapersson, 2016-07-08 16:12:48]

Description was changed from

==========
Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=
==========

to

==========
Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=
==========

[åsapersson, 2016-07-08 16:12:49]

asapersson@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2016-07-10 17:43:03]

https://codereview.webrtc.org/2136533002/diff/40001/webrtc/video/end_to_end_t...
File webrtc/video/end_to_end_tests.cc (right):

https://codereview.webrtc.org/2136533002/diff/40001/webrtc/video/end_to_end_t...
webrtc/video/end_to_end_tests.cc:2133: EXPECT_EQ(2,
metrics::NumSamples("WebRTC.Call.LifetimeInSeconds"));
This sounds very flaky to me, deleting the call etc. might not be instant. Maybe
EXPECT_GE?

https://codereview.webrtc.org/2136533002/diff/40001/webrtc/video/end_to_end_t...
webrtc/video/end_to_end_tests.cc:2140: EXPECT_EQ(1,
metrics::NumSamples("WebRTC.Video.SendStreamLifetimeInSeconds"));
Same here?

[åsapersson, 2016-07-25 07:54:04]

https://codereview.webrtc.org/2136533002/diff/40001/webrtc/video/end_to_end_t...
File webrtc/video/end_to_end_tests.cc (right):

https://codereview.webrtc.org/2136533002/diff/40001/webrtc/video/end_to_end_t...
webrtc/video/end_to_end_tests.cc:2133: EXPECT_EQ(2,
metrics::NumSamples("WebRTC.Call.LifetimeInSeconds"));
On 2016/07/10 17:43:03, pbos-webrtc wrote:
> This sounds very flaky to me, deleting the call etc. might not be instant.
Maybe
> EXPECT_GE?

Checks that the metric is reported twice, once for the sender_call and once for
the receiver_call.

https://codereview.webrtc.org/2136533002/diff/40001/webrtc/video/end_to_end_t...
webrtc/video/end_to_end_tests.cc:2140: EXPECT_EQ(1,
metrics::NumSamples("WebRTC.Video.SendStreamLifetimeInSeconds"));
On 2016/07/10 17:43:03, pbos-webrtc wrote:
> Same here?

Reported once for the send/receive stream.

[pbos-webrtc, 2016-07-25 09:26:22]

lgtm, thanks

[åsapersson, 2016-07-25 11:35:54]

The CQ bit was checked by asapersson@webrtc.org

[commit-bot: I haz the power, 2016-07-25 11:35:57]

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

[commit-bot: I haz the power, 2016-07-25 11:40:26]

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

[commit-bot: I haz the power, 2016-07-25 11:40:27]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/6954)

[åsapersson, 2016-07-25 11:45:45]

asapersson@webrtc.org changed reviewers:
+ magjed@webrtc.org

[åsapersson, 2016-07-25 11:46:16]

Magnus, can you have a look at
webrtc/api/androidtests/src/org/webrtc/PeerConnectionTest.java

[magjed_webrtc, 2016-07-26 13:14:07]

webrtc/api/androidtests/src/org/webrtc/PeerConnectionTest.java lgtm

[åsapersson, 2016-07-27 05:55:28]

The CQ bit was checked by asapersson@webrtc.org

[commit-bot: I haz the power, 2016-07-27 05:55:34]

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

[commit-bot: I haz the power, 2016-07-27 07:39:12]

Description was changed from

==========
Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=
==========

to

==========
Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=
==========

[commit-bot: I haz the power, 2016-07-27 07:39:13]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-07-27 07:39:22]

Description was changed from

==========
Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=
==========

to

==========
Only update codec type histogram if lifetime is long enough (10 sec).
Add metrics for Call/VideoSendStream/VideoReceiveStream lifetime.

BUG=

Committed: https://crrev.com/4374a09f9bfb24e21ba1510fd6dbe40defada6c3
Cr-Commit-Position: refs/heads/master@{#13537}
==========

[commit-bot: I haz the power, 2016-07-27 07:39:23]

Patchset 4 (id:??) landed as
https://crrev.com/4374a09f9bfb24e21ba1510fd6dbe40defada6c3
Cr-Commit-Position: refs/heads/master@{#13537}