662500 - Drop pref that forces OOP tabs

Status: RESOLVED FIXED

Description

[:Felipe Gomes (needinfo me!)]

Created attachment 537762 [details] [diff] [review]
Patch

A pref that forces oop tabs only breaks stuff (no code that is not expecting remote browsers will handle it correctly) and will never be supported, so we should drop it from the code. 

Also, I'd like to reuse this pref name as the pref that will switch the front end to use remote browsers, to nicely match the force-disable "dom.ipc.tabs.disabled" pref.
(any objections? dom.ipc.tabs.disabled is from libpref but dom.ipc.tabs.enabled will be in browser)

Every host app who wants to request remote browsers should use the remote="true" attribute.

(I made the changes in fennec-tile-testapp for correctness, but I don't think that's used anymore..)

Comment 1

[:Felipe Gomes (needinfo me!)]

oh actually dom.ipc.tabs.disabled is not from libpref, it's not defined anywhere.. even better

Comment 2

[:Felipe Gomes (needinfo me!)]

http://hg.mozilla.org/projects/electrolysis/rev/c68fab138909

Comment 3

[Marco Bonardo [::mak]]

I backed out everything from central since Android and Maemo were unhappy about the push these changes were part of.

Comment 4

[:Felipe Gomes (needinfo me!)]

http://hg.mozilla.org/mozilla-central/rev/faae99972d35