http://build.webkit.org/results/Qt%20Linux%20Release/r89506%20%2834502%29/results.html
I skipped the failing tests: http://trac.webkit.org/changeset/89569 But it doesn't solve the real bug which should be fixed or the original patch should be rolled out.
r89503 changed the timings of the callback that tells the embedder that a load is complete. From the layout test diffs you linked, it looks like the notifications aren't being completed before the test finishes and the next one starts. Feel free to grab me on IRC if there's anything I can do to help with this.
Ossy, since r89503 was mostly reverted, should we unskip the notifications tests?
(In reply to comment #3) > Ossy, since r89503 was mostly reverted, should we unskip the notifications tests? They were unskipped (accidentally :)) ): http://trac.webkit.org/changeset/89732/trunk/LayoutTests/platform/qt/Skipped
The tests started to fail again after http://trac.webkit.org/changeset/90471: fast/notifications/notifications-no-icon.html (flaky) fast/notifications/notifications-double-show.html fast/notifications/notifications-with-permission.html fast/notifications/notifications-without-permission.html Skipped tests in http://trac.webkit.org/changeset/90570 and reopening the bug. http://build.webkit.org/results/Qt%20Linux%20Release/r90560%20(35105)/results.html
Before http://trac.webkit.org/changeset/89569, we would wait for the notification icon to load before transitioning to committed state, and the test had enough time to complete. With http://trac.webkit.org/changeset/89569, we transition to committed state before the load is finished, thus the tests are failing. I think the behavior of Notifications is still correct, but the tests need to be modified, so that they wait for the icon to finish loading. I am preparing a patch for that.
Created attachment 100144 [details] Patch Modified the tests to wait until icon loading is finished.
Comment on attachment 100144 [details] Patch LGTM, r=me
Comment on attachment 100144 [details] Patch Rejecting attachment 100144 [details] from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=ec2-cq-03', '--port..." exit_code: 1 Last 500 characters of output: cripts/webkitpy/common/system/executive.py", line 423, in run_command close_fds=self._should_close_fds()) File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/system/executive.py", line 479, in popen return subprocess.Popen(*args, **kwargs) File "/usr/lib/python2.6/subprocess.py", line 623, in __init__ errread, errwrite) File "/usr/lib/python2.6/subprocess.py", line 1141, in _execute_child raise child_exception TypeError: execv() arg 2 must contain only strings Full output: http://queues.webkit.org/results/9013318
(In reply to comment #9) > (From update of attachment 100144 [details]) > Rejecting attachment 100144 [details] from commit-queue. > > Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=ec2-cq-03', '--port..." exit_code: 1 > > Last 500 characters of output: > cripts/webkitpy/common/system/executive.py", line 423, in run_command > close_fds=self._should_close_fds()) > File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/system/executive.py", line 479, in popen > return subprocess.Popen(*args, **kwargs) > File "/usr/lib/python2.6/subprocess.py", line 623, in __init__ > errread, errwrite) > File "/usr/lib/python2.6/subprocess.py", line 1141, in _execute_child > raise child_exception > TypeError: execv() arg 2 must contain only strings > > Full output: http://queues.webkit.org/results/9013318 CQ still hates my name. :(( Could you land it manually?
Comment on attachment 100144 [details] Patch Rejecting attachment 100144 [details] from commit-queue. Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=ec2-cq-03', '--port..." exit_code: 1 Last 500 characters of output: cripts/webkitpy/common/system/executive.py", line 423, in run_command close_fds=self._should_close_fds()) File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/system/executive.py", line 479, in popen return subprocess.Popen(*args, **kwargs) File "/usr/lib/python2.6/subprocess.py", line 623, in __init__ errread, errwrite) File "/usr/lib/python2.6/subprocess.py", line 1141, in _execute_child raise child_exception TypeError: execv() arg 2 must contain only strings Full output: http://queues.webkit.org/results/9014194
Committed r90471: <http://trac.webkit.org/changeset/90741>