Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up

Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes should be renamed in separate CLs.

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

[hbos_chromium, 2016-09-02 15:33:29]

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

[commit-bot: I haz the power, 2016-09-02 15:33:39]

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

[hbos_chromium, 2016-09-02 15:33:55]

Description was changed from

==========
Renamed WebRTCStats.h to RTCLegacyStats.h and cleaned it up.

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

to

==========
Renamed WebRTCStats.h to RTCLegacyStats.h and cleaned it up.

Clean-up:
- Removed WebRTCStatsType and WebRTCStatsMemberName
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  Renamed WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator

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

[commit-bot: I haz the power, 2016-09-02 15:43:45]

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

[commit-bot: I haz the power, 2016-09-02 15:43:46]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[hbos_chromium, 2016-09-02 15:49:28]

Description was changed from

==========
Renamed WebRTCStats.h to RTCLegacyStats.h and cleaned it up.

Clean-up:
- Removed WebRTCStatsType and WebRTCStatsMemberName
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  Renamed WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator

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

to

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes will be renamed in separate CLs.

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

[hbos_chromium, 2016-09-02 15:55:23]

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

[commit-bot: I haz the power, 2016-09-02 15:55:44]

Dry run: 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-09-02 20:12:43]

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

[commit-bot: I haz the power, 2016-09-02 20:12:44]

Dry run: Exceeded global retry quota

[hbos_chromium, 2016-09-05 09:58:17]

Description was changed from

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes will be renamed in separate CLs.

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

to

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes should be renamed in separate CLs.

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

[hbos_chromium, 2016-09-05 10:03:17]

hbos@chromium.org changed reviewers:
+ mkwst@chromium.org, perkj@chromium.org

[hbos_chromium, 2016-09-05 10:03:17]

Please take a look, mkwst and perkj

[Mike West, 2016-09-05 10:05:44]

LGTM. Will you be removing the legacy files at some point in the future?

[hbos_chromium, 2016-09-05 10:12:15]

PTAL perkj.

On 2016/09/05 10:05:44, Mike West (OOO until 29th) wrote:
> LGTM. Will you be removing the legacy files at some point in the future?

Yes at some point we shall remove the legacy getStats method and will be able to
remove lots of code. But this is far in the future, implementing a new API,
deprecating an old JavaScript API...

[perkj_chrome, 2016-09-05 10:15:49]

On 2016/09/05 10:12:15, hbos_chromium wrote:
> PTAL perkj.
> 
> On 2016/09/05 10:05:44, Mike West (OOO until 29th) wrote:
> > LGTM. Will you be removing the legacy files at some point in the future?
> 
> Yes at some point we shall remove the legacy getStats method and will be able
to
> remove lots of code. But this is far in the future, implementing a new API,
> deprecating an old JavaScript API...

lgtm

[hbos_chromium, 2016-09-05 10:21:45]

The CQ bit was checked by hbos@chromium.org

[commit-bot: I haz the power, 2016-09-05 10:22:01]

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-09-05 11:52:46]

Description was changed from

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes should be renamed in separate CLs.

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

to

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes should be renamed in separate CLs.

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

[commit-bot: I haz the power, 2016-09-05 11:52:47]

Committed patchset #3 (id:40001)

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

Description was changed from

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes should be renamed in separate CLs.

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

to

==========
Renamed WebRTCStats.h to WebRTCLegacyStats.h and cleaned it up.

Changes:
- WebRTCStats.h renamed to WebRTCLegacyStats.h
- Removed WebRTCStatsType and WebRTCStatsMemberName - not used
- Renamed WebRTCStats and WebRTCStatsMemberIterator to
  WebRTCLegacyStats and WebRTCLegacyStatsMemberIterator
- Updated affected code

These classes/enums are used to surface stats from the WebRTC repo to
Blink. They were originally added to support both the old stats and
the new stats which are currently being added to the WebRTC repo.

Because we decided to add the new stats using a new stats collection
API, we'll want to surface the new stats into Blink differently. We
rename these classes/enums as to not confuse the classes relating to
the new stats ("RTCStats") with the classes relating to the old stats
("RTCLegacyStatsReport").

More classes should be renamed in separate CLs.

BUG=chromium:627816

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

[commit-bot: I haz the power, 2016-09-05 11:54:02]

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