Closed Bug 707364 Opened 13 years ago Closed 12 years ago

inline a nested function in nsPrompter.js

Categories

(Toolkit :: General, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: fryn, Assigned: fryn)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
The function-local function getChromeWindow probably exists, because it was once called in multiple branches of code.
Since that's no longer the case, we can just inline the function body for both clarity and performance.
Attachment #578767 - Flags: review?(dolske)
Attachment #578767 - Flags: review?(dolske) → review+
https://hg.mozilla.org/integration/fx-team/rev/06c0a1711186
Summary: remove unnecessary nested function in nsPrompter.js → inline a nested function in nsPrompter.js
https://hg.mozilla.org/mozilla-central/rev/06c0a1711186
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.