Remove XBL scope pref

RESOLVED FIXED in mozilla23

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bholley, Assigned: bholley)

Tracking

unspecified
mozilla23
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
Once we enable XBL scopes in bug 834697 and wait long enough to be confident in them, we'll want to remove the pref machinery to allow for various refactoring and cleanup.
(Assignee)

Updated

6 years ago
Blocks: 834701
(Assignee)

Updated

6 years ago
Blocks: 834707
(Assignee)

Updated

6 years ago
Blocks: 825392
(Assignee)

Updated

6 years ago
Blocks: 857356
(Assignee)

Comment 1

6 years ago
Created attachment 744201 [details] [diff] [review]
Stop checking dom.xbl_scopes. v1
(Assignee)

Comment 2

6 years ago
Created attachment 744202 [details] [diff] [review]
Fix tests that munge dom.xbl_scopes. v1
(Assignee)

Updated

6 years ago
Attachment #744201 - Flags: review?(bzbarsky)
(Assignee)

Updated

6 years ago
Attachment #744202 - Flags: review?(bzbarsky)
(Assignee)

Comment 4

6 years ago
This is green.
Comment on attachment 744201 [details] [diff] [review]
Stop checking dom.xbl_scopes. v1

>--- a/content/base/src/nsINode.cpp
>+++ b/content/base/src/nsINode.cpp
>@@ -723,18 +722,17 @@ nsINode::GetUserData(JSContext* aCx, const nsAString& aKey, ErrorResult& aError)
> }
> 
> //static
> bool
> nsINode::ShouldExposeUserData(JSContext* aCx, JSObject* /* unused */)
> {
>   JSCompartment* compartment = js::GetContextCompartment(aCx);
>   return xpc::AccessCheck::isChrome(compartment) ||
>-         xpc::IsXBLScope(compartment) ||
>-         !XPCJSRuntime::Get()->XBLScopesEnabled();
>+         xpc::IsXBLScope(compartment);
> }

Could you rename the function back to IsChromeOrXBL()?
Comment on attachment 744201 [details] [diff] [review]
Stop checking dom.xbl_scopes. v1

r=me
Attachment #744201 - Flags: review?(bzbarsky) → review+
Comment on attachment 744202 [details] [diff] [review]
Fix tests that munge dom.xbl_scopes. v1

> +  // loading the real test. This stuff is no longer behind a test, but we just

s/behind a test/behind a pref/

r=me
Attachment #744202 - Flags: review?(bzbarsky) → review+

Comment 9

6 years ago
https://hg.mozilla.org/mozilla-central/rev/d56decca3e60
https://hg.mozilla.org/mozilla-central/rev/e4074e65f6e2
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.