1750106 - Use default member initialization in nsWindow where appropriate on Windows

Status: RESOLVED FIXED

(Core :: Widget: Win32, task, P3)

Description

[David Parks [:handyman]]

We should shift nsWindow from a constructor with a long list of lines initializing member variables to using default member initialization.

Comment 1

[Chris Martin [:cmartin]]

Attachment: Bug 1750106 - Use static initializers in Win32 nsWindow constructor

Also, just generally bring Win32 nsWindow more in line with our coding
standards.

Comment 2

[Chris Martin [:cmartin]]

Attachment: Bug 1750106 - Use default and list initialization for Win32 nsWindow

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0a3a3747998b
Use default and list initialization for Win32 nsWindow r=handyman

Comment 4

[Cristian Tuns]

Backed out for causing multiple failures nsWindow.cpp

• Backout link
• Push with failures
• Failure Log
• Failure line: Assertion failure: !mParent, at /builds/worker/checkouts/gecko/widget/windows/nsWindow.cpp:7390
• And also other failures on windows. https://treeherder.mozilla.org/logviewer?job_id=369126449&repo=autoland

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6012e2664c8b
Use default and list initialization for Win32 nsWindow r=handyman

Comment 6

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/6012e2664c8b