Closed Bug 834701 Opened 12 years ago Closed 11 years ago

Remove the XBL bit

Categories

(Core :: XPConnect, defect)

Product:
Core
Component:
XPConnect
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla24

People

(Reporter: bholley, Assigned: bholley)

References

Details

Attachments

(3 files, 1 obsolete file)

Part 1 - Stop depending on the XBL bit in IsCallerXBL. v1
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Part 2 - Stop setting the XBL script in the browser. v1
(deleted), patch
bzbarsky
: review+
Details | Diff | Splinter Review
Part 3 - Remove the XBL bit from spidermonkey. v1
(deleted), patch
luke
: review+
Details | Diff | Splinter Review
Once XBL scopes are no longer behind a pref, we'll no longer have a need for the XBL bit on JSScripts, and can remove it.
Attached patch Kill the xbl bit. v1 (obsolete) (deleted) — — Splinter Review
This will probably need some rebasing once the time comes, but most of the work is done here.
Comment on attachment 706374 [details] [diff] [review] Kill the xbl bit. v1 https://tbpl.mozilla.org/?tree=Try&rev=f44eeb0fb385
Attachment #706374 - Attachment is obsolete: true
Attachment #751809 - Flags: review?(bzbarsky)
Attachment #751810 - Flags: review?(luke)
Attachment #751811 - Flags: review?(luke)
Comment on attachment 751810 [details] [diff] [review] Part 2 - Stop setting the XBL script in the browser. v1 Er, this.
Attachment #751810 - Flags: review?(luke) → review?(bzbarsky)
Comment on attachment 751811 [details] [diff] [review] Part 3 - Remove the XBL bit from spidermonkey. v1 I never knew thee.
Attachment #751811 - Flags: review?(luke) → review+
green on try.
Comment on attachment 751809 [details] [diff] [review] Part 1 - Stop depending on the XBL bit in IsCallerXBL. v1 r=me
Attachment #751809 - Flags: review?(bzbarsky) → review+
Comment on attachment 751810 [details] [diff] [review] Part 2 - Stop setting the XBL script in the browser. v1 r=me
Attachment #751810 - Flags: review?(bzbarsky) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: