Specify TaskType of posted Task explicitly in popup UIs (8)

Specify TaskType of posted Task explicitly in popup UIs

This CL assigns TaskType::UserInteraction to the popup menu event and
notification event.

For Notification API event, the spec doesn't specify the task source
explicitly. TaskType::UserInteraction should be the best fit here.
https://notifications.spec.whatwg.org/

For popup menus, the event has non-web event source.
TaskType::UserInteraction is the best fit here too.

BUG=624696
Committed: https://crrev.com/10f963b41ab8dcea765528f187bde5c252e960ad
Cr-Commit-Position: refs/heads/master@{#439389}

[tzik, 2016-12-18 14:06:49]

Description was changed from

==========
Specify TaskType of posted Task explicitly in popup UIs (8)

BUG=
==========

to

==========
Specify TaskType of posted Task explicitly in popup UIs



BUG=624696
==========

[tzik, 2016-12-18 14:21:29]

Description was changed from

==========
Specify TaskType of posted Task explicitly in popup UIs



BUG=624696
==========

to

==========
Specify TaskType of posted Task explicitly in popup UIs

This CL assigns TaskType::UserInteraction to the popup menu event and
notification event.

For Notification API event, the spec doesn't specify the task source
explicitly. TaskType::UserInteraction should be the best fit here. 
https://notifications.spec.whatwg.org/

For popup menus, the event has non-web event source.
TaskType::UserInteraction is the best fit here too.

BUG=624696
==========

[tzik, 2016-12-18 14:21:35]

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

[commit-bot: I haz the power, 2016-12-18 14:21:47]

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

[tzik, 2016-12-18 14:41:50]

tzik@chromium.org changed reviewers:
+ haraken@chromium.org

[tzik, 2016-12-18 14:41:51]

PTAL

[haraken, 2016-12-18 14:56:34]

LGTM

[commit-bot: I haz the power, 2016-12-18 16:38:45]

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

[commit-bot: I haz the power, 2016-12-18 16:38:46]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-12-18 23:28:21]

The CQ bit was checked by tzik@chromium.org

[tzik, 2016-12-18 23:28:22]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2589493003/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2016-12-18 23:28:30]

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-12-19 00:51:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482103701977170,
"parent_rev": "4373c1fcef5bb1b239f6bd07a465fd477b5313af", "commit_rev":
"44c6be5707fbaa0842e60d6778652a579fe4491a"}

[commit-bot: I haz the power, 2016-12-19 00:51:34]

Description was changed from

==========
Specify TaskType of posted Task explicitly in popup UIs

This CL assigns TaskType::UserInteraction to the popup menu event and
notification event.

For Notification API event, the spec doesn't specify the task source
explicitly. TaskType::UserInteraction should be the best fit here. 
https://notifications.spec.whatwg.org/

For popup menus, the event has non-web event source.
TaskType::UserInteraction is the best fit here too.

BUG=624696
==========

to

==========
Specify TaskType of posted Task explicitly in popup UIs

This CL assigns TaskType::UserInteraction to the popup menu event and
notification event.

For Notification API event, the spec doesn't specify the task source
explicitly. TaskType::UserInteraction should be the best fit here. 
https://notifications.spec.whatwg.org/

For popup menus, the event has non-web event source.
TaskType::UserInteraction is the best fit here too.

BUG=624696

Review-Url: https://codereview.chromium.org/2589493003
==========

[commit-bot: I haz the power, 2016-12-19 00:51:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-19 00:53:41]

Description was changed from

==========
Specify TaskType of posted Task explicitly in popup UIs

This CL assigns TaskType::UserInteraction to the popup menu event and
notification event.

For Notification API event, the spec doesn't specify the task source
explicitly. TaskType::UserInteraction should be the best fit here. 
https://notifications.spec.whatwg.org/

For popup menus, the event has non-web event source.
TaskType::UserInteraction is the best fit here too.

BUG=624696

Review-Url: https://codereview.chromium.org/2589493003
==========

to

==========
Specify TaskType of posted Task explicitly in popup UIs

This CL assigns TaskType::UserInteraction to the popup menu event and
notification event.

For Notification API event, the spec doesn't specify the task source
explicitly. TaskType::UserInteraction should be the best fit here.
https://notifications.spec.whatwg.org/

For popup menus, the event has non-web event source.
TaskType::UserInteraction is the best fit here too.

BUG=624696

Committed: https://crrev.com/10f963b41ab8dcea765528f187bde5c252e960ad
Cr-Commit-Position: refs/heads/master@{#439389}
==========

[commit-bot: I haz the power, 2016-12-19 00:53:42]

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