Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.
No code changes.

BUG=NONE

Committed: https://crrev.com/78ff2ac7064126a8211a46ca88c4ad087b573fd1
Cr-Commit-Position: refs/heads/master@{#399534}

[mfomitchev, 2016-06-13 18:49:37]

Description was changed from

==========
Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.

BUG=NONE
==========

to

==========
Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.
No code changes.

BUG=NONE
==========

[mfomitchev, 2016-06-13 18:49:47]

mfomitchev@chromium.org changed reviewers:
+ msw@chromium.org

[msw, 2016-06-13 19:15:26]

lgtm with a nit.

https://codereview.chromium.org/2064813002/diff/1/chrome/browser/ui/ash/launc...
File chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc (right):

https://codereview.chromium.org/2064813002/diff/1/chrome/browser/ui/ash/launc...
chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc:1099: bool
ChromeLauncherControllerImpl::GetAppIDForShelfIDConst(
This is unused and not actually a ShelfDelegate override, feel free to remove.

[mfomitchev, 2016-06-13 20:32:30]

https://codereview.chromium.org/2064813002/diff/1/chrome/browser/ui/ash/launc...
File chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc (right):

https://codereview.chromium.org/2064813002/diff/1/chrome/browser/ui/ash/launc...
chrome/browser/ui/ash/launcher/chrome_launcher_controller_impl.cc:1099: bool
ChromeLauncherControllerImpl::GetAppIDForShelfIDConst(
On 2016/06/13 19:15:25, msw wrote:
> This is unused and not actually a ShelfDelegate override, feel free to remove.

This, will remove in a separate CL (don't want to change code here).

[mfomitchev, 2016-06-13 20:50:46]

The CQ bit was checked by mfomitchev@chromium.org

[commit-bot: I haz the power, 2016-06-13 20:51:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2064813002/1

[commit-bot: I haz the power, 2016-06-13 21:06:29]

Description was changed from

==========
Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.
No code changes.

BUG=NONE
==========

to

==========
Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.
No code changes.

BUG=NONE
==========

[commit-bot: I haz the power, 2016-06-13 21:06:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-13 21:06:32]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-13 21:09:15]

Description was changed from

==========
Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.
No code changes.

BUG=NONE
==========

to

==========
Fixing the order of methods in ChromeLauncherControllerImpl.

Fixing the order of methods in .cc to match that in .h.
No code changes.

BUG=NONE

Committed: https://crrev.com/78ff2ac7064126a8211a46ca88c4ad087b573fd1
Cr-Commit-Position: refs/heads/master@{#399534}
==========

[commit-bot: I haz the power, 2016-06-13 21:09:16]

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