1423628 - Stop processing native events in the content process on Windows

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, enhancement, P1)

Description

[Bob Owen (:bobowen)]

Bug 1384336 caused some regressions, so this bug is to track getting native event processing turned off in the content process again.

As well as the existing benefits, this should mean that we can try and roll out the Alternate Desktop again for the Windows content process sandbox.
(The thing that is causing issues for that is certain Anti-Viruses causing the CreateWindow call for the hidden message-only window in nsAppShell to fail.
We don't need that Window if we're not pumping native events.)

Comment 1

[Bob Owen (:bobowen)]

Hi Bill, were there any other regressions that you know of apart from bugs 1395187 and 1396984?

Also, I wasn't sure if this bug needs to block bug 1350432?

Comment 2

[Bill McCloskey [inactive unless it's an emergency] (:billm)]

I think bug 1395536 is also a dependency. This bug does block the scheduling work. Thanks for looking at this, Bob.

Comment 3

[Bob Owen (:bobowen)]

Thanks Bill.

Comment 4

[Bob Owen (:bobowen)]

Attachment: Let not processing native events ride the trains on Windows

Comment 5

[Jim Mathies [:jimm]]

Comment on attachment 8954732 [details] [diff] [review]
Let not processing native events ride the trains on Windows

Review of attachment 8954732 [details] [diff] [review]:
-----------------------------------------------------------------

woot \o/

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/818464a2fb0e
Let not processing native events ride the trains on Windows. r=jimm

Comment 7

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/818464a2fb0e