Delete dead code VP8EncoderImpl::UpdateCodecFrameSize.

Delete dead code VP8EncoderImpl::UpdateCodecFrameSize.

BUG=webrtc:7349
Review-Url: https://codereview.webrtc.org/2855473006
Cr-Commit-Position: refs/heads/master@{#17969}
Committed: https://chromium.googlesource.com/external/webrtc/+/fc31a43e3652df4c8e03bfff89858c4858d79b89

[brandtr, 2017-05-02 07:32:35]

brandtr@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2017-05-02 08:12:33]

lgtm if you remove it from the header too. :)

https://codereview.webrtc.org/2855473006/diff/1/webrtc/modules/video_coding/c...
File webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc (left):

https://codereview.webrtc.org/2855473006/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc:773: int
VP8EncoderImpl::UpdateCodecFrameSize(int width, int height) {
This should also be removed in the header.

[brandtr, 2017-05-02 08:38:11]

Thanks. Working late?

https://codereview.webrtc.org/2855473006/diff/1/webrtc/modules/video_coding/c...
File webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc (left):

https://codereview.webrtc.org/2855473006/diff/1/webrtc/modules/video_coding/c...
webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc:773: int
VP8EncoderImpl::UpdateCodecFrameSize(int width, int height) {
On 2017/05/02 08:12:33, pbos-webrtc wrote:
> This should also be removed in the header.

Right, that sounds like a good idea... :)

[pbos-webrtc, 2017-05-02 08:39:35]

On 2017/05/02 08:38:11, brandtr wrote:
> Thanks. Working late?

Nah, I'm in MUC right now. :)

>
https://codereview.webrtc.org/2855473006/diff/1/webrtc/modules/video_coding/c...
> File webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc (left):
> 
>
https://codereview.webrtc.org/2855473006/diff/1/webrtc/modules/video_coding/c...
> webrtc/modules/video_coding/codecs/vp8/vp8_impl.cc:773: int
> VP8EncoderImpl::UpdateCodecFrameSize(int width, int height) {
> On 2017/05/02 08:12:33, pbos-webrtc wrote:
> > This should also be removed in the header.
> 
> Right, that sounds like a good idea... :)

[brandtr, 2017-05-02 08:51:42]

Description was changed from

==========
Delete dead code VP8EncoderImpl::UpdateCodecFrameSize.

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

to

==========
Delete dead code VP8EncoderImpl::UpdateCodecFrameSize.

BUG=webrtc:7349
==========

[brandtr, 2017-05-02 08:51:49]

The CQ bit was checked by brandtr@webrtc.org

[brandtr, 2017-05-02 08:51:50]

The patchset sent to the CQ was uploaded after l-g-t-m from pbos@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2855473006/#ps20001 (title:
"Remove from header too.")

[commit-bot: I haz the power, 2017-05-02 08:52:00]

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-05-02 09:21:17]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493715109721060,
"parent_rev": "bf2c049a12302141725af0a7407022908fda7efb", "commit_rev":
"fc31a43e3652df4c8e03bfff89858c4858d79b89"}

[commit-bot: I haz the power, 2017-05-02 09:21:21]

Description was changed from

==========
Delete dead code VP8EncoderImpl::UpdateCodecFrameSize.

BUG=webrtc:7349
==========

to

==========
Delete dead code VP8EncoderImpl::UpdateCodecFrameSize.

BUG=webrtc:7349

Review-Url: https://codereview.webrtc.org/2855473006
Cr-Commit-Position: refs/heads/master@{#17969}
Committed:
https://chromium.googlesource.com/external/webrtc/+/fc31a43e3652df4c8e03bfff8...
==========

[commit-bot: I haz the power, 2017-05-02 09:21:22]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/fc31a43e3652df4c8e03bfff8...