Closed
Bug 662500
Opened 14 years ago
Closed 14 years ago
Drop pref that forces OOP tabs
Categories
(Core :: DOM: Navigation, defect)
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: Felipe, Assigned: Felipe)
Details
(Whiteboard: [e10s])
Attachments
(1 file)
|
3.57 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter 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)
|
Assignee | |
Comment 1•14 years ago
|
||
oh actually dom.ipc.tabs.disabled is not from libpref, it's not defined anywhere.. even better
|
||
Updated•14 years ago
|
Attachment #537762 -
Flags: review?(benjamin) → review+
|
|
Assignee | |
Comment 2•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [e10s] → [e10s][landed in e10s]
|
||
Comment 3•14 years ago
|
||
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•14 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 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.
Description
•