Bug 27727 - Transformations have CSS values of DOMCSSValueList, not DOMWebkitCSSTransformationValue
Summary: Transformations have CSS values of DOMCSSValueList, not DOMWebkitCSSTransform...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-07-27 11:39 PDT by Andy Matuschak
Modified: 2009-08-31 12:01 PDT (History)
3 users (show)

See Also:


Attachments
Test case demonstrating the problem (2.32 MB, application/octet-stream)
2009-07-27 11:39 PDT, Andy Matuschak
no flags Details
Patch (1.63 KB, patch)
2009-08-28 16:03 PDT, Simon Fraser (smfr)
mrowe: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Matuschak 2009-07-27 11:39:31 PDT
Created attachment 33561 [details]
Test case demonstrating the problem

Overview:
To parse a -webkit-transform property, the developer needs to be able to determine the types of transformations described within (translation, rotation, etc). There are definitions for an DOMWebkitCSSTransformationValue interface that would provide for this, but they're never actually provided by the DOM when queried.

I'm encountering this problem when using Obj-C bindings, so it's possible that when queried via JS, this doesn't happen.

Steps to reproduce:
1. Call getPropertyCSSValue("-webkit-transform") on an element with a transformation.
2. Examine the CSSValueList returned, note the type of each item.

Actual results:
The items are CSSValueLists.

Expected results:
The items should be WebkitCSSTransformationValues.

See attached test case.
Comment 1 Simon Fraser (smfr) 2009-07-27 12:27:02 PDT
kitClass(WebCore::CSSValue* impl) in DOMCSS.mm needs to be educated about different types of CSSValueLists, but I'm not sure if we should expand the list of CSSValue types in CSSValue.idl. It looks like this same problem would occur with CSSTimingFunctionValue, CSSUnicodeRangeValue, CSSBorderImageValue, CSSFontFaceSrcValue, CSSFunctionValue, CSSImageGeneratorValue, CSSReflectValue etc etc.
Comment 2 Simon Fraser (smfr) 2009-07-28 11:51:52 PDT
Dave Hyatt says to add a new CSSValueType for CSSTransformationValue
Comment 3 Simon Fraser (smfr) 2009-08-28 16:03:39 PDT
Created attachment 38760 [details]
Patch
Comment 4 Mark Rowe (bdash) 2009-08-28 16:07:14 PDT
Comment on attachment 38760 [details]
Patch

r=me
Comment 5 Simon Fraser (smfr) 2009-08-28 16:08:24 PDT
http://trac.webkit.org/changeset/47877
Comment 6 Simon Fraser (smfr) 2009-08-31 12:01:39 PDT
Test added in http://trac.webkit.org/changeset/47901