Move UserActivityDetector to ui/base/user_activity/.

Move UserActivityDetector to ui/base/user_activity/.

Move UserActivityDetector and UserActivityObserver out of
ui/wm/core/ so that other classes in ui/base/ can depend on
them. This is needed in order to be able to later move
chrome/browser/idle_chromeos.cc out of chrome/ so that the
chrome.idle API can eventually be moved to extensions/.

BUG=446320
Committed: https://crrev.com/2d4e62d2cbd43d83fc43d58f0fdaa904633c33c1
Cr-Commit-Position: refs/heads/master@{#310773}

[Daniel Erat, 2015-01-08 02:51:02]

derat@chromium.org changed reviewers:
+ oshima@chromium.org, sky@chromium.org

[Daniel Erat, 2015-01-08 02:51:02]

oshima: please look at athena.
scott: please look at everything else!

[sky, 2015-01-08 03:31:44]

I tend to think of ui/events as more about event processing and event structures
and not so much activity. Did you consider ui/user_activity? I don't feel
particularly strongly, but tend to think ui/user_activity would be a better fit.

I'm tempted to ask Sadrul for his thoughts, but looks like he is OOO.

[sadrul, 2015-01-08 04:16:09]

On 2015/01/08 03:31:44, sky wrote:
> I tend to think of ui/events as more about event processing and event
structures
> and not so much activity. Did you consider ui/user_activity? I don't feel
> particularly strongly, but tend to think ui/user_activity would be a better
fit.

+1 (or if we expect to factor out a number of other such event-handlers, then
perhaps move them into //ui/event_handlers/ instead of into //ui/events/)

> 
> I'm tempted to ask Sadrul for his thoughts, but looks like he is OOO.

[oshima, 2015-01-08 09:02:46]

On 2015/01/08 04:16:09, sadrul (OOO) wrote:
> On 2015/01/08 03:31:44, sky wrote:
> > I tend to think of ui/events as more about event processing and event
> structures
> > and not so much activity. Did you consider ui/user_activity? I don't feel
> > particularly strongly, but tend to think ui/user_activity would be a better
> fit.
> 
> +1 (or if we expect to factor out a number of other such event-handlers, then
> perhaps move them into //ui/event_handlers/ instead of into //ui/events/)
> 
> > 
> > I'm tempted to ask Sadrul for his thoughts, but looks like he is OOO.

Another option is components/user_activity_detector. I'm fine either way.

[Daniel Erat, 2015-01-08 14:19:44]

okay, all thoughts that i had. thanks! :-)

i'll go with ui/user_activity/; i didn't know that a directory directly below
ui/ was an option.

after this change, i'll also be wanting to move chrome/idle* under ui/. are you
all fine with me using ui/idle/ for that?

[Daniel Erat, 2015-01-08 15:16:57]

started switching to ui/user_activity, but creating a new unit test target for
such a tiny directory seems dumb. any other suggestions? ui/base/user_activity
(or is this too specific for ui/base?)?

[Daniel Erat, 2015-01-08 21:23:57]

another look? after some discussion with scott, i went with
ui/base/user_activity.

[oshima, 2015-01-08 21:31:23]

athena lgtm. you may tbr athena from now on as we'll soon remove it from build.

[sky, 2015-01-08 23:40:11]

LGTM - thanks

[Daniel Erat, 2015-01-08 23:47:11]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2015-01-08 23:49:24]

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

[commit-bot: I haz the power, 2015-01-09 00:38:37]

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

[commit-bot: I haz the power, 2015-01-09 00:38:38]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)

[Daniel Erat, 2015-01-09 15:15:20]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2015-01-09 15:16:10]

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

[commit-bot: I haz the power, 2015-01-09 16:09:47]

Committed patchset #7 (id:110001)

[commit-bot: I haz the power, 2015-01-09 16:11:08]

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