FilePlayer: Remove unused default values for arguments

FilePlayer: Remove unused default values for arguments

The functions in question were virtual, so we would've wanted to get
rid of the default values even if callers had relied on them.

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

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

Patchset #1 (id:1) has been deleted

[kwiberg-webrtc, 2016-08-17 10:00:28]

Patchset #1 (id:20001) has been deleted

[kwiberg-webrtc, 2016-08-17 10:18:15]

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

[ossu, 2016-08-17 11:59:22]

Alright, looks sensible!
What's the ultimate goal?
lgtm

[kwiberg-webrtc, 2016-08-17 13:13:53]

On 2016/08/17 11:59:22, ossu wrote:
> Alright, looks sensible!
> What's the ultimate goal?

To get rid of the most glaring style guide violations in these files. I did
almost all the work before I went on vacation, so I don't recall why anymore,
but I'm a sensible person so I assume I had a good reason.

[ossu, 2016-08-17 13:20:17]

On 2016/08/17 13:13:53, kwiberg-webrtc wrote:
> On 2016/08/17 11:59:22, ossu wrote:
> > Alright, looks sensible!
> > What's the ultimate goal?
> 
> To get rid of the most glaring style guide violations in these files. I did
> almost all the work before I went on vacation, so I don't recall why anymore,
> but I'm a sensible person so I assume I had a good reason.

Good enough for me! :)

[kwiberg-webrtc, 2016-08-17 16:14:41]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-08-17 16:14:48]

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 16:27:09]

Committed patchset #1 (id:40001)

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

Description was changed from

==========
FilePlayer: Remove unused default values for arguments

The functions in question were virtual, so we would've wanted to get
rid of the default values even if callers had relied on them.
==========

to

==========
FilePlayer: Remove unused default values for arguments

The functions in question were virtual, so we would've wanted to get
rid of the default values even if callers had relied on them.

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

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

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