Closed
Bug 1409960
Opened 8 years ago
Closed 8 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•8 years ago
|
||
Attachment #8919994 -
Flags: review?(bzbarsky)
Comment 2•8 years ago
|
||
Comment on attachment 8919994 [details] [diff] [review]
Remove ClassInfoData
r=me
Attachment #8919994 -
Flags: review?(bzbarsky) → review+
| Assignee | ||
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/3b3a3a585d0d97233737efb1573670f34689bc63
Bug 1409960 - Remove ClassInfoData. r=bz.
Comment 4•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 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
•