RTCStats operator== bugfix

RTCStats operator== bugfix

a == b would return true if a.member is defined even if b.member is
undefined if their values were equal. We would say that b does not have
a value in that case but its value_ member would still be initialized to
something that is being compared to. Bugfix makes sure not to do value
comparison in this case if b is undefined.

BUG=chromium:627816
Committed: https://crrev.com/28747969cd2530114759e9d61ee711b887189d72
Cr-Commit-Position: refs/heads/master@{#15172}

[hbos, 2016-11-21 14:32:04]

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

[commit-bot: I haz the power, 2016-11-21 14:32:15]

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

[hbos, 2016-11-21 14:32:54]

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

[hbos, 2016-11-21 14:32:54]

Please take a look, guidou

[commit-bot: I haz the power, 2016-11-21 15:02:21]

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

[commit-bot: I haz the power, 2016-11-21 15:02:23]

Dry run: This issue passed the CQ dry run.

[guidou, 2016-11-21 17:15:42]

The CQ bit was checked by guidou@webrtc.org

[commit-bot: I haz the power, 2016-11-21 17:15:45]

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

[guidou, 2016-11-21 17:15:46]

The CQ bit was unchecked by guidou@webrtc.org

[commit-bot: I haz the power, 2016-11-21 17:15:46]

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

[commit-bot: I haz the power, 2016-11-21 17:15:47]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-webrtc-committers".
Note that this has nothing to do with OWNERS files.

[guidou, 2016-11-21 17:15:57]

The CQ bit was checked by guidou@webrtc.org

[guidou, 2016-11-21 17:15:58]

lgtm

[commit-bot: I haz the power, 2016-11-21 17:16:04]

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-11-21 17:17:41]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479748557568720, "parent_rev":
"5aff43493d765fe86407eb22b253aed463516358", "commit_rev":
"eacfe33256f70525ac8950a61248dd54b0b14b2e"}

[commit-bot: I haz the power, 2016-11-21 17:17:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-21 17:18:00]

Description was changed from

==========
RTCStats operator== bugfix

a == b would return true if a.member is defined even if b.member is
undefined if their values were equal. We would say that b does not have
a value in that case but its value_ member would still be initialized to
something that is being compared to. Bugfix makes sure not to do value
comparison in this case if b is undefined.

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

to

==========
RTCStats operator== bugfix

a == b would return true if a.member is defined even if b.member is
undefined if their values were equal. We would say that b does not have
a value in that case but its value_ member would still be initialized to
something that is being compared to. Bugfix makes sure not to do value
comparison in this case if b is undefined.

BUG=chromium:627816

Committed: https://crrev.com/28747969cd2530114759e9d61ee711b887189d72
Cr-Commit-Position: refs/heads/master@{#15172}
==========

[commit-bot: I haz the power, 2016-11-21 17:18:01]

Patchset 1 (id:??) landed as
https://crrev.com/28747969cd2530114759e9d61ee711b887189d72
Cr-Commit-Position: refs/heads/master@{#15172}

[hta-webrtc, 2016-11-21 17:41:54]

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

[hta-webrtc, 2016-11-21 17:41:54]

lgtm

good catch.