Closed Bug 662500 Opened 14 years ago Closed 14 years ago

Drop pref that forces OOP tabs

Categories

(Core :: DOM: Navigation, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: Felipe, Assigned: Felipe)

Details

(Whiteboard: [e10s])

Attachments

(1 file)

Attached patch PatchSplinter Review
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..)
Attachment #537762 - Flags: review?(benjamin)
oh actually dom.ipc.tabs.disabled is not from libpref, it's not defined anywhere.. even better
Attachment #537762 - Flags: review?(benjamin) → review+
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [e10s] → [e10s][landed in e10s]
I backed out everything from central since Android and Maemo were unhappy about the push these changes were part of.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
Resolution: --- → FIXED
Whiteboard: [e10s][landed in e10s] → [e10s]
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: