delete redundant members in ViEEncoder

delete redundant members in ViEEncoder

This information is already available in another member, and
storing it in more places only creates more opportunities
for bugs.

BUG=None

Review-Url: https://codereview.webrtc.org/2613713002
Cr-Commit-Position: refs/heads/master@{#16094}
Committed: https://chromium.googlesource.com/external/webrtc/+/93f16d74fc1249d574ae3504945985970a10975c

[kthelgason, 2017-01-04 12:58:21]

The CQ bit was checked by kthelgason@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 12:58:26]

Dry run: 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-01-04 13:45:00]

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

[commit-bot: I haz the power, 2017-01-04 13:45:01]

Dry run: This issue passed the CQ dry run.

[kthelgason, 2017-01-10 09:17:42]

kthelgason@webrtc.org changed reviewers:
+ perkj@webrtc.org

[kthelgason, 2017-01-10 09:17:43]

Please take a look when you have time. A small cleanup.

[perkj_webrtc, 2017-01-11 08:32:51]

lgtm

https://codereview.webrtc.org/2613713002/diff/10003/webrtc/video/vie_encoder.h
File webrtc/video/vie_encoder.h (right):

https://codereview.webrtc.org/2613713002/diff/10003/webrtc/video/vie_encoder....
webrtc/video/vie_encoder.h:145: int GetPixelCount() { return width * height; }
const

[kthelgason, 2017-01-11 09:06:58]

The CQ bit was checked by kthelgason@webrtc.org

[kthelgason, 2017-01-11 09:06:59]

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

[commit-bot: I haz the power, 2017-01-11 09:07:11]

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-01-11 09:11:00]

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

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

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

[kthelgason, 2017-01-11 10:49:27]

kthelgason@webrtc.org changed reviewers:
+ stefan@webrtc.org

[kthelgason, 2017-01-11 10:49:28]

+stefan for owner LGTM

[kthelgason, 2017-01-16 10:08:14]

Description was changed from

==========
delete redundant members in ViEEncoder

This information is already available in another member, and
storing it in more places only creates more opportunities
for bugs.

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

to

==========
delete redundant members in ViEEncoder

This information is already available in another member, and
storing it in more places only creates more opportunities
for bugs.

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

[kthelgason, 2017-01-16 10:08:15]

kthelgason@webrtc.org changed reviewers:
+ sprang@webrtc.org
- stefan@webrtc.org

[kthelgason, 2017-01-16 10:08:37]

+sprang@ for owner lgtm

[sprang_webrtc, 2017-01-16 10:12:19]

lgtm with nit

https://codereview.webrtc.org/2613713002/diff/50001/webrtc/video/vie_encoder.h
File webrtc/video/vie_encoder.h (right):

https://codereview.webrtc.org/2613713002/diff/50001/webrtc/video/vie_encoder....
webrtc/video/vie_encoder.h:145: int GetPixelCount() const { return width *
height; }
Maybe use inline style naming?

[kthelgason, 2017-01-16 12:32:37]

The CQ bit was checked by kthelgason@webrtc.org

[kthelgason, 2017-01-16 12:32:38]

The patchset sent to the CQ was uploaded after l-g-t-m from sprang@webrtc.org,
perkj@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2613713002/#ps70001 (title:
"fix nit")

[commit-bot: I haz the power, 2017-01-16 12:32: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, 2017-01-16 14:15:23]

CQ is committing da patch.

Bot data: {"patchset_id": 70001, "attempt_start_ts": 1484569957847210,
"parent_rev": "e78d26669eeedd862150ff799d1e174bed1cfd28", "commit_rev":
"93f16d74fc1249d574ae3504945985970a10975c"}

[commit-bot: I haz the power, 2017-01-16 14:15:26]

Description was changed from

==========
delete redundant members in ViEEncoder

This information is already available in another member, and
storing it in more places only creates more opportunities
for bugs.

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

to

==========
delete redundant members in ViEEncoder

This information is already available in another member, and
storing it in more places only creates more opportunities
for bugs.

BUG=None

Review-Url: https://codereview.webrtc.org/2613713002
Cr-Commit-Position: refs/heads/master@{#16094}
Committed:
https://chromium.googlesource.com/external/webrtc/+/93f16d74fc1249d574ae35049...
==========

[commit-bot: I haz the power, 2017-01-16 14:15:27]

Committed patchset #5 (id:70001) as
https://chromium.googlesource.com/external/webrtc/+/93f16d74fc1249d574ae35049...