Added Launcher_MinimizeTask UMA user action.

Added Launcher_MinimizeTask UMA user action.

The Launcher_MinimzeTask user action is recorded when the user activates a
shelf icon causing a window to be minimized.

TEST=ShelfViewTest.Launcher_MinimizeTaskIsRecordedWhenWindowIsMinimized

BUG=471358
Committed: https://crrev.com/f29a78fcc51cde6b9a8bd1a00b2c0c2fa799580c
Cr-Commit-Position: refs/heads/master@{#326834}

[bruthig, 2015-04-21 21:51:43]

bruthig@chromium.org changed reviewers:
+ asvitkine@chromium.org, oshima@chromium.org, rkaplow@chromium.org

[bruthig, 2015-04-21 21:51:44]

rkaplow@|asvitkine@: Please review changes in tools/metrics/*

oshima@: Please review changes in ash/*

Note: This review is based off of the pending review here:
https://codereview.chromium.org/1097393002/

[oshima, 2015-04-21 22:52:15]

https://codereview.chromium.org/1098193003/diff/1/ash/shelf/shelf_view_unitte...
File ash/shelf/shelf_view_unittest.cc (right):

https://codereview.chromium.org/1098193003/diff/1/ash/shelf/shelf_view_unitte...
ash/shelf/shelf_view_unittest.cc:1846: TEST_F(ShelfViewTest,
Launcher_MinimizeTaskIsRecordedWhenNewWindowIsCreated) {
The name sounds odd. Shouldn't this be something like

MinimizeTaskIsRecordedWhenItemIsSelected

?

[bruthig, 2015-04-21 23:06:46]

Fixed incorrect test name.

oshima@ can you please have another look at ash/*?

https://codereview.chromium.org/1098193003/diff/1/ash/shelf/shelf_view_unitte...
File ash/shelf/shelf_view_unittest.cc (right):

https://codereview.chromium.org/1098193003/diff/1/ash/shelf/shelf_view_unitte...
ash/shelf/shelf_view_unittest.cc:1846: TEST_F(ShelfViewTest,
Launcher_MinimizeTaskIsRecordedWhenNewWindowIsCreated) {
On 2015/04/21 22:52:15, oshima wrote:
> The name sounds odd. Shouldn't this be something like
> 
> MinimizeTaskIsRecordedWhenItemIsSelected
> 
> ?

Done.

[oshima, 2015-04-21 23:13:33]

ash lgtm

[rkaplow, 2015-04-22 14:23:43]

lgtm

probably should have mentioned this includes the code from your other CL.

This also LGTM, but will need asvitkine review as well

[Alexei Svitkine (slow), 2015-04-22 15:57:01]

lgtm

[bruthig, 2015-04-23 13:55:51]

The CQ bit was checked by bruthig@chromium.org

[commit-bot: I haz the power, 2015-04-23 13:56:03]

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

[commit-bot: I haz the power, 2015-04-23 14:00:08]

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

[commit-bot: I haz the power, 2015-04-23 14:00:09]

Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[bruthig, 2015-04-24 17:09:22]

The CQ bit was checked by bruthig@chromium.org

[bruthig, 2015-04-24 17:09:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org, oshima@chromium.org, rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/1098193003/#ps40001
(title: "Rebase on master.")

[commit-bot: I haz the power, 2015-04-24 17:09:52]

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

[bruthig, 2015-04-24 17:10:53]

I rebased on master.  No need to review.

[commit-bot: I haz the power, 2015-04-24 18:05:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-04-24 18:06:29]

Patchset 3 (id:??) landed as
https://crrev.com/f29a78fcc51cde6b9a8bd1a00b2c0c2fa799580c
Cr-Commit-Position: refs/heads/master@{#326834}