RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.

RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
NOTRY=True

Committed: https://crrev.com/b20f38792ee87d76a0c4a47e05fbd9a8a8789c29
Cr-Commit-Position: refs/heads/master@{#14509}

[hbos, 2016-10-04 09:11:07]

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

[hbos, 2016-10-04 09:11:08]

Please take a look hta

[hta-webrtc, 2016-10-04 13:53:45]

lgtm

https://codereview.webrtc.org/2387343002/diff/1/webrtc/stats/test/rtcteststats.h
File webrtc/stats/test/rtcteststats.h (right):

https://codereview.webrtc.org/2387343002/diff/1/webrtc/stats/test/rtcteststat...
webrtc/stats/test/rtcteststats.h:41: RTCStatsMember<bool>                    
m_bool;
Is this clang-format compliant?

Clang-format may not be wiser than me, but it's uniform.

[hbos, 2016-10-04 20:24:20]

https://codereview.webrtc.org/2387343002/diff/1/webrtc/stats/test/rtcteststats.h
File webrtc/stats/test/rtcteststats.h (right):

https://codereview.webrtc.org/2387343002/diff/1/webrtc/stats/test/rtcteststat...
webrtc/stats/test/rtcteststats.h:41: RTCStatsMember<bool>                    
m_bool;
On 2016/10/04 13:53:45, hta-webrtc wrote:
> Is this clang-format compliant?
> 
> Clang-format may not be wiser than me, but it's uniform.

It's not, I'll change it back to one space.

[hbos, 2016-10-04 20:24:25]

The CQ bit was checked by hbos@webrtc.org

[hbos, 2016-10-04 20:24:25]

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/2387343002/#ps20001 (title:
"Removed spaces between RTCStatsMember and variable name")

[commit-bot: I haz the power, 2016-10-04 20:24:33]

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

[hbos, 2016-10-04 21:23:55]

The CQ bit was unchecked by hbos@webrtc.org

[hbos, 2016-10-04 21:24:46]

Can't land with the android_arm64_rel queuing longer than timeout, but all other
bots are green, landing with NOTRY

[hbos, 2016-10-04 21:25:00]

Description was changed from

==========
RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
==========

to

==========
RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
NOTRY=True
==========

[hbos, 2016-10-04 21:25:13]

The CQ bit was checked by hbos@webrtc.org

[commit-bot: I haz the power, 2016-10-04 21:25:20]

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-10-04 21:37:13]

Description was changed from

==========
RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
NOTRY=True
==========

to

==========
RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
NOTRY=True
==========

[commit-bot: I haz the power, 2016-10-04 21:37:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-04 21:37:19]

Description was changed from

==========
RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
NOTRY=True
==========

to

==========
RTCStatsMember<bool> and RTCStatsMember<std::vector<bool>> added.
RTCStatsMemberInterface::Type's kBool and kSequenceBool.

This means that RTCStats-derived classes ("RTCStats-derived
dictionaries"[1]) can contain boolean and sequence of boolean members.

[1] https://w3c.github.io/webrtc-stats/

BUG=chromium:627816
NOTRY=True

Committed: https://crrev.com/b20f38792ee87d76a0c4a47e05fbd9a8a8789c29
Cr-Commit-Position: refs/heads/master@{#14509}
==========

[commit-bot: I haz the power, 2016-10-04 21:37:20]

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