Closed
Bug 764618
Opened 12 years ago
Closed 12 years ago
Remove IsOnPrefWhitelist and URIIsChromeOrInPref
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: qdot, Assigned: qdot)
References
Details
Attachments
(1 file)
6.50 KB,
patch
|
gwagner
:
review+
|
Details | Diff | Splinter Review |
nsContentUtils::URIIsChromeOrInPref and nsContentUtils::IsOnPrefWhitelist do the exact same thing. Remove IsOnPrefWhitelist and replace all calls with URIIsChromeOrInPref.
Updated•12 years ago
|
Component: DOM: Device Interfaces → DOM
OS: Linux → All
QA Contact: device-interfaces → general
Hardware: x86_64 → All
Assignee | ||
Comment 1•12 years ago
|
||
Now that we have the permissions API, both of these functions are obsolete and should be removed.
Summary: Replace IsOnPrefWhitelist with URIIsChromeOrInPref and remove IsOnPrefWhitelist → Remove IsOnPrefWhitelist and URIIsChromeOrInPref
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Comment 2•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Attachment #653547 -
Flags: review?(anygregor)
Comment 3•12 years ago
|
||
Comment on attachment 653547 [details] [diff] [review] Patch 1 (v1): Remove IsOnPrefWhitelist and URIIsChromeOrInPref \o/
Attachment #653547 -
Flags: review?(anygregor) → review+
Assignee | ||
Comment 4•12 years ago
|
||
Try: https://tbpl.mozilla.org/?tree=Try&rev=067d8af1813c
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/02227f11b424
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/02227f11b424
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•