Introduce lock screen app manager interface

Introduce lock screen app manager interface

The manager will be responsible for managing the lock screen note
taking app (if one is selected) in the profile used for lock
screen apps. StateController starts the manager when the user session
is locked and stops it when user session is unlocked.
Additionally, StateController will use lock screen AppManager to
determine whether a lock screen enabled note taking app is set up, and to
dispatch launch event to the app in question.

Currently, the app manager implementation is stubbed out - it will be
added in subsequent patches.

BUG=715781
Review-Url: https://codereview.chromium.org/2949943003
Cr-Commit-Position: refs/heads/master@{#481279}
Committed: https://chromium.googlesource.com/chromium/src/+/1d75a1d92beee4cb3a492375d5f55094b7f82229

[tbarzic, 2017-06-20 22:31:04]

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

[commit-bot: I haz the power, 2017-06-20 22:31:27]

Dry run: CQ is trying da patch.

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

[tbarzic, 2017-06-20 22:43:58]

tbarzic@chromium.org changed reviewers:
+ xiyuan@chromium.org

[tbarzic, 2017-06-20 22:43:59]

Splitting out (uncontested) parts of https://codereview.chromium.org/2902293002/
to unblock few other cls.

Can you please take a look (the only difference to that cl is AppManagerImpl is
stubbed out).

[commit-bot: I haz the power, 2017-06-20 22:47:01]

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

[commit-bot: I haz the power, 2017-06-20 22:47:02]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[xiyuan, 2017-06-21 16:09:47]

lgtm

https://codereview.chromium.org/2949943003/diff/1/chrome/browser/chromeos/loc...
File chrome/browser/chromeos/lock_screen_apps/app_manager.h (right):

https://codereview.chromium.org/2949943003/diff/1/chrome/browser/chromeos/loc...
chrome/browser/chromeos/lock_screen_apps/app_manager.h:10: #include
"base/callback.h"
nit: Include callback_forward.h here and callback.h in cc where needed.

[tbarzic, 2017-06-21 16:56:18]

https://codereview.chromium.org/2949943003/diff/1/chrome/browser/chromeos/loc...
File chrome/browser/chromeos/lock_screen_apps/app_manager.h (right):

https://codereview.chromium.org/2949943003/diff/1/chrome/browser/chromeos/loc...
chrome/browser/chromeos/lock_screen_apps/app_manager.h:10: #include
"base/callback.h"
On 2017/06/21 16:09:47, xiyuan wrote:
> nit: Include callback_forward.h here and callback.h in cc where needed.

Done.

[tbarzic, 2017-06-21 16:56:22]

The CQ bit was checked by tbarzic@chromium.org

[tbarzic, 2017-06-21 16:56:22]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org
Link to the patchset: https://codereview.chromium.org/2949943003/#ps20001
(title: ".")

[commit-bot: I haz the power, 2017-06-21 16:57:03]

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, 2017-06-21 17:48:18]

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

[commit-bot: I haz the power, 2017-06-21 17:48:18]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[tbarzic, 2017-06-21 17:50:21]

The CQ bit was checked by tbarzic@chromium.org

[commit-bot: I haz the power, 2017-06-21 17:50:36]

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, 2017-06-21 19:44:22]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498067421613230,
"parent_rev": "36cfb953ea04dac15159b516cd9ce741c6b7384b", "commit_rev":
"1d75a1d92beee4cb3a492375d5f55094b7f82229"}

[commit-bot: I haz the power, 2017-06-21 19:44:34]

Description was changed from

==========
Introduce lock screen app manager interface

The manager will be responsible for managing the lock screen note
taking app (if one is selected) in the profile used for lock
screen apps. StateController starts the manager when the user session
is locked and stops it when user session is unlocked.
Additionally, StateController will use lock screen AppManager to
determine whether a lock screen enabled note taking app is set up, and to
dispatch launch event to the app in question.

Currently, the app manager implementation is stubbed out - it will be
added in subsequent patches.

BUG=715781
==========

to

==========
Introduce lock screen app manager interface

The manager will be responsible for managing the lock screen note
taking app (if one is selected) in the profile used for lock
screen apps. StateController starts the manager when the user session
is locked and stops it when user session is unlocked.
Additionally, StateController will use lock screen AppManager to
determine whether a lock screen enabled note taking app is set up, and to
dispatch launch event to the app in question.

Currently, the app manager implementation is stubbed out - it will be
added in subsequent patches.

BUG=715781

Review-Url: https://codereview.chromium.org/2949943003
Cr-Commit-Position: refs/heads/master@{#481279}
Committed:
https://chromium.googlesource.com/chromium/src/+/1d75a1d92beee4cb3a492375d5f5...
==========

[commit-bot: I haz the power, 2017-06-21 19:44:35]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1d75a1d92beee4cb3a492375d5f5...