WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
UNCONFIRMED
95061
[Tests] Make fast/canvas/canvas-scale-shadowBlur.html in conformance with spec.
https://bugs.webkit.org/show_bug.cgi?id=95061
Summary
[Tests] Make fast/canvas/canvas-scale-shadowBlur.html in conformance with spec.
Dongseong Hwang
Reported
2012-08-27 02:13:35 PDT
This test failed on firefox because this test assumed the pixel apart from 2σ is #0000. However, the pixel apart from 2σ is not #0000 mathematically. The exact value of gaussian function f(2σ) is 0.1353. This means we should check shouldBeTrue('d[0] == 255 && d[3] < 14'), but some platforms do not render anything on the pixel apart from 2σ. i.e. skia. So we need complex checking mechanism for d[0] and d[3] in order to be harmonized with both the mathematical expectation and the practical expectation.
Attachments
Patch
(8.69 KB, patch)
2012-08-27 02:15 PDT
,
Dongseong Hwang
no flags
Details
Formatted Diff
Diff
Archive of layout-test-results from gce-cr-linux-04
(315.32 KB, application/zip)
2012-08-27 10:01 PDT
,
WebKit Review Bot
no flags
Details
Patch
(8.74 KB, patch)
2012-08-27 16:23 PDT
,
Dongseong Hwang
no flags
Details
Formatted Diff
Diff
Archive of layout-test-results from gce-cr-linux-05
(634.21 KB, application/zip)
2012-08-27 18:00 PDT
,
WebKit Review Bot
no flags
Details
Patch
(17.55 KB, patch)
2012-08-27 21:09 PDT
,
Dongseong Hwang
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Dongseong Hwang
Comment 1
2012-08-27 02:15:15 PDT
Created
attachment 160675
[details]
Patch
WebKit Review Bot
Comment 2
2012-08-27 10:01:55 PDT
Comment on
attachment 160675
[details]
Patch
Attachment 160675
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://queues.webkit.org/results/13622139
New failing tests: platform/chromium/virtual/gpu/fast/canvas/canvas-scale-shadowBlur.html fast/canvas/canvas-scale-shadowBlur.html
WebKit Review Bot
Comment 3
2012-08-27 10:01:57 PDT
Created
attachment 160737
[details]
Archive of layout-test-results from gce-cr-linux-04 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: gce-cr-linux-04 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid
Eric Seidel (no email)
Comment 4
2012-08-27 13:19:39 PDT
Looks like your updated tests fails on teh cr-linux bot.
Dongseong Hwang
Comment 5
2012-08-27 16:22:20 PDT
(In reply to
comment #4
)
> Looks like your updated tests fails on teh cr-linux bot.
Yes, cr-linux bot failed. However, I can not understand why cr-linux bot applied only new test file without the new expected file belonging to this patch. I think it maybe flaky nature of cr-linux. I'll upload this patch again.
Dongseong Hwang
Comment 6
2012-08-27 16:23:08 PDT
Created
attachment 160851
[details]
Patch
WebKit Review Bot
Comment 7
2012-08-27 18:00:24 PDT
Comment on
attachment 160851
[details]
Patch
Attachment 160851
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://queues.webkit.org/results/13639115
New failing tests: platform/chromium/virtual/gpu/fast/canvas/canvas-scale-shadowBlur.html fast/canvas/canvas-scale-shadowBlur.html
WebKit Review Bot
Comment 8
2012-08-27 18:00:27 PDT
Created
attachment 160874
[details]
Archive of layout-test-results from gce-cr-linux-05 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: gce-cr-linux-05 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid
Dongseong Hwang
Comment 9
2012-08-27 21:09:28 PDT
Created
attachment 160893
[details]
Patch
Dongseong Hwang
Comment 10
2012-08-27 21:15:10 PDT
(In reply to
comment #5
)
> (In reply to
comment #4
) > > Looks like your updated tests fails on teh cr-linux bot.
I had misunderstood the reason. There are 4 platform specific expected files for relieving this problem. cr-linux bot failed, because I did not update the expected file of cr-linux. We does not need 4 platform specific expected files anymore, so I removed those.
Anders Carlsson
Comment 11
2014-02-05 11:12:07 PST
Comment on
attachment 160893
[details]
Patch Clearing review flag on patches from before 2014. If this patch is still relevant, please reset the r? flag.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug