Bug 81029 - css3/filters/custom/custom-filter-property-computed-style.html passes in browser, fails in DRT
Summary: css3/filters/custom/custom-filter-property-computed-style.html passes in brow...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, LayoutTestFailure
Depends on:
Blocks:
 
Reported: 2012-03-13 13:26 PDT by Tim Horton
Modified: 2014-03-02 10:08 PST (History)
6 users (show)

See Also:


Attachments
patch to land failing expectations (20.58 KB, patch)
2012-03-13 13:49 PDT, Tim Horton
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Horton 2012-03-13 13:26:36 PDT
Running this test in DRT, I see:

Custom with vertex shader : custom(url(vertex.shader))
FAIL removeBaseURL(filterStyle.getPropertyValue('-webkit-filter')) should be custom(url(vertex.shader) none, 1 1 filter-box). Was none.
FAIL jsWrapperClass(filterRule) should be CSSValueList. Was CSSPrimitiveValue.
FAIL jsWrapperClass(filterRule.__proto__) should be CSSValueListPrototype. Was CSSPrimitiveValuePrototype.
FAIL jsWrapperClass(filterRule.constructor) should be CSSValueListConstructor. Was CSSPrimitiveValueConstructor.
FAIL filterRule.length should be 1 (of type number). Was undefined (of type undefined).
FAIL subRule.operationType should be 12. Threw exception TypeError: 'undefined' is not an object (evaluating 'subRule.operationType')
FAIL removeBaseURL(subRule.cssText) should be custom(url(vertex.shader) none, 1 1 filter-box). Threw exception TypeError: 'undefined' is not an object (evaluating 'subRule.cssText')

etc.

Running this test in Safari, I see:

Custom with vertex shader : custom(url(vertex.shader))
PASS removeBaseURL(filterStyle.getPropertyValue('-webkit-filter')) is 'custom(url(vertex.shader) none, 1 1 filter-box)'
PASS jsWrapperClass(filterRule) is 'CSSValueList'
PASS jsWrapperClass(filterRule.__proto__) is 'CSSValueListPrototype'
PASS jsWrapperClass(filterRule.constructor) is 'CSSValueListConstructor'
PASS filterRule.length is 1
PASS subRule.operationType is WebKitCSSFilterValue.CSS_FILTER_CUSTOM
PASS removeBaseURL(subRule.cssText) is 'custom(url(vertex.shader) none, 1 1 filter-box)'

etc.
Comment 1 Radar WebKit Bug Importer 2012-03-13 13:27:32 PDT
<rdar://problem/11040673>
Comment 2 Tim Horton 2012-03-13 13:49:12 PDT
Created attachment 131708 [details]
patch to land failing expectations
Comment 3 Tim Horton 2012-03-13 13:50:14 PDT
TestFailures suggests it might be flaky; I've never seen it pass.
Comment 4 Tim Horton 2012-03-13 14:37:47 PDT
Landed failing results in http://trac.webkit.org/changeset/110612
Comment 6 Tim Horton 2012-03-13 18:40:19 PDT
(In reply to comment #5)
> This either passes under WKTR, or is just flakey:
> 
> http://build.webkit.org/results/Lion%20Intel%20Debug%20(WebKit2%20Tests)/r110645%20(4872)/css3/filters/custom/custom-filter-property-computed-style-pretty-diff.html

Passing with WKTR and not DRT would vaguely match the fact that it passes in Safari (with WK2) and not DRT.
Comment 7 Tim Horton 2012-03-13 18:42:28 PDT
(In reply to comment #6)
> (In reply to comment #5)
> > This either passes under WKTR, or is just flakey:
> > 
> > http://build.webkit.org/results/Lion%20Intel%20Debug%20(WebKit2%20Tests)/r110645%20(4872)/css3/filters/custom/custom-filter-property-computed-style-pretty-diff.html
> 
> Passing with WKTR and not DRT would vaguely match the fact that it passes in Safari (with WK2) and not DRT.

Nope, it passes in a WK1 Safari window too.
Comment 8 Jessie Berlin 2012-03-15 09:53:01 PDT
Landed expected passing results for mac-wk2 in http://trac.webkit.org/changeset/110855.