Closed Bug 1156515 Opened 5 years ago Closed 5 years ago

Remove "New e10s window" option

Categories

(Firefox :: General, defect)

34 Branch
x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 40
Tracking Status
e10s + ---
firefox40 --- fixed

People

(Reporter: billm, Assigned: billm)

Details

Attachments

(1 file)

If we remove this option, then the only way to get e10s is to enable it globally. That simplifies some things. It means that BrowserTabsRemoteAutostart() is a conservative check for whether e10s is in use.
Assignee: nobody → wmccloskey
Comment on attachment 8598986 [details] [diff] [review]
patch

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

::: browser/base/content/browser-menubar.inc
@@ -30,5 @@
>  #ifdef E10S_TESTING_ONLY
> -                <menuitem id="menu_newRemoteWindow"
> -                          label="New e10s Window"
> -                          hidden="true"
> -                          command="Tools:RemoteWindow"/>

should also remove the Tools:RemoteWindow command, http://mxr.mozilla.org/mozilla-central/source/browser/base/content/browser-sets.inc#117
Attachment #8598986 - Flags: review?(felipc) → review+
https://hg.mozilla.org/mozilla-central/rev/d10105a75835
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
You need to log in before you can comment on or make changes to this bug.