Cros Tablet: Add UMA stats for the user close actions in tablet mode.

Cros Tablet: Add UMA stats for the user close actions in tablet mode.

There are three ways to close a browser/app window in tablet mode:
1) from the close button in the title bar area on the top right corner
of the window
2) from the close button in overview mode
3) from the shelf icon context menu

The goal is to evaluate the three ways to close a window in tablet mode
to see if we can remove the title bar for a tablet window.

BUG=732983
Review-Url: https://codereview.chromium.org/2946543002
Cr-Commit-Position: refs/heads/master@{#480489}
Committed: https://chromium.googlesource.com/chromium/src/+/80f52fd8f2b2ed19058e8f819962272a68387bdd

[xdai1, 2017-06-16 22:08:08]

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

[commit-bot: I haz the power, 2017-06-16 22:08:25]

Dry run: CQ is trying da patch.

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

[xdai1, 2017-06-16 22:10:17]

xdai@chromium.org changed reviewers:
+ asvitkine@chromium.org, oshima@chromium.org

[xdai1, 2017-06-16 22:10:17]

oshima@, asvitkine@, could you help review this CL please? Thanks!

asvitkine@: tools/metrics/actions/actions.xml
oshima@: all other files

[commit-bot: I haz the power, 2017-06-16 22:50:54]

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

[commit-bot: I haz the power, 2017-06-16 22:50:55]

Dry run: This issue passed the CQ dry run.

[oshima, 2017-06-16 23:33:00]

lgtm

[Alexei Svitkine (slow), 2017-06-19 14:48:56]

lgtm

https://codereview.chromium.org/2946543002/diff/1/tools/metrics/actions/actio...
File tools/metrics/actions/actions.xml (right):

https://codereview.chromium.org/2946543002/diff/1/tools/metrics/actions/actio...
tools/metrics/actions/actions.xml:16596: tablet mode.
If this is CrOS-specific, please mention it in the description.

[xdai1, 2017-06-19 17:05:20]

Thanks for the review!

https://codereview.chromium.org/2946543002/diff/1/tools/metrics/actions/actio...
File tools/metrics/actions/actions.xml (right):

https://codereview.chromium.org/2946543002/diff/1/tools/metrics/actions/actio...
tools/metrics/actions/actions.xml:16596: tablet mode.
On 2017/06/19 14:48:56, Alexei Svitkine (very slow) wrote:
> If this is CrOS-specific, please mention it in the description.

Done.

[xdai1, 2017-06-19 17:05:27]

The CQ bit was checked by xdai@chromium.org

[xdai1, 2017-06-19 17:05:27]

The patchset sent to the CQ was uploaded after l-g-t-m from oshima@chromium.org,
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/2946543002/#ps20001
(title: "Address asvitkine@'s comments.")

[commit-bot: I haz the power, 2017-06-19 17:05:55]

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-19 17:43:44]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1497891927352430,
"parent_rev": "313a0d36569b810ddf6060860c35f9deba7c922b", "commit_rev":
"80f52fd8f2b2ed19058e8f819962272a68387bdd"}

[commit-bot: I haz the power, 2017-06-19 17:43:58]

Description was changed from

==========
Cros Tablet: Add UMA stats for the user close actions in tablet mode.

There are three ways to close a browser/app window in tablet mode:
1) from the close button in the title bar area on the top right corner
of the window
2) from the close button in overview mode
3) from the shelf icon context menu

The goal is to evaluate the three ways to close a window in tablet mode
to see if we can remove the title bar for a tablet window.

BUG=732983
==========

to

==========
Cros Tablet: Add UMA stats for the user close actions in tablet mode.

There are three ways to close a browser/app window in tablet mode:
1) from the close button in the title bar area on the top right corner
of the window
2) from the close button in overview mode
3) from the shelf icon context menu

The goal is to evaluate the three ways to close a window in tablet mode
to see if we can remove the title bar for a tablet window.

BUG=732983

Review-Url: https://codereview.chromium.org/2946543002
Cr-Commit-Position: refs/heads/master@{#480489}
Committed:
https://chromium.googlesource.com/chromium/src/+/80f52fd8f2b2ed19058e8f819962...
==========

[commit-bot: I haz the power, 2017-06-19 17:44:00]

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