Closed Bug 488911 Opened 11 years ago Closed 11 years ago

StretchDIBits takes too long - switch between optimizations at run time

Categories

(Core :: Widget: Win32, defect, critical)

All
Windows CE
defect
Not set
critical

Tracking

()

RESOLVED FIXED

People

(Reporter: blassey, Assigned: vlad)

References

()

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #484864 +++

We will have more than one code path for painting on windows, right now the code path is chosen at build time with preprocessor macros. It would be better to choose the path at run time based on prefs.
This depends on bug 488910

This patch adds a mozilla.widget.render-mode pref, that takes an int that corresponds to the new WinRenderMode enum.  It's only read in at start time, though we could add a pref observer to toggle it at runtime as well.

Most of this is just an indentation change, sadly; I can see if I can coax a diff -w patch out of hg if that would be easier to review.
Assignee: nobody → vladimir
Attachment #373749 - Flags: review?
Attachment #373749 - Flags: review? → review?(jmathies)
Hmm, can you post a updated patch? This didn't apply cleanly to m-c.
It won't, not without the patches in bug 488910.
Attachment #373749 - Flags: review?(jmathies) → review+
Comment on attachment 373749 [details] [diff] [review]
use pref to determine this at runtime

Alright, just eye balling it everything looked ok.
http://hg.mozilla.org/mozilla-central/rev/7fc81369ff6a
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.