Closed Bug 1348177 Opened 3 years ago Closed 3 years ago

Remove GetProcAddress for shell32 since we support Windows 7+

Categories

(Core :: General, enhancement)

55 Branch
Unspecified
Windows
enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: m_kato, Assigned: m_kato)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

For compatibility of Windows XP, we use GetProcAddress to use Vista or 7+ API.  But now, we drop XP and Vista support, so we remove it.
Blocks: xp-eol
Attachment #8848378 - Flags: review?(jmathies)
Comment on attachment 8848378 [details]
Bug 1348177 - Part 1. Remove GetProcAddress for shell32 to use Vista+ API (image part).

https://reviewboard.mozilla.org/r/121276/#review123316
Attachment #8848378 - Flags: review?(tnikkel) → review+
Comment on attachment 8848378 [details]
Bug 1348177 - Part 1. Remove GetProcAddress for shell32 to use Vista+ API (image part).

https://reviewboard.mozilla.org/r/121276/#review126788
Attachment #8848378 - Flags: review?(jmathies) → review+
Comment on attachment 8848379 [details]
Bug 1348177 - Part 2. Remove GetProcAddress for shell32 to use Vista+/7+ API (tookit part).

https://reviewboard.mozilla.org/r/121278/#review126790
Attachment #8848379 - Flags: review?(jmathies) → review+
Comment on attachment 8848380 [details]
Bug 1348177 - Part 3. Remove GetProcAddress for shell32 to use Vista+/7+ API (widget part).

https://reviewboard.mozilla.org/r/121280/#review126792
Attachment #8848380 - Flags: review?(jmathies) → review+
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/a4c591fb03fe
Part 1. Remove GetProcAddress for shell32 to use Vista+ API (image part). r=jimm,tnikkel
https://hg.mozilla.org/integration/autoland/rev/c8c7fbbee8a8
Part 2. Remove GetProcAddress for shell32 to use Vista+/7+ API (tookit part). r=jimm
https://hg.mozilla.org/integration/autoland/rev/134a3c1bf3d1
Part 3. Remove GetProcAddress for shell32 to use Vista+/7+ API (widget part). r=jimm
You need to log in before you can comment on or make changes to this bug.