Closed Bug 355179 Opened 13 years ago Closed 13 years ago

Vista - Sets the current process as dpi aware

Categories

(Core :: Widget, defect)

x86
Windows Vista
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dougt, Assigned: dougt)

References

Details

(Keywords: fixed1.8.1.1)

Attachments

(1 file, 1 obsolete file)

On vista, there is the ability to change the dpi used.  If we do not mark the application as DPI aware, when going into this mode Firefox text looks blurry.

Vista provides an API to mark the application as DPI aware.  We should use it:

http://msdn.microsoft.com/library/default.asp?url=/library/en-us/winui/winui/windowsuserinterface/windowing/windows/windowreference/windowfunctions/setprocessdpiaware.asp
Blocks: 352420
Attached patch patch v.1 (obsolete) — Splinter Review
Comment on attachment 241012 [details] [diff] [review]
patch v.1

this patch isn't something to checkin.
Attachment #241012 - Attachment is obsolete: true
Attached patch patch v.2Splinter Review
fixed cast.  wondering if we can avoid LoadLibrary.  it is fine to do it this way, but I remember there exists way to avoid the LoadLibrary when the library is implicitly linked.
Attachment #241017 - Flags: review?
Comment on attachment 241017 [details] [diff] [review]
patch v.2

r=vlad.
Attachment #241017 - Flags: review? → review+
on the trunk for a while.  safe fix.
Flags: blocking1.8.1.1?
Checking in widget/src/windows/nsToolkit.cpp;
/cvsroot/mozilla/widget/src/windows/nsToolkit.cpp,v  <--  nsToolkit.cpp
new revision: 3.69; previous revision: 3.68
done

i thought i checked this in a while ago, turns out I didn't.  It has been in my tree for a while.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Flags: blocking1.8.1.1? → blocking1.8.1.1+
Checking in nsToolkit.cpp;
/cvsroot/mozilla/widget/src/windows/nsToolkit.cpp,v  <--  nsToolkit.cpp
new revision: 3.48.4.8; previous revision: 3.48.4.7
done
Keywords: fixed1.8.1.1
You need to log in before you can comment on or make changes to this bug.