Bug 95546 – [DRT] Make simulating a web click on a notification a queued task

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Jon Lee]

When simulating a user click on a web notification, make sure that the click is treated like a 0-timer task, rather than clicking synchronously. It mimics reality more accurately.

[Radar WebKit Bug Importer]

<rdar://problem/12214170>

[Jon Lee]

Created attachment 161730 [details]
Patch

[Alexey Proskuryakov]

Comment on attachment 161730 [details]
Patch

r=me

I suggest adding an ASSERT(!m_hasPendingWebNotificationClick) when DRT is dumping test results - the m_hasPendingWebNotificationClick check in dispatched block is insufficient for safety. If one test calls simulateWebNotificationClick, finishes, and then another one does the same, we'll end up dispatching original notificationID, breaking the second test.

[Alexey Proskuryakov]

To be clear, r+ is conditional on fixing this.

A test that leaves an outstanding notification clearly has a bug, and an assertion would go a longer way towards preventing flakiness than silently passing the error condition to the next test. The dispatched block is going to be executed anyway during the next test, and there is no guarantee that m_hasPendingWebNotificationClick will still be false by that time

[Jon Lee]

Comment on attachment 161730 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=161730&action=review

> Tools/DumpRenderTree/mac/TestRunnerMac.mm:331
> +    m_hasPendingWebNotificationClick = false;

Per ap's comments, removing this reset, and instead add an assert in dump().

[Jon Lee]

Committed r127298: <http://trac.webkit.org/changeset/127298>