The default bug view has changed. See this FAQ.

Set some preferences for e10s builds

RESOLVED FIXED in Firefox 8

Status

()

Firefox
Preferences
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Felipe, Assigned: Felipe)

Tracking

Trunk
Firefox 8
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [e10s])

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
Created attachment 538184 [details] [diff] [review]
Patch

Make builds built with --enable-e10s-compat to turn on e10s by default and disable hardware acceleration
Attachment #538184 - Flags: review?(dolske)
(Assignee)

Comment 1

6 years ago
Created attachment 538189 [details] [diff] [review]
Use pref

(Was gonna post this in a separate bug but it seems unnecessary)

Uses the pref dom.ipc.tabs.enabled to create new tabs in the remote process.

This won't cover the first tab in the browser as the <browser> element is hard-coded in tabbrowser.. we'll need support to switch the remote="true" on/off for that

(and I think it's a good idea to leave the first tab as non-remote for the time being)
Attachment #538189 - Flags: review?(dolske)
Comment on attachment 538184 [details] [diff] [review]
Patch

Review of attachment 538184 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with addressing the one comment either way.

::: browser/app/profile/firefox.js
@@ +905,5 @@
> +#ifdef MOZ_E10S_COMPAT
> +pref("dom.ipc.tabs.enabled", true);
> +#else
> +pref("dom.ipc.tabs.enabled", false);
> +#endif

Hmm. I'm inclined to say that this pref should remain hidden, since we're still a ways from supporting Firefox in E10S. When we're farther along and want people to start testing and experimenting, then we can include it (still defaulted to false).

Alternatively, I think I'd be ok with only exposing the pref on Nightly builds.
Attachment #538184 - Flags: review?(dolske) → review+
Comment on attachment 538189 [details] [diff] [review]
Use pref

Conversely to the above, this might as well be done without the #ifdef. If someone sets this pref, better to have stuff break right away than silently work unchanged until the day we un-ifdef this.
Attachment #538189 - Flags: review?(dolske) → review+
I backed out everything from central since Android and Maemo were unhappy about the push these changes were part of.
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/mozilla-central/rev/b12c57161b96
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 8
You need to log in before you can comment on or make changes to this bug.