Do not delete notification ids unknown by the display service

Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056, 700343
Review-Url: https://codereview.chromium.org/2739943006
Cr-Commit-Position: refs/heads/master@{#456041}
Committed: https://chromium.googlesource.com/chromium/src/+/865a357dda206267aafd20d23382b452cec8c084

[Miguel Garcia, 2017-03-09 19:01:17]

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

[commit-bot: I haz the power, 2017-03-09 19:01:46]

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

[Miguel Garcia, 2017-03-09 19:04:22]

miguelg@chromium.org changed reviewers:
+ peter@chromium.org

[Miguel Garcia, 2017-03-09 19:04:23]

This is just a temporary patch so we can merge it into M58 I will rollback the
revert shortly after.

The problem we have right now is that since the async flow has not landed yet
XPC alerts are not retrieved by GetDisplayedNotifications which means that if
you display an alert then call GetDisplayedNotifications the displayed alert
will no longer respond to click events.

[commit-bot: I haz the power, 2017-03-09 20:58:40]

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

[commit-bot: I haz the power, 2017-03-09 20:58:41]

Dry run: This issue passed the CQ dry run.

[Peter Beverloo, 2017-03-09 21:27:17]

lgtm, but please leave the //chrome/test/ stuff alone. We don't ship it and it
avoids the churn.

[Peter Beverloo, 2017-03-09 21:27:49]

On 2017/03/09 21:27:17, Peter Beverloo wrote:
> lgtm, but please leave the //chrome/test/ stuff alone. We don't ship it and it
> avoids the churn.

(That should read //content/test/)

[Miguel Garcia, 2017-03-10 09:00:44]

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

[commit-bot: I haz the power, 2017-03-10 09:00:51]

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, 2017-03-10 10:36:54]

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

[commit-bot: I haz the power, 2017-03-10 10:36:54]

Dry run: This issue passed the CQ dry run.

[Miguel Garcia, 2017-03-10 11:37:24]

Description was changed from

==========
Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056
==========

to

==========
Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056, 700343
==========

[Miguel Garcia, 2017-03-10 11:37:36]

Description was changed from

==========
Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056, 700343
==========

to

==========
Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056,700343
==========

[Miguel Garcia, 2017-03-10 11:37:48]

The CQ bit was checked by miguelg@chromium.org

[Miguel Garcia, 2017-03-10 11:37:48]

The patchset sent to the CQ was uploaded after l-g-t-m from peter@chromium.org
Link to the patchset: https://codereview.chromium.org/2739943006/#ps20001
(title: "keep conent/test/* files")

[commit-bot: I haz the power, 2017-03-10 11:38:00]

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-03-10 11:42:05]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489145868299730,
"parent_rev": "de209f3db7bbfda754be262cef35b67340c37774", "commit_rev":
"865a357dda206267aafd20d23382b452cec8c084"}

[commit-bot: I haz the power, 2017-03-10 11:42:43]

Description was changed from

==========
Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056,700343
==========

to

==========
Do not delete notification ids unknown by the display service

This is a temporary set of reverts so they can go into M58.

Revert "clean up LayoutTestNotificationManager overrides"
This reverts commit 0e9764ee9211235cb379f45df8bf983af2b0b0aa.
Revert "Test the platform notification context synchronize operation"
This reverts commit f3f824cd5032eac3c694bd9b7a5c05a2e45ebc5c.
Revert "Fix a couple of notification TODOs"
This reverts commit e50bfa204461eba626ab5891799669f9bf8231be.

BUG=571056,700343

Review-Url: https://codereview.chromium.org/2739943006
Cr-Commit-Position: refs/heads/master@{#456041}
Committed:
https://chromium.googlesource.com/chromium/src/+/865a357dda206267aafd20d23382...
==========

[commit-bot: I haz the power, 2017-03-10 11:42:45]

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