Closed
Bug 1409960
Opened 7 years ago
Closed 7 years ago
Remove ClassInfoData
Categories
(Core :: Security: CAPS, enhancement)
Core
Security: CAPS
Tracking
()
RESOLVED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
Details
Attachments
(1 file)
5.10 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
This class does some caching of an nsIClassInfo's flags and name. Presumably it made sense at some time in the past, but now it's only used by CanCreateWrapper(). It's simpler to just directly get the flags and name when needed, so this patch does that and removes the class.
Assignee | ||
Comment 1•7 years ago
|
||
Attachment #8919994 -
Flags: review?(bzbarsky)
Comment 2•7 years ago
|
||
Comment on attachment 8919994 [details] [diff] [review] Remove ClassInfoData r=me
Attachment #8919994 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 3•7 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/3b3a3a585d0d97233737efb1573670f34689bc63 Bug 1409960 - Remove ClassInfoData. r=bz.
Comment 4•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3b3a3a585d0d
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in
before you can comment on or make changes to this bug.
Description
•