|Summary:||WebKitCSSTransformValue needs DOM interface|
|Product:||WebKit||Reporter:||Dean Jackson <dino>|
|Version:||528+ (Nightly build)|
|Bug Depends on:||19861|
|Bug Blocks:||19028, 19864|
Description Dean Jackson 2008-07-02 18:40:58 PDT
Comment 1 Dean Jackson 2008-07-02 18:41:41 PDT
Created attachment 22056 [details] Adds IDL and updates projects
Comment 2 Eric Seidel (no email) 2008-07-04 11:19:00 PDT
Comment on attachment 22056 [details] Adds IDL and updates projects Why the WebKit prefix on the JS class name? I've never seen us do that before...
Comment 3 Dave Hyatt 2008-07-04 12:41:03 PDT
Because we expect that it will change significantly. Doing will avoid name clashes and let us make changes that won't break backwards compatibility with the phone. We should have done this with <canvas>... it would have saved us a lot of headaches.
Comment 4 Simon Fraser (smfr) 2008-07-07 20:25:03 PDT
+ // OperationTypes + const unsigned short CSS_MATRIX = 1; + const unsigned short CSS_ROTATE = 3; + const unsigned short CSS_TRANSLATE = 5; + const unsigned short CSS_SCALE = 7; + const unsigned short CSS_SKEWX = 9; + const unsigned short CSS_SKEWY = 10; + const unsigned short CSS_TRANSLATEX = 16; + const unsigned short CSS_TRANSLATEY = 17; + const unsigned short CSS_SCALEX = 19; + const unsigned short CSS_SCALEY = 20; I think a comment is required here to warn people from making the values contiguous.
Comment 5 Dave Hyatt 2008-07-08 11:14:42 PDT
Comment on attachment 22056 [details] Adds IDL and updates projects r=me
Comment 6 Dean Jackson 2008-07-08 15:09:56 PDT
I'll explain why there are gaps in the enum.
Comment 7 Dean Jackson 2008-07-09 18:03:55 PDT
Done in 35084 http://trac.webkit.org/changeset/35084