Closed
Bug 340028
Opened 18 years ago
Closed 18 years ago
Rendering problem in <svg:image> on intel macs
Categories
(Core :: SVG, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: ahayes, Assigned: tor)
References
()
Details
(Keywords: fixed1.8.1, verified1.8.0.5)
Attachments
(4 files)
997 bytes,
patch
|
Details | Diff | Splinter Review | |
139.27 KB,
image/jpeg
|
Details | |
125.26 KB,
image/jpeg
|
Details | |
2.48 KB,
patch
|
jhpedemonte
:
review+
shaver
:
approval-branch-1.8.1+
dveditz
:
approval1.8.0.5+
|
Details | Diff | Splinter Review |
When an image is loaded via <svg:image>, it is not rendered correctly. <http://www.croczilla.com/svg/samples/circles2/circles2.xml>
Reporter | ||
Comment 1•18 years ago
|
||
Sorry. I should have explicitly mentioned in the description that I am refering to a problem on intel macs and have seen the problem on the following browsers when running without rosetta: Camino Version 2006060104 (1.0+) Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.0.4) Gecko/20060508 Firefox/1.5.0.4 Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1a3) Gecko/20060526 BonEcho/2.0a3
Comment 2•18 years ago
|
||
This bug is a followup to bug 327744.
Reporter | ||
Comment 3•18 years ago
|
||
Adding Jay and Josh since they were also involved in sorting out bug 327744.
Comment 4•18 years ago
|
||
Amos: Did you mean to cc another Jay? Because I don't think I'll be much help here (I'm the QA/Talkback guy). ;-)
Reporter | ||
Comment 5•18 years ago
|
||
Pleased to meet you Jay. I'm a relatively new bugzilla participant. My interest in some parts of Firefox stems from having a lab of intel macs & people who are trying to build an online atlas that uses Firefox SVG quite heavily. :) Sorry about the CC out of the blue. There are a couple of these mac&intel&svg bugs left and since you seemed to be involved in triage on bug 327744, I thought maybe you'd want to see this one as well. I'd love to see all the endian bugs solved for some 1.5.0.x version of Firefox in the not too distant future. Unfortunately, my capacity at the moment is to log bugs and produce testcases. Anyway, if there is interest/capacity to automate SVG rendering tests, I'd be happy to help create a few SVG files for a testsuite.
Attachment #225022 -
Attachment description: untested patch for os-x intel svg:image → untested branch patch for os-x intel svg:image
Not pretty, but I'd prefer to do the minimal amount of change to the code logic on the branch.
Assignee: general → tor
Status: NEW → ASSIGNED
Attachment #225289 -
Flags: review?(jhpedemonte)
Updated•18 years ago
|
Attachment #225289 -
Flags: review?(jhpedemonte) → review+
Reporter | ||
Comment 10•18 years ago
|
||
Thanks for the new patch tor. I have tested this patch on the MOZILLA_1_8_0 branch using the universal build instructions at: http://developer.mozilla.org/en/docs/Mac_OS_X_Universal_Binaries plus the following additions to the .mozconfig: ac_add_options --enable-canvas ac_add_options --enable-svg and confirm that the circles2.xml crockzilla example now renders properly on intel mac. I also tried setting my build to "Open using Rosetta" and confirm that the circles2.xml test case still works there too.
Flags: blocking1.8.1?
Flags: blocking1.8.0.5?
Comment 11•18 years ago
|
||
Please get landed on trunk and 1.8 branch (after module owner approval, which I guess could be tor himself) for verification, then ask for 1.8.0.5 approval. Code freeze is approaching fast...
Flags: blocking1.8.0.5? → blocking1.8.0.5+
Updated•18 years ago
|
Flags: blocking1.8.1? → blocking1.8.1+
Attachment #225289 -
Flags: approval1.8.0.5?
Attachment #225289 -
Flags: approval-branch-1.8.1?(shaver)
Comment on attachment 225289 [details] [diff] [review] tested branch intel os-x svg:image patch 181=shaver
Attachment #225289 -
Flags: approval-branch-1.8.1?(shaver) → approval-branch-1.8.1+
Comment 14•18 years ago
|
||
Comment on attachment 225289 [details] [diff] [review] tested branch intel os-x svg:image patch approved for 1.8.0 branch, a=dveditz for drivers
Attachment #225289 -
Flags: approval1.8.0.5? → approval1.8.0.5+
Assignee | ||
Comment 16•18 years ago
|
||
Checked in on trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment 17•18 years ago
|
||
Verified using test images provided, on: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.0.5) Gecko/20060619 Firefox/1.5.0.5 Macbook Pro, MacOS X 10.4.6 Image is now rendered as it is in the Wintel version.
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.0.5 → verified1.8.0.5
You need to log in
before you can comment on or make changes to this bug.
Description
•