Bug 36216 - [Qt] GraphicsLayer: support fill-modes
: [Qt] GraphicsLayer: support fill-modes
Status: CLOSED FIXED
: WebKit
Layout and Rendering
: 528+ (Nightly build)
: All All
: P3 Normal
Assigned To:
:
: Qt
: 37746
: 36191
  Show dependency treegraph
 
Reported: 2010-03-17 06:00 PST by
Modified: 2010-04-20 13:05 PST (History)


Attachments
implemet fill modes in GraphicsLayerQt (6.52 KB, patch)
2010-04-16 16:23 PST, Noam Rosenthal
no flags Review Patch | Details | Formatted Diff | Diff
removed unnecessary function (6.20 KB, patch)
2010-04-16 16:37 PST, Noam Rosenthal
no flags Review Patch | Details | Formatted Diff | Diff
re-submitting the patch, this time with a couple of more null-checks I overlooked in the first patch. All the regressed tests worked for me after this patch. (3.92 KB, patch)
2010-04-19 00:07 PST, Noam Rosenthal
kenneth: review-
commit-queue: commit‑queue-
Review Patch | Details | Formatted Diff | Diff
changelog was mistakenly omitted from previous patch (6.28 KB, patch)
2010-04-19 11:36 PST, Noam Rosenthal
no flags Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2010-03-17 06:00:38 PST
fill-modes are a new webkit CSS concept that allows for compound sequential animations.
This needs to be special-cased in GraphicsLayer.

test: LayoutTests/animations/fill-mode-transform.html
------- Comment #1 From 2010-04-16 16:23:14 PST -------
Created an attachment (id=53576) [details]
implemet fill modes in GraphicsLayerQt
------- Comment #2 From 2010-04-16 16:36:00 PST -------
r=me, please remove the unused fillBackwards function
------- Comment #3 From 2010-04-16 16:37:11 PST -------
Created an attachment (id=53580) [details]
removed unnecessary function
------- Comment #4 From 2010-04-16 19:05:00 PST -------
(From update of attachment 53580 [details])
Clearing flags on attachment: 53580

Committed r57770: <http://trac.webkit.org/changeset/57770>
------- Comment #5 From 2010-04-16 19:05:05 PST -------
All reviewed patches have been landed.  Closing bug.
------- Comment #6 From 2010-04-16 19:23:32 PST -------
http://trac.webkit.org/changeset/57770 might have broken Qt Linux Release
------- Comment #7 From 2010-04-16 23:42:41 PST -------
(In reply to comment #6)
> http://trac.webkit.org/changeset/57770 might have broken Qt Linux Release
Rollout committed in http://trac.webkit.org/changeset/57776
------- Comment #8 From 2010-04-16 23:44:59 PST -------
rollout bug: https://bugs.webkit.org/show_bug.cgi?id=37746
crashed caused by the original bug: http://build.webkit.org/results/Qt%20Linux%20Release/r57770%20%2810253%29/results.html

First I tried to do a clean build, but it didn't resolve the crashes,
so I had to rollout the patch.
------- Comment #9 From 2010-04-18 16:28:28 PST -------
(From update of attachment 53576 [details])
Cleared Antti Koivisto's review+ from obsolete attachment 53576 [details] so that this bug does not appear in http://webkit.org/pending-commit.
------- Comment #10 From 2010-04-19 00:07:37 PST -------
Created an attachment (id=53655) [details]
re-submitting the patch, this time with a couple of more null-checks I overlooked in the first patch. All the regressed tests worked for me after this patch.
------- Comment #11 From 2010-04-19 09:32:01 PST -------
(From update of attachment 53655 [details])
Rejecting patch 53655 from commit-queue.

Found no modified ChangeLogs, cannot create a commit message.
All changes require a ChangeLog.  See:
http://webkit.org/coding/contributing.html
------- Comment #12 From 2010-04-19 10:35:11 PST -------
You are right, how could I have missed that :-(. I will r- it for that reason.
------- Comment #13 From 2010-04-19 11:04:30 PST -------
(In reply to comment #10)
> Created an attachment (id=53655) [details] [details]
> re-submitting the patch, this time with a couple of more null-checks I
> overlooked in the first patch. All the regressed tests worked for me after this
> patch.

I tested it on bot, it will work without crashes.
------- Comment #14 From 2010-04-19 11:36:24 PST -------
Created an attachment (id=53695) [details]
changelog was mistakenly omitted from previous patch
------- Comment #15 From 2010-04-20 01:28:11 PST -------
(From update of attachment 53695 [details])
Clearing flags on attachment: 53695

Committed r57878: <http://trac.webkit.org/changeset/57878>
------- Comment #16 From 2010-04-20 01:28:18 PST -------
All reviewed patches have been landed.  Closing bug.
------- Comment #17 From 2010-04-20 13:05:19 PST -------
Revision r57878 cherry-picked into qtwebkit-2.0 with commit 6b307dc15ad5180ba95bb97df76f0cfcc2e8bf22