Measure usage of RTCSessionDescription's type and sdp attributes

Measure usage of RTCSessionDescription's type and sdp attributes

Also measure missing members in the init dict, that is how the members
would end up as null, and will show how often adding required would
throw a TypeError due to missing type.

BUG=662898
R=guidou@chromium.org

Committed: https://crrev.com/3fdd951cdccbd48de88767e6156bad40ff52738d
Cr-Commit-Position: refs/heads/master@{#430662}

[foolip, 2016-11-08 11:01:12]

The CQ bit was checked by foolip@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-08 11:01:54]

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

[Guido Urdaneta, 2016-11-08 11:06:29]

lgtm

[foolip, 2016-11-08 11:17:54]

Description was changed from

==========
Measure usage of RTCSessionDescription's type and sdp attributes

BUG=662898
R=guidou@chromium.org
==========

to

==========
Measure usage of RTCSessionDescription's type and sdp attributes

Also measure missing members in the init dict, that is how the members
would end up as null, and will show how often adding required would
throw a TypeError due to missing type.

BUG=662898
R=guidou@chromium.org
==========

[foolip, 2016-11-08 11:18:23]

On 2016/11/08 11:06:29, Guido Urdaneta wrote:
> lgtm

Sorry, I realized there was something more to measure. Added an "Also ..." to
the description, can you take another look?

[Guido Urdaneta, 2016-11-08 11:19:43]

lgtm

[Mike West, 2016-11-08 11:31:19]

mkwst@chromium.org changed reviewers:
+ mkwst@chromium.org

[Mike West, 2016-11-08 11:31:19]

histograms.xml LGTM

[foolip, 2016-11-08 11:47:44]

The CQ bit was checked by foolip@chromium.org

[foolip, 2016-11-08 11:47:44]

The patchset sent to the CQ was uploaded after l-g-t-m from guidou@chromium.org,
mkwst@chromium.org
Link to the patchset: https://codereview.chromium.org/2490543002/#ps40001
(title: "add missing "be"")

[commit-bot: I haz the power, 2016-11-08 11:48:04]

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

[commit-bot: I haz the power, 2016-11-08 16:28:19]

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

[commit-bot: I haz the power, 2016-11-08 16:28:20]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[foolip, 2016-11-08 16:33:44]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-11-08 16:34:08]

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

[commit-bot: I haz the power, 2016-11-08 18:29:56]

Description was changed from

==========
Measure usage of RTCSessionDescription's type and sdp attributes

Also measure missing members in the init dict, that is how the members
would end up as null, and will show how often adding required would
throw a TypeError due to missing type.

BUG=662898
R=guidou@chromium.org
==========

to

==========
Measure usage of RTCSessionDescription's type and sdp attributes

Also measure missing members in the init dict, that is how the members
would end up as null, and will show how often adding required would
throw a TypeError due to missing type.

BUG=662898
R=guidou@chromium.org
==========

[commit-bot: I haz the power, 2016-11-08 18:29:57]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-08 18:38:50]

Description was changed from

==========
Measure usage of RTCSessionDescription's type and sdp attributes

Also measure missing members in the init dict, that is how the members
would end up as null, and will show how often adding required would
throw a TypeError due to missing type.

BUG=662898
R=guidou@chromium.org
==========

to

==========
Measure usage of RTCSessionDescription's type and sdp attributes

Also measure missing members in the init dict, that is how the members
would end up as null, and will show how often adding required would
throw a TypeError due to missing type.

BUG=662898
R=guidou@chromium.org

Committed: https://crrev.com/3fdd951cdccbd48de88767e6156bad40ff52738d
Cr-Commit-Position: refs/heads/master@{#430662}
==========

[commit-bot: I haz the power, 2016-11-08 18:38:51]

Patchset 3 (id:??) landed as
https://crrev.com/3fdd951cdccbd48de88767e6156bad40ff52738d
Cr-Commit-Position: refs/heads/master@{#430662}