Closed Bug 869891 Opened 8 years ago Closed 8 years ago

seamonkey should declare itself dpiAware on windows via the .exe manifest

Categories

(SeaMonkey :: OS Integration, defect)

x86
Windows 7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.20

People

(Reporter: jfkthame, Unassigned)

References

Details

Attachments

(1 file)

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

In bug 866365, we removed the call to SetProcessDPIAware from XRE_mainInit(), and instead added the dpiAware element to firefox.exe's manifest. (This is the MS recommended way for an application to declare itself dpi-aware, as per MSDN documentation.)

So to maintain its existing behavior on hi-dpi windows systems, seamonkey should make the same manifest addition. Bug 866365 is likely to be uplifted to mozilla-22, I think, so that's also where the manifest change should be made.
Comment on attachment 746894 [details] [diff] [review]
declare seamonkey.exe as dpi-aware in its manifest

Thanks for porting this!
Attachment #746894 - Flags: review?(neil) → review+
https://hg.mozilla.org/comm-central/rev/e3d49393a3b2

I'm not sure what seamonkey's release schedule is like, or how the version numbers relate to gecko versions, but if there's going to be a release based on gecko 22, you'll probably want to backport this.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
(In reply to Jonathan Kew (:jfkthame) from comment #3)
> https://hg.mozilla.org/comm-central/rev/e3d49393a3b2
> 
> I'm not sure what seamonkey's release schedule is like

Same as FF's.

> or how the version numbers relate to gecko versions

Current SM versions are 2.<FF/Gecko version - 3>, so Gecko 22 matches SM 2.19.

> but if there's going to be a release based
> on gecko 22, you'll probably want to backport this.

I guess so.
Comment on attachment 746894 [details] [diff] [review]
declare seamonkey.exe as dpi-aware in its manifest

[Approval Request Comment]
See bug 866365 comment 46. As of now, approval-mozilla-aurora is pending.
Attachment #746894 - Flags: approval-comm-aurora?
Comment on attachment 746894 [details] [diff] [review]
declare seamonkey.exe as dpi-aware in its manifest

> See bug 866365 comment 46. As of now, approval-mozilla-aurora is pending.
Conditional a=me based on 866365 landing on mozilla-aurora
Attachment #746894 - Flags: approval-comm-aurora? → approval-comm-aurora+
Target Milestone: --- → seamonkey2.20
You need to log in before you can comment on or make changes to this bug.