RESOLVED FIXED74464
Share blend progress code 
https://bugs.webkit.org/show_bug.cgi?id=74464
Summary Share blend progress code
Simon Fraser (smfr)
Reported 2011-12-13 16:57:52 PST
Share blend progress code
Attachments
Patch (31.18 KB, patch)
2011-12-13 17:01 PST, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (31.20 KB, patch)
2011-12-13 17:16 PST, Simon Fraser (smfr) 		dino: review+
webkit.review.bot: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2011-12-13 17:01:45 PST
Created attachment 119110 [details] Patch
Gustavo Noronha (kov)
Comment 2 2011-12-13 17:11:14 PST
Comment on attachment 119110 [details] Patch Attachment 119110 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/10876032
Simon Fraser (smfr)
Comment 3 2011-12-13 17:16:03 PST
Created attachment 119116 [details] Patch
WebKit Review Bot
Comment 4 2011-12-13 18:34:13 PST
Comment on attachment 119116 [details] Patch Attachment 119116 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10875046 New failing tests: fast/gradients/css3-radial-gradients.html fast/gradients/css3-repeating-radial-gradients.html
Gyuyoung Kim
Comment 5 2011-12-13 18:48:40 PST
Comment on attachment 119116 [details] Patch Attachment 119116 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/10871059
Dean Jackson
Comment 6 2011-12-13 20:19:19 PST
Comment on attachment 119116 [details] Patch Looks good other than the failures. I can't see where radial gradient is failing. GLSL uses the term "mix" where we use "blend" - I don't suppose it is worth the effort to change, and blend makes more sense.
Simon Fraser (smfr)
Comment 7 2011-12-13 21:38:14 PST
(In reply to comment #4) > (From update of attachment 119116 [details]) > Attachment 119116 [details] did not pass chromium-ews (chromium-xvfb): > Output: http://queues.webkit.org/results/10875046 > > New failing tests: > fast/gradients/css3-radial-gradients.html > fast/gradients/css3-repeating-radial-gradients.html Why don't these test failures show up in the output: http://queues.webkit.org/results/10875046 ?
Simon Fraser (smfr)
Comment 8 2011-12-13 21:54:42 PST
http://trac.webkit.org/changeset/102740
Dean Jackson
Comment 10 2011-12-14 08:32:27 PST
(In reply to comment #9) > This patch broke Windows builds: > http://build.webkit.org/builders/Chromium%20Win%20Release/builds/36788/steps/compile-webkit/logs/stdio > http://build.webkit.org/builders/Windows%20Release%20%28Build%29/builds/24728/steps/compile-webkit/logs/stdio > > lround isn't available on Visual Studio although it's part of ANSI C. Weird. lround was already being used - this patch just moved it.
Dean Jackson
Comment 11 2011-12-14 08:42:41 PST
Seems that the build error was fixed by http://trac.webkit.org/changeset/102745
Simon Fraser (smfr)
Comment 12 2011-12-14 08:44:30 PST
Ah, I was just about to make the same change. Thanks morrita!
Note You need to log in before you can comment on or make changes to this bug.