Closed Bug 1013905 Opened 8 years ago Closed 8 years ago

Bind lazyRequireGetter correctly in Loader.jsm

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 32

People

(Reporter: manishearth, Assigned: manishearth)

References

Details

Attachments

(1 file)

Attached patch PatchSplinter Review
`lazyRequireGetter()` from toolkit/devtools/Loader.jsm doesn't seem to exist when Loader.jsm is imported. This can be fixed by binding it similar to what is done to require() here[1]

 [1]:http://dxr.mozilla.org/mozilla-central/source/toolkit/devtools/Loader.jsm#258
Attachment #8426176 - Flags: review?(jwalker)
Blocks: 1004485
Attachment #8426176 - Flags: review?(jwalker) → review+
Assignee: nobody → manishearth
Status: NEW → ASSIGNED
Keywords: checkin-needed
OS: Linux → All
Hardware: x86_64 → All
https://hg.mozilla.org/mozilla-central/rev/6355753d755b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 32
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.