Status

()

defect
RESOLVED FIXED
7 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

7 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

7 years ago
Blocks: 834701
Assignee

Updated

7 years ago
Blocks: 834707
Assignee

Updated

6 years ago
Blocks: 825392
Assignee

Updated

6 years ago
Blocks: 857356
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+
https://hg.mozilla.org/mozilla-central/rev/d56decca3e60
https://hg.mozilla.org/mozilla-central/rev/e4074e65f6e2
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.