Cleanup: Remove unused arguments, member variable

Cleanup: Remove unused arguments, member variable

Renamed method to reflect what it does, instead of how it's implemented.

BUG=None

Committed: https://crrev.com/30188759b05f607fd5a664bc95e78159b91340bf
Cr-Commit-Position: refs/heads/master@{#427176}

[oshima, 2016-10-21 14:00:19]

oshima@chromium.org changed reviewers:
+ varkha@chromium.org

[oshima, 2016-10-21 14:04:12]

The CQ bit was checked by oshima@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-21 14:04:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[varkha, 2016-10-21 14:17:15]

Thanks for doing this! lgtm with a couple nits.

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
File ash/common/wm/overview/scoped_transform_overview_window.h (right):

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
ash/common/wm/overview/scoped_transform_overview_window.h:124: // Hides the
window header whose size is given in
nit: Incomplete sentence?

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
ash/common/wm/overview/scoped_transform_overview_window.h:127: // Shows the
window header that is hidden by HideHeader();
s/;/.

[varkha, 2016-10-21 14:19:55]

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
File ash/common/wm/overview/scoped_transform_overview_window.h (right):

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
ash/common/wm/overview/scoped_transform_overview_window.h:178: gfx::Transform
overview_transform_;
nit: Did you mean to remove this?

[commit-bot: I haz the power, 2016-10-21 14:49:03]

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

[commit-bot: I haz the power, 2016-10-21 14:49:03]

Dry run: This issue passed the CQ dry run.

[oshima, 2016-10-24 20:45:14]

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
File ash/common/wm/overview/scoped_transform_overview_window.h (right):

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
ash/common/wm/overview/scoped_transform_overview_window.h:124: // Hides the
window header whose size is given in
On 2016/10/21 14:17:15, varkha wrote:
> nit: Incomplete sentence?

Done.

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
ash/common/wm/overview/scoped_transform_overview_window.h:127: // Shows the
window header that is hidden by HideHeader();
On 2016/10/21 14:17:15, varkha wrote:
> s/;/.

Done.

https://codereview.chromium.org/2433233005/diff/1/ash/common/wm/overview/scop...
ash/common/wm/overview/scoped_transform_overview_window.h:178: gfx::Transform
overview_transform_;
On 2016/10/21 14:19:55, varkha wrote:
> nit: Did you mean to remove this?

Yes. Done.

[oshima, 2016-10-24 20:45:28]

The CQ bit was checked by oshima@chromium.org

[oshima, 2016-10-24 20:45:28]

The patchset sent to the CQ was uploaded after l-g-t-m from varkha@chromium.org
Link to the patchset: https://codereview.chromium.org/2433233005/#ps20001
(title: "Cleanup: Remove unused arguments, member variable")

[commit-bot: I haz the power, 2016-10-24 20:46:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 23:08:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-24 23:13:30]

Description was changed from

==========
Cleanup: Remove unused arguments, member variable

Renamed method to reflect what it does, instead of how it's implemented.

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

to

==========
Cleanup: Remove unused arguments, member variable

Renamed method to reflect what it does, instead of how it's implemented.

BUG=None

Committed: https://crrev.com/30188759b05f607fd5a664bc95e78159b91340bf
Cr-Commit-Position: refs/heads/master@{#427176}
==========

[commit-bot: I haz the power, 2016-10-24 23:13:31]

Patchset 2 (id:??) landed as
https://crrev.com/30188759b05f607fd5a664bc95e78159b91340bf
Cr-Commit-Position: refs/heads/master@{#427176}