Removed Ash.AppList.TimeBetweenTaskSwitches and Ash.Tab.TimeBetweenSwitchToExistingTabUserActions h…

Removed Ash.AppList.TimeBetweenTaskSwitches and Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
TEST=ash_unittests

Committed: https://crrev.com/9a7916b557200bb2ea8291732158e200f434078b
Cr-Commit-Position: refs/heads/master@{#422885}

[bruthig, 2016-10-03 18:10:31]

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

[commit-bot: I haz the power, 2016-10-03 18:10:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bruthig, 2016-10-03 18:11:22]

bruthig@chromium.org changed reviewers:
+ sky@chromium.org

[bruthig, 2016-10-03 18:11:23]

sky@, can you PTAL?

[commit-bot: I haz the power, 2016-10-03 18:29:17]

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

[commit-bot: I haz the power, 2016-10-03 18:29:19]

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

[sky, 2016-10-03 19:11:51]

https://codereview.chromium.org/2390793002/diff/1/ash/common/metrics/task_swi...
File ash/common/metrics/task_switch_source.h (left):

https://codereview.chromium.org/2390793002/diff/1/ash/common/metrics/task_swi...
ash/common/metrics/task_switch_source.h:17: APP_LIST,
Are these values written to a histogram?

[bruthig, 2016-10-04 14:16:33]

Description was changed from

==========
Removed Ash.AppList.TimeBetweenTaskSwitches and
Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
==========

to

==========
Removed Ash.AppList.TimeBetweenTaskSwitches and
Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
TEST=ash_unittests
==========

[bruthig, 2016-10-04 14:16:50]

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

[bruthig, 2016-10-04 14:17:09]

https://codereview.chromium.org/2390793002/diff/1/ash/common/metrics/task_swi...
File ash/common/metrics/task_switch_source.h (left):

https://codereview.chromium.org/2390793002/diff/1/ash/common/metrics/task_swi...
ash/common/metrics/task_switch_source.h:17: APP_LIST,
On 2016/10/03 19:11:51, sky wrote:
> Are these values written to a histogram?

No, they are used to look up a histogram name.

[commit-bot: I haz the power, 2016-10-04 14:17:10]

Dry run: 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, 2016-10-04 15:20:56]

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

[commit-bot: I haz the power, 2016-10-04 15:20:57]

Dry run: This issue passed the CQ dry run.

[sky, 2016-10-04 18:20:17]

Ok, LGTM. And thanks for the cleanup!

[bruthig, 2016-10-04 18:49:25]

bruthig@chromium.org changed reviewers:
+ rkaplow@chromium.org

[bruthig, 2016-10-04 18:49:25]

rkaplow@, can you take a quick look at histograms.xml?

[rkaplow, 2016-10-04 19:24:11]

lgtm

[bruthig, 2016-10-04 19:26:47]

The CQ bit was checked by bruthig@chromium.org

[commit-bot: I haz the power, 2016-10-04 19:27:14]

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, 2016-10-04 19:34:51]

Description was changed from

==========
Removed Ash.AppList.TimeBetweenTaskSwitches and
Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
TEST=ash_unittests
==========

to

==========
Removed Ash.AppList.TimeBetweenTaskSwitches and
Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
TEST=ash_unittests
==========

[commit-bot: I haz the power, 2016-10-04 19:34:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-04 19:37:52]

Description was changed from

==========
Removed Ash.AppList.TimeBetweenTaskSwitches and
Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
TEST=ash_unittests
==========

to

==========
Removed Ash.AppList.TimeBetweenTaskSwitches and
Ash.Tab.TimeBetweenSwitchToExistingTabUserActions histograms.

These histograms are triggered in the chrome process but are actually
recorded in ash (which is in a different process). This is problematic
for the mus+ash refactoring and since there are no clear decisions
being made from these metrics they are being removed.

BUG=616581
TEST=ash_unittests

Committed: https://crrev.com/9a7916b557200bb2ea8291732158e200f434078b
Cr-Commit-Position: refs/heads/master@{#422885}
==========

[commit-bot: I haz the power, 2016-10-04 19:37:53]

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