Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Drop pref that forces OOP tabs

RESOLVED FIXED in mozilla8

Status

()

Core
Document Navigation
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: Felipe, Assigned: Felipe)

Tracking

unspecified
mozilla8
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [e10s])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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..)
Attachment #537762 - Flags: review?(benjamin)
(Assignee)

Comment 1

6 years ago
oh actually dom.ipc.tabs.disabled is not from libpref, it's not defined anywhere.. even better
Attachment #537762 - Flags: review?(benjamin) → review+
(Assignee)

Comment 2

6 years ago
http://hg.mozilla.org/projects/electrolysis/rev/c68fab138909
Status: ASSIGNED → RESOLVED
Last Resolved: 6 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 → ---
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/mozilla-central/rev/faae99972d35
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 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.