Fix NotificationUIManagerTest::testShowAndCloseMultipleNotifications.

Fix NotificationUIManagerTest::testShowAndCloseMultipleNotifications.

Do this by making sure that we don't unregister the Service Worker for
each notification that's being shown, which actually closes the existing
ones on the platform.

BUG=472926
Committed: https://crrev.com/22156329c147f636817d6f0005ba1de21b325b80
Cr-Commit-Position: refs/heads/master@{#325436}

[Peter Beverloo, 2015-04-14 17:46:53]

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

[Peter Beverloo, 2015-04-14 17:46:54]

PTAL

[Michael van Ouwerkerk, 2015-04-15 11:02:09]

Nice find! lgtm, the nit is optional

https://codereview.chromium.org/1082273003/diff/1/chrome/test/data/notificati...
File chrome/test/data/notifications/notification_test_utils.js (right):

https://codereview.chromium.org/1082273003/diff/1/chrome/test/data/notificati...
chrome/test/data/notifications/notification_test_utils.js:5: var
g_activated_service_worker_promise = null;
nit: use camelCase for js, the entire language is that way.

[Peter Beverloo, 2015-04-15 17:13:35]

Thanks!

https://codereview.chromium.org/1082273003/diff/1/chrome/test/data/notificati...
File chrome/test/data/notifications/notification_test_utils.js (right):

https://codereview.chromium.org/1082273003/diff/1/chrome/test/data/notificati...
chrome/test/data/notifications/notification_test_utils.js:5: var
g_activated_service_worker_promise = null;
On 2015/04/15 11:02:09, Michael van Ouwerkerk wrote:
> nit: use camelCase for js, the entire language is that way.

Done.

[Peter Beverloo, 2015-04-16 13:27:17]

The CQ bit was checked by peter@chromium.org

[Peter Beverloo, 2015-04-16 13:27:17]

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

[commit-bot: I haz the power, 2015-04-16 13:27:49]

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

[commit-bot: I haz the power, 2015-04-16 13:49:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-16 13:50:42]

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