Start managed_mode refactor.

Start managed_mode refactor.

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

[Bernhard Bauer, 2014-06-04 08:57:42]

I think some unit tests need to be updated as well.

https://codereview.chromium.org/316813002/diff/1/chrome/browser/managed_mode/...
File chrome/browser/managed_mode/managed_user_settings_service.h (right):

https://codereview.chromium.org/316813002/diff/1/chrome/browser/managed_mode/...
chrome/browser/managed_mode/managed_user_settings_service.h:82: // when itself
is (de)activated.
Language nit: you only need the "self" part if you use active (i.e. "when it
(de)activates itself"), not for passive.

[Marc Treib, 2014-06-04 09:23:25]

Tests updated!

https://codereview.chromium.org/316813002/diff/1/chrome/browser/managed_mode/...
File chrome/browser/managed_mode/managed_user_settings_service.h (right):

https://codereview.chromium.org/316813002/diff/1/chrome/browser/managed_mode/...
chrome/browser/managed_mode/managed_user_settings_service.h:82: // when itself
is (de)activated.
On 2014/06/04 08:57:42, Bernhard Bauer wrote:
> Language nit: you only need the "self" part if you use active (i.e. "when it
> (de)activates itself"), not for passive.

Done.

[Bernhard Bauer, 2014-06-04 11:18:44]

LGTM once you get the Android build error fixed.

FTR, you should be able to build for Android on a Linux machine (see
https://code.google.com/p/chromium/wiki/AndroidBuildInstructions), so you don't
have to wait for the trybots every time.

[Marc Treib, 2014-06-11 08:11:26]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2014-06-11 08:13:12]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/treib@chromium.org/316813002/60001

[commit-bot: I haz the power, 2014-06-11 18:09:30]

Change committed as 276452