Closed
Bug 721136
Opened 13 years ago
Closed 13 years ago
transform property serialization doesn't normalize case
Categories
(Core :: DOM: CSS Object Model, defect)
Tracking
()
RESOLVED
FIXED
mozilla13
People
(Reporter: ayg, Assigned: ayg)
Details
Attachments
(1 file, 1 obsolete file)
|
4.88 KB,
patch
|
ayg
:
review+
|
Details | Diff | Splinter Review |
data:text/html,<!DOCTYPE html>
<script>
document.head.style.MozTransform = 'RotatE( 3rAd )scalex(2) translateX(10%)';
document.documentElement.textContent =
document.head.style.MozTransform
</script>
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:
https://www.w3.org/Bugs/Public/show_bug.cgi?id=15710
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.
We'd probably want to fix this in either CSSParserImpl::ParseFunction or CSSParserImpl::ParseSingleTransform.
|
Assignee | |
Comment 2•13 years ago
|
||
Dunno if this is the right way to do it, but it works for me.
Assignee: nobody → ayg
Attachment #596754 -
Flags: review?(dbaron)
|
|
Assignee | |
Updated•13 years ago
|
Whiteboard: [autoland]
|
||
Updated•13 years ago
|
Whiteboard: [autoland] → [autoland-in-queue]
|
|
||
Comment 3•13 years ago
|
||
Autoland Patchset:
Patches: 596754
Branch: mozilla-central => try
Destination: http://hg.mozilla.org/try/pushloghtml?changeset=abb8726f04a7
Try run started, revision abb8726f04a7. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=abb8726f04a7
Comment on attachment 596754 [details] [diff] [review]
Patch v1
>+ default:
>+ ;
Make this "break;" instead of ";".
r=dbaron with that
Attachment #596754 -
Flags: review?(dbaron) → review+
|
|
Assignee | |
Comment 5•13 years ago
|
||
Attachment #596754 -
Attachment is obsolete: true
Attachment #596775 -
Flags: review+
|
|
||
Comment 6•13 years ago
|
||
Try run for abb8726f04a7 is complete.
Detailed breakdown of the results available here:
https://tbpl.mozilla.org/?tree=Try&rev=abb8726f04a7
Results (out of 211 total builds):
exception: 2
success: 175
warnings: 20
failure: 14
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/autolanduser@mozilla.com-abb8726f04a7
|
|
||
Updated•13 years ago
|
Whiteboard: [autoland-in-queue]
|
|
Assignee | |
Updated•13 years ago
|
Keywords: checkin-needed
|
||
Comment 7•13 years ago
|
||
|
||
Comment 8•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•