1309599 - Update e10s related nsAppRunner code for current RTL support

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect)

Description

[Jim Mathies [:jimm]]

Need to do a checkup on the system add-on to determine how it handles RTL decisions. Thew following releases should be targeted:

RTL/e10s platform support:

WIN = 48
OSX = 50 
LINUX = 52

Felipe, do we need to make some changes here or are we good? I'm guessing linux needs to be added to the list since that bug just closed out.

RTL bugs:
Bug 1033483 - [e10s][RTL] Do bidi state detection in the parent process on Linux
Bug 1033488 - [e10s][RTL] Do bidi state detection in the parent process on Windows
Bug 1277831 - Enable e10s for Windows and OSX RTL users

Comment 1

[:Felipe Gomes (needinfo for replies!)]

The RTL decision is entirely made by nsAppRunner.cpp. So you just need to make sure that the block of code from bug 1277831 is correctly existent or not on each version. FWIW, my understanding was that both Win and OSX were supported from version 50.  (and not 48 for Win as you mentioned)

Comment 2

[Jim Mathies [:jimm]]

(In reply to :Felipe Gomes (needinfo me!) from comment #1)
> The RTL decision is entirely made by nsAppRunner.cpp. So you just need to
> make sure that the block of code from bug 1277831 is correctly existent or
> not on each version. FWIW, my understanding was that both Win and OSX were
> supported from version 50.  (and not 48 for Win as you mentioned)

Thanks. I'll get this updated here.

Comment 3

[Jim Mathies [:jimm]]

Attachment: patch

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee3fbfa087a5
Turn e10s on for RTL linux users. r=felipe

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/ee3fbfa087a5