Scaling goes into inifnite loop

VERIFIED FIXED in mozilla0.9.8

Status

()

Core
Printing: Output
--
critical
VERIFIED FIXED
17 years ago
16 years ago

People

(Reporter: rods (gone), Assigned: dcone (gone))

Tracking

({hang})

Trunk
mozilla0.9.8
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

1.97 KB, patch
dcone (gone)
: review+
Marc Attinasi
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

17 years ago
Steps to reproduce:
1) edit pagesetupdialog.xul
2) enable the scaling UI controls
3) rebuild xpfe/communicator directory
4) run mozilla
5) open Page Setup
6) set scaling to 70 or 80 %
7) Print Preview the mozilla page 
8) it goes into an inifnite loop building the tile

nsImageWin::BuildTile(H
(Reporter)

Comment 1

17 years ago
THis blocks turning on scaling
Severity: normal → blocker
Target Milestone: --- → mozilla0.9.9
(Reporter)

Comment 2

17 years ago
If I back out your change from 1/9 nsImageWin.cpp 3.84 it works fine.
(Reporter)

Updated

17 years ago
Blocks: 120075
(Reporter)

Updated

17 years ago
Target Milestone: mozilla0.9.9 → mozilla0.9.8
(Reporter)

Comment 3

17 years ago
Created attachment 65060 [details] [diff] [review]
patch

fixes a couple of var names and rounding error
(Assignee)

Comment 4

17 years ago
Comment on attachment 65060 [details] [diff] [review]
patch

r=dcone.  This works.. me and Rod worked on and tested this fix.
Attachment #65060 - Flags: review+

Comment 5

17 years ago
Comment on attachment 65060 [details] [diff] [review]
patch

sr=attinasi
Attachment #65060 - Flags: superreview+

Updated

17 years ago
Severity: blocker → critical
Keywords: hang
(Reporter)

Comment 6

17 years ago
fixed
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 7

16 years ago
verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.