Chromium Code Reviews Chromium Code Reviews
Issue 1312623003:
Flakyness: fix notifications/ tests. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@webmidi-flakyness| Index: LayoutTests/http/tests/notifications/click-document.html |
| diff --git a/LayoutTests/http/tests/notifications/click-document.html b/LayoutTests/http/tests/notifications/click-document.html |
| index 98341f6088884b2945523cd4a3882dcdb10939f5..642726774bc9b26be08c97ad491cd6f07c08b51f 100644 |
| --- a/LayoutTests/http/tests/notifications/click-document.html |
| +++ b/LayoutTests/http/tests/notifications/click-document.html |
| @@ -5,15 +5,18 @@ |
| <script src="../resources/testharness.js"></script> |
| <script src="../resources/testharnessreport.js"></script> |
| <script src="resources/test-helpers.js"></script> |
| + <script src="../resources/permissions-helper.js"></script> |
| </head> |
| <body> |
| <script> |
| // Tests that Document-bound notifications will have their onclick event |
| // fired when they're being clicked on. When the test is being ran |
| // manually, grant Notification permission and click on the notification. |
| - if (window.testRunner) |
| - testRunner.setPermission('notifications', 'granted', location.origin, location.origin); |
| + PermissionsHelper.setPermission('notifications', 'granted').then(function() { |
| + var script = document.createElement('script'); |
|
johnme
2015/08/26 12:27:32
Nit: other http/tests/notifications/ use 4-space i
mlamouri (slow - plz ping)
2015/08/31 16:31:42
Done.
|
| + script.src = 'resources/click-event-test.js'; |
| + document.body.appendChild(script); |
| + }); |
| </script> |
| - <script src="resources/click-event-test.js"></script> |
| </body> |
| </html> |
