Closed Bug 549668 Opened 15 years ago Closed 15 years ago

mozilla.widget.render-mode is not handled correctly

Categories

(Core :: Graphics, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Tracking Status
blocking1.9.2 --- .2+
status1.9.2 --- .2-fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

Details

Attachments

(1 file)

The code in gfxWindowsPlatform to handle the render-mode preference does not correctly check that the preference value is valid. This results in complete failure of rendering (blank windows) if the value is out of range. STR: use a current Minefield trunk build and set mozilla.widget.render-mode to 6, to experiment with Direct2D rendering. Quit Minefield, and launch Firefox 3.6 with the same profile. Windows will be completely blank, because Gecko 1.9.2 doesn't recognize render-mode 6. Looks like the problem is the use of || instead of && when checking that the value is in range.
Attachment #429786 - Attachment is patch: true
Attachment #429786 - Attachment mime type: application/octet-stream → text/plain
Attachment #429786 - Flags: review?(vladimir)
Comment on attachment 429786 [details] [diff] [review] correctly check the validity of the render-mode value Indeed.
Attachment #429786 - Flags: review?(vladimir) → review+
Blocks: 549272
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee: nobody → jfkthame
Comment on attachment 429786 [details] [diff] [review] correctly check the validity of the render-mode value Requesting approval for 1.9.2.2 - this is a simple bugfix with no risk, and may be critical to users who experiment with D2D using 3.7-alpha or trunk builds and then go back to running 3.6.
Attachment #429786 - Flags: approval1.9.2.2?
Comment on attachment 429786 [details] [diff] [review] correctly check the validity of the render-mode value a=beltzner for 1.9.2.2; jdrew tells me that without this, people who test Direct2D builds will be boned if they switch back to 1.9.2.2 or 1.9.1.9. Joe said he'd even check it in.
Attachment #429786 - Flags: approval1.9.2.2?
Attachment #429786 - Flags: approval1.9.2.2+
Attachment #429786 - Flags: approval1.9.1.9+
Comment on attachment 429786 [details] [diff] [review] correctly check the validity of the render-mode value This pref is not present on the 1.9.1 branch.
Attachment #429786 - Flags: approval1.9.1.9+
blocking1.9.2: --- → ?
blocking1.9.2: ? → .2+
verified FIXED using the steps to reproduce in comment 0 ... from build: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre) Gecko/20100322 Minefield/3.7a4pre to Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.2) Gecko/20100316 Firefox/3.6.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: