FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.

NOPRESUBMIT=true

Committed: https://crrev.com/144dd270560a5a5630414a6ed462e425346cb1eb
Cr-Commit-Position: refs/heads/master@{#13794}

[kwiberg-webrtc, 2016-06-08 12:07:31]

Description was changed from

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of an interface that are only ever exposed
via "create" functions, so the entire class definitions can be put in
an anonymous namespace in the .cc files that defines the "create"
functions.
==========

to

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.
==========

[kwiberg-webrtc, 2016-06-09 08:17:06]

Patchset #1 (id:1) has been deleted

[kwiberg-webrtc, 2016-06-09 08:17:20]

Patchset #1 (id:20001) has been deleted

[kwiberg-webrtc, 2016-08-16 12:00:26]

Description was changed from

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.
==========

to

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.

NOPRESUBMIT=true
==========

[kwiberg-webrtc, 2016-08-16 20:43:49]

kwiberg@webrtc.org changed reviewers:
+ ossu@webrtc.org

[kwiberg-webrtc, 2016-08-16 20:43:49]

I've just moved code around, and changed it as little as possible (since it's
hard to read diffs where code is both moved and modified).

[ossu, 2016-08-17 09:28:24]

lgtm!

[kwiberg-webrtc, 2016-08-17 09:32:21]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-08-17 09:32:27]

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-08-17 09:46:55]

Description was changed from

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.

NOPRESUBMIT=true
==========

to

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.

NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-08-17 09:46:56]

Committed patchset #3 (id:80001)

[commit-bot: I haz the power, 2016-08-17 09:47:02]

Description was changed from

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.

NOPRESUBMIT=true
==========

to

==========
FileRecorderImpl and FilePlayerImpl don't need their own .h and .cc files

They are implementations of interfaces that are only ever exposed
via "create" functions, so the entire class definitions can be put in
anonymous namespaces in the .cc files that defines the "create"
functions.

NOPRESUBMIT=true

Committed: https://crrev.com/144dd270560a5a5630414a6ed462e425346cb1eb
Cr-Commit-Position: refs/heads/master@{#13794}
==========

[commit-bot: I haz the power, 2016-08-17 09:47:03]

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