Remove static initializer from AppListView::SetNextPaintCallback().

Remove static initializer from AppListView::SetNextPaintCallback().

BUG=317171
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=235692

[Vadim Sh., 2013-11-14 07:27:21]

Hello. I did some updates on isolateserver.appspot.com, and it affected your
linux_rel try job (compile failed with 'Swarming Error').
I've restarted it. Sorry for that.

[benwells, 2013-11-15 02:13:58]

https://codereview.chromium.org/68173019/diff/1/chrome/browser/ui/app_list/ap...
File chrome/browser/ui/app_list/app_list_service.cc (right):

https://codereview.chromium.org/68173019/diff/1/chrome/browser/ui/app_list/ap...
chrome/browser/ui/app_list/app_list_service.cc:56: void
RecordStartupInfo(StartupType startup_type, const base::Time& start_time) {
What calls this?

https://codereview.chromium.org/68173019/diff/1/chrome/browser/ui/app_list/ap...
File chrome/browser/ui/app_list/app_list_service.h (right):

https://codereview.chromium.org/68173019/diff/1/chrome/browser/ui/app_list/ap...
chrome/browser/ui/app_list/app_list_service.h:11: #include
"base/callback_forward.h"
Nit: not needed.

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
File ui/app_list/views/app_list_view.cc (right):

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
ui/app_list/views/app_list_view.cc:7: #include "base/callback.h"
Nit: this seems to be not needed anymore.

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
File ui/app_list/views/app_list_view.h (right):

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
ui/app_list/views/app_list_view.h:8: #include "base/callback_forward.h"
Nit: this ain't needed anymore.

[koz (OOO until 15th September), 2013-11-17 23:16:19]

https://codereview.chromium.org/68173019/diff/1/chrome/browser/ui/app_list/ap...
File chrome/browser/ui/app_list/app_list_service.cc (right):

https://codereview.chromium.org/68173019/diff/1/chrome/browser/ui/app_list/ap...
chrome/browser/ui/app_list/app_list_service.cc:56: void
RecordStartupInfo(StartupType startup_type, const base::Time& start_time) {
On 2013/11/15 02:13:59, benwells wrote:
> What calls this?

Oops, it's meant to be called below when we set the next paint callback.

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
File ui/app_list/views/app_list_view.cc (right):

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
ui/app_list/views/app_list_view.cc:7: #include "base/callback.h"
On 2013/11/15 02:13:59, benwells wrote:
> Nit: this seems to be not needed anymore.

Done.

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
File ui/app_list/views/app_list_view.h (right):

https://codereview.chromium.org/68173019/diff/1/ui/app_list/views/app_list_vi...
ui/app_list/views/app_list_view.h:8: #include "base/callback_forward.h"
On 2013/11/15 02:13:59, benwells wrote:
> Nit: this ain't needed anymore.

Done.

[benwells, 2013-11-18 05:13:31]

lgtm with nits

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
File chrome/browser/ui/app_list/app_list_service.cc (right):

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
chrome/browser/ui/app_list/app_list_service.cc:8: #include "base/callback.h"
Nit: are bind.h and callback.h still needed?

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
File chrome/browser/ui/app_list/app_list_service.h (right):

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
chrome/browser/ui/app_list/app_list_service.h:11: #include
"base/callback_forward.h"
Nit: this isn't needed anymore.

[koz (OOO until 15th September), 2013-11-18 06:31:07]

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
File chrome/browser/ui/app_list/app_list_service.cc (right):

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
chrome/browser/ui/app_list/app_list_service.cc:8: #include "base/callback.h"
On 2013/11/18 05:13:32, benwells wrote:
> Nit: are bind.h and callback.h still needed?

Done.

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
File chrome/browser/ui/app_list/app_list_service.h (right):

https://codereview.chromium.org/68173019/diff/160001/chrome/browser/ui/app_li...
chrome/browser/ui/app_list/app_list_service.h:11: #include
"base/callback_forward.h"
On 2013/11/18 05:13:32, benwells wrote:
> Nit: this isn't needed anymore.

Done.

[commit-bot: I haz the power, 2013-11-18 06:31:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/koz@chromium.org/68173019/240001

[commit-bot: I haz the power, 2013-11-18 11:04:36]

Change committed as 235692