Last Comment Bug 721136 - transform property serialization doesn't normalize case
: transform property serialization doesn't normalize case
Product: Core
Classification: Components
Component: DOM: CSS Object Model (show other bugs)
: Trunk
: x86 Linux
: -- minor (vote)
: mozilla13
Assigned To: :Aryeh Gregor (away until August 15)
Depends on:
  Show dependency treegraph
Reported: 2012-01-25 11:23 PST by :Aryeh Gregor (away until August 15)
Modified: 2012-02-16 03:02 PST (History)
2 users (show)
dao+bmo: in‑testsuite+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Patch v1 (4.87 KB, patch)
2012-02-13 12:41 PST, :Aryeh Gregor (away until August 15)
dbaron: review+
Details | Diff | Review
Patch v2 (4.88 KB, patch)
2012-02-13 13:23 PST, :Aryeh Gregor (away until August 15)
ayg: review+
Details | Diff | Review

Description :Aryeh Gregor (away until August 15) 2012-01-25 11:23:40 PST
data:text/html,<!DOCTYPE html>
<script> = 'RotatE( 3rAd )scalex(2) translateX(10%)';
document.documentElement.textContent =

Results (adjusted for prefixes):

IE9, Chrome 17 dev: rotate(3rad) scaleX(2) translateX(10%)
Firefox 12.0a1: RotatE(3rad) scalex(2) translateX(10%)
Opera Next 12.00 alpha: rotate(171.89deg) scaleX(2) translateX(10%)

Behavior here is not currently specified, and I filed a spec bug on it:

I recommended the IE/Chrome behavior, since it makes the most sense.  Either the input should be returned as-is with no normalization, or everything should be normalized, including function names' case.  But all browsers but Opera agree units shouldn't be mangled.
Comment 1 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2012-01-25 11:45:33 PST
We'd probably want to fix this in either CSSParserImpl::ParseFunction or CSSParserImpl::ParseSingleTransform.
Comment 2 :Aryeh Gregor (away until August 15) 2012-02-13 12:41:07 PST
Created attachment 596754 [details] [diff] [review]
Patch v1

Dunno if this is the right way to do it, but it works for me.
Comment 3 Mozilla RelEng Bot 2012-02-13 12:45:00 PST
Autoland Patchset:
	Patches: 596754
	Branch: mozilla-central => try
Try run started, revision abb8726f04a7. To cancel or monitor the job, see:
Comment 4 David Baron :dbaron: ⌚️UTC-7 (review requests must explain patch) 2012-02-13 13:02:27 PST
Comment on attachment 596754 [details] [diff] [review]
Patch v1

>+    default:
>+      ;

Make this "break;" instead of ";".

r=dbaron with that
Comment 5 :Aryeh Gregor (away until August 15) 2012-02-13 13:23:38 PST
Created attachment 596775 [details] [diff] [review]
Patch v2
Comment 6 Mozilla RelEng Bot 2012-02-13 22:16:00 PST
Try run for abb8726f04a7 is complete.
Detailed breakdown of the results available here:
Results (out of 211 total builds):
    exception: 2
    success: 175
    warnings: 20
    failure: 14
Builds (or logs if builds failed) available at:
Comment 8 Marco Bonardo [::mak] 2012-02-16 03:02:58 PST

Note You need to log in before you can comment on or make changes to this bug.