1154739 - On Desktop, only enable APZ for E10S windows

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[David Anderson [:dvander] - inactive, e-mail if emergency]

None of the Desktop platforms receive input off the main thread, so they effectively require E10S. That may change for some platforms, but not right now.

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e944fc5e8702

If this sticks I'll follow up with a backout after the next m-c merge.

Comment 2

[David Anderson [:dvander] - inactive, e-mail if emergency]

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/ffde08dc5ceb

Comment 3

[David Anderson [:dvander] - inactive, e-mail if emergency]

(And do a m.d.platform post)

Comment 4

[Kartikaya Gupta (email:[e-mail])]

Attachment: Part 1 - Rename a flag

Comment 5

[Kartikaya Gupta (email:[e-mail])]

Attachment: Part 2 - Disable APZ on non-e10s windows

I need to test these patches on all platforms still.

Comment 6

[Kartikaya Gupta (email:[e-mail])]

Attachment: Part 2 - Disable APZ on non-e10s windows on desktop

Comment 7

[Kartikaya Gupta (email:[e-mail])]

I tested this on OSX, Linux, and Windows, with e10s enabled and disabled by default, using "open new e10s window" and "open new non-e10s window". Works as intended (i.e. APZ enabled in all e10s windows but not otherwise).

Comment 8

[Markus Stange [:mstange]]

Comparing the dxr search results of "electrolysis" and "e10s" seems to indicate that e10s is the preferred name.

Comment 9

[Kartikaya Gupta (email:[e-mail])]

I'm always unsure how to camelcase e10s :)

Comment 10

[Fusion Future]

Hello.
apz.allow_checkerboarding option fails to work on 0423 and 0424 Nightly builds, which causes the smooth scroll very laggy.

Comment 11

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

Comment on attachment 8596761 [details] [diff] [review]
Part 1 - Rename a flag

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

Let's call it mMultiProcessWindow. We use gMultiProcessBrowser for similar things in the frontend. Also, I don't like to use codenames in source code.

Comment 12

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/666f287e0dab
https://hg.mozilla.org/integration/mozilla-inbound/rev/91110ad64d32

Comment 13

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/666f287e0dab
https://hg.mozilla.org/mozilla-central/rev/91110ad64d32

Comment 14

[Maksim Lebedev]

I am really doubt about decision of this bug.
Why it was needed? As result we have bug 1162009.