Summary: | Implement a WebKitTransitionEvent constructor for JSC | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kentaro Hara <haraken> | ||||||||
Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | abarth, ap, dominicc, rniwa, sam, simon.fraser, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 67824 | ||||||||||
Attachments: |
|
Description
Kentaro Hara
2011-10-11 03:42:35 PDT
Created attachment 110496 [details]
Patch
Please do the animation events at the same time. (In reply to comment #2) > Please do the animation events at the same time. Constructor for WebKitAnimationEvent is already on trunk both for JSC and for V8. (bug 67825 and bug 67922). Or, do you mean that we need more test cases where WebKitAnimationEvent and WebKitAnimationEvent are used at the same time (like animations/transition-and-animation-*.html)? (In reply to comment #3) > (In reply to comment #2) > > Please do the animation events at the same time. > > Constructor for WebKitAnimationEvent is already on trunk both for JSC and for V8. (bug 67825 and bug 67922). Ah, I didn't realize they were done. Comment on attachment 110496 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=110496&action=review > Source/WebCore/dom/WebKitTransitionEvent.cpp:35 > + , elapsedTime(0.0) Just use 0, not 0.0 Created attachment 110627 [details]
rebased patch for commit
Comment on attachment 110627 [details] rebased patch for commit Rejecting attachment 110627 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 1 Last 500 characters of output: 4ffd39561b4144c727ef44bdcacdff4e49fffb1b r97219 = 2d6bddd541d82ee7ac8dfd329feb66e56e81c258 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. Updating chromium port dependencies using gclient... ________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' Updating webkit projects from gyp files... Full output: http://queues.webkit.org/results/10030741 Comment on attachment 110627 [details]
rebased patch for commit
If the review flag isn't set, you need to fill out the reviewer In the changelog.
Created attachment 110630 [details]
rebased patch for commit
(In reply to comment #8) > (From update of attachment 110627 [details]) > If the review flag isn't set, you need to fill out the reviewer In the changelog. Ah, thank you for letting me know! Comment on attachment 110627 [details] rebased patch for commit Clearing flags on attachment: 110627 Committed r97225: <http://trac.webkit.org/changeset/97225> Comment on attachment 110630 [details] rebased patch for commit Rejecting attachment 110630 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: 13bfc3c27b3ef9b9dbaaef33caa8673d70b81b81 r97225 = 9dbc3f5be8d6a8831e5a895939bfd3c36d7076d0 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. Updating chromium port dependencies using gclient... ________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium' Updating webkit projects from gyp files... Full output: http://queues.webkit.org/results/10031656 Rebaseline done in http://trac.webkit.org/changeset/97236. |