Remove unused RemoteBitrateEstimator::IncomingPacketFeedbackVector()

Remove unused RemoteBitrateEstimator::IncomingPacketFeedbackVector()

BUG=None

Review-Url: https://codereview.webrtc.org/2721463003
Cr-Commit-Position: refs/heads/master@{#17058}
Committed: https://chromium.googlesource.com/external/webrtc/+/5419ac8c026bdbd4f5690a03af106bc3cb7211dd

[elad.alon_webrtc.org, 2017-02-27 10:45:43]

elad.alon@webrtc.org changed reviewers:
+ michaelt@webrtc.org, minyue@webrtc.org, ossu@webrtc.org, solenberg@webrtc.org,
sprang@webrtc.org, stefan@webrtc.org

[elad.alon_webrtc.org, 2017-02-27 10:45:44]

Ready

[stefan-webrtc, 2017-03-01 22:35:30]

lgtm

[michaelt, 2017-03-02 12:46:26]

Shouldn't we remove this as well ?
https://cs.chromium.org/chromium/src/third_party/webrtc/modules/remote_bitrat...

[elad.alon_webrtc.org, 2017-03-02 13:13:53]

On 2017/03/02 12:46:26, michaelt wrote:
> Shouldn't we remove this as well ?
>
https://cs.chromium.org/chromium/src/third_party/webrtc/modules/remote_bitrat...

Definitely. Thanks for catching.

[elad.alon_webrtc.org, 2017-03-02 13:14:51]

On 2017/03/02 13:13:53, elad.alon wrote:
> On 2017/03/02 12:46:26, michaelt wrote:
> > Shouldn't we remove this as well ?
> >
>
https://cs.chromium.org/chromium/src/third_party/webrtc/modules/remote_bitrat...
> 
> Definitely. Thanks for catching.

I don't see anyone using that class (MockRemoteBitrateEstimator), btw, so we
might want to remove it altogether? What do people think?

[elad.alon_webrtc.org, 2017-03-02 16:51:45]

On 2017/03/02 13:14:51, elad.alon wrote:
> On 2017/03/02 13:13:53, elad.alon wrote:
> > On 2017/03/02 12:46:26, michaelt wrote:
> > > Shouldn't we remove this as well ?
> > >
> >
>
https://cs.chromium.org/chromium/src/third_party/webrtc/modules/remote_bitrat...
> > 
> > Definitely. Thanks for catching.
> 
> I don't see anyone using that class (MockRemoteBitrateEstimator), btw, so we
> might want to remove it altogether? What do people think?

Class removed in https://codereview.webrtc.org/2731523002.

[stefan-webrtc, 2017-03-03 19:36:06]

add BUG=none

lgtm

[elad.alon_webrtc.org, 2017-03-06 09:55:38]

The CQ bit was checked by elad.alon@webrtc.org

[commit-bot: I haz the power, 2017-03-06 09:55:46]

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, 2017-03-06 09:59:56]

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

[commit-bot: I haz the power, 2017-03-06 09:59:57]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/14516)

[elad.alon_webrtc.org, 2017-03-06 10:17:00]

The CQ bit was checked by elad.alon@webrtc.org

[elad.alon_webrtc.org, 2017-03-06 10:17:01]

The patchset sent to the CQ was uploaded after l-g-t-m from stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2721463003/#ps50001 (title:
"Rebased")

[commit-bot: I haz the power, 2017-03-06 10:17:09]

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, 2017-03-06 10:21:58]

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

[commit-bot: I haz the power, 2017-03-06 10:21:59]

Try jobs failed on following builders:
  presubmit on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/14518)

[elad.alon_webrtc.org, 2017-03-06 10:24:49]

Description was changed from

==========
Remove unused RemoteBitrateEstimator::IncomingPacketFeedbackVector()

BUG=
==========

to

==========
Remove unused RemoteBitrateEstimator::IncomingPacketFeedbackVector()

BUG=None
==========

[elad.alon_webrtc.org, 2017-03-06 10:25:04]

The CQ bit was checked by elad.alon@webrtc.org

[commit-bot: I haz the power, 2017-03-06 10:25:09]

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, 2017-03-06 11:11:06]

CQ is committing da patch.

Bot data: {"patchset_id": 50001, "attempt_start_ts": 1488795904979320,
"parent_rev": "dea7f4f46f3d75eb01bad17537c3df2ce9ec4115", "commit_rev":
"5419ac8c026bdbd4f5690a03af106bc3cb7211dd"}

[commit-bot: I haz the power, 2017-03-06 11:11:09]

Description was changed from

==========
Remove unused RemoteBitrateEstimator::IncomingPacketFeedbackVector()

BUG=None
==========

to

==========
Remove unused RemoteBitrateEstimator::IncomingPacketFeedbackVector()

BUG=None

Review-Url: https://codereview.webrtc.org/2721463003
Cr-Commit-Position: refs/heads/master@{#17058}
Committed:
https://chromium.googlesource.com/external/webrtc/+/5419ac8c026bdbd4f5690a03a...
==========

[commit-bot: I haz the power, 2017-03-06 11:11:10]

Committed patchset #4 (id:50001) as
https://chromium.googlesource.com/external/webrtc/+/5419ac8c026bdbd4f5690a03a...