Open
Bug 1082527
Opened 10 years ago
Updated 2 years ago
Use LibHandle's mappable field in CustomElf
Categories
(Core :: mozglue, defect)
Tracking
()
NEW
People
(Reporter: glandium, Assigned: glandium)
References
(Depends on 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•10 years ago
|
||
Bug 802240 added an API to get a mappable for SystemElf objects, but for some
reason this wasn't consolidated with the existing mappable field in CustomElf.
Attachment #8504673 -
Flags: review?(nfroyd)
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8504673 [details] [diff] [review]
Use LibHandle's mappable field in CustomElf
Review of attachment 8504673 [details] [diff] [review]:
-----------------------------------------------------------------
WFM
Attachment #8504673 -
Flags: review?(nfroyd) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
Somehow, this breaks TestANRReporter (more than it already is according to the pile of bugs treeherder suggests).
|
|
Assignee | |
Comment 4•10 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #1)
> Created attachment 8504673 [details] [diff] [review]
> Use LibHandle's mappable field in CustomElf
>
> Bug 802240 added an API to get a mappable for SystemElf objects, but for some
> reason this wasn't consolidated with the existing mappable field in
> CustomElf.
And the reason for this is bug 1083014.
Can't land this until that's fixed. Considering it's more work than I'm ready to put right now, I'll find a workaround.
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•