Bug 168194 – [GTK] Test /webkit2/WebKitWebExtension/form-controls-associated-signal is flaky

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Michael Catanzaro]

The GTK+ API test /webkit2/WebKitWebExtension/form-controls-associated-signal fails roughly half the time:

ERROR:../../Tools/TestWebKitAPI/Tests/WebKit2Gtk/TestWebExtensions.cpp:223:void didAssociateFormControlsCallback(GDBusConnection*, const char*, const char*, const char*, const char*, GVariant*, WebViewTest*): assertion failed: (!g_strcmp0(formIds, FORM_ID FORM2_ID) || !g_strcmp0(formIds, INPUT_ID))

The test expects this callback to be called first with the string "form-idform2-id"; however, it is often called with "form2-idform-id" instead. Sergio, do you think this is a problem that needs to be fixed, or does it not matter and we just need to update the assertion?

[Michael Catanzaro]

Skipped in r212208: <http://trac.webkit.org/changeset/212208>

[Michael Catanzaro]

(In reply to Michael Catanzaro from comment #0)
> The test expects this callback to be called first with the string
> "form-idform2-id"; however, it is often called with "form2-idform-id"
> instead. Sergio, do you think this is a problem that needs to be fixed, or
> does it not matter and we just need to update the assertion?

We just need to update the assertion.

[Michael Catanzaro]

Created attachment 330166 [details]
Patch

[EWS Watchlist]

Comment on attachment 330166 [details]
Patch

Attachment 330166 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/5815929

New failing tests:
imported/w3c/web-platform-tests/service-workers/service-worker/ServiceWorkerGlobalScope/update.https.html

[EWS Watchlist]

Created attachment 330170 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6

[WebKit Commit Bot]

Comment on attachment 330166 [details]
Patch

Clearing flags on attachment: 330166

Committed r226326: <https://trac.webkit.org/changeset/226326>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.