Bug 188356

Created attachment 346653 [details]
Patch

Antoine, why doesn't compositing/visible-rect/animated-from-none.html have the <!-- webkit-test-runner [ enableWebAnimationsCSSIntegration=true ] --> ?

Comment on attachment 346653 [details]
Patch

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

> LayoutTests/compositing/visible-rect/animated-from-none.html:38
> +            animated.addEventListener('webkitAnimationStart', function() {

Can be animationstart

> LayoutTests/compositing/visible-rect/animated.html:40
> +            animated.addEventListener('webkitAnimationStart', function() {

Ditto.

Comment on attachment 346653 [details]
Patch

Rejecting attachment 346653 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 346653, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Logging in as commit-queue@webkit.org...
Fetching: https://bugs.webkit.org/attachment.cgi?id=346653&action=edit
Fetching: https://bugs.webkit.org/show_bug.cgi?id=188356&ctype=xml&excludefield=attachmentdata
Processing 1 patch from 1 bug.
Updating working directory
Processing patch 346653 from bug 188356.
Fetching: https://bugs.webkit.org/attachment.cgi?id=346653
Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/LayoutTests/ChangeLog' is out of date
W: 87d5bc62303f5333b2e24243136f1dc87622a368 and refs/remotes/origin/master differ, using rebase:
:040000 040000 c6e36a9700d077d72373475d3468a0c6183f6e4b 9cdc6db6a8dbdf0098bb0b34b81fd7487692d566 M	LayoutTests
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	LayoutTests/ChangeLog

ERROR from SVN:
Item is out of date: File '/trunk/LayoutTests/ChangeLog' is out of date
W: 87d5bc62303f5333b2e24243136f1dc87622a368 and refs/remotes/origin/master differ, using rebase:
:040000 040000 c6e36a9700d077d72373475d3468a0c6183f6e4b 9cdc6db6a8dbdf0098bb0b34b81fd7487692d566 M	LayoutTests
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
From https://git.webkit.org/git/WebKit
   bb7abf9e0f2..1a0f376f55c  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 234636 = bb7abf9e0f2e04467febcc9594f5a1dcef5058dd
r234637 = 1a0f376f55cd7e733aef22d35d21857c8eb92931
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: https://webkit-queues.webkit.org/results/8782018

Comment on attachment 346653 [details]
Patch

Clearing flags on attachment: 346653

Committed r234641: <https://trac.webkit.org/changeset/234641>

All reviewed patches have been landed.  Closing bug.

<rdar://problem/42992420>

(In reply to Simon Fraser (smfr) from comment #2)
> Antoine, why doesn't compositing/visible-rect/animated-from-none.html have
> the <!-- webkit-test-runner [ enableWebAnimationsCSSIntegration=true ] --> ?

Some test don't have that yet because they worked equally with or without the flag on. When we _finally_ turn Web Animations on by default on trunk (sometime in the second half of September is the current plan) then this won't be necessary anymore.