Closed
Bug 707364
Opened 13 years ago
Closed 12 years ago
inline a nested function in nsPrompter.js
Categories
(Toolkit :: General, defect)
Toolkit
General
Tracking
()
RESOLVED
FIXED
mozilla11
People
(Reporter: fryn, Assigned: fryn)
Details
Attachments
(1 file)
1.87 KB,
patch
|
Dolske
:
review+
|
Details | Diff | Splinter 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)
Updated•13 years ago
|
Attachment #578767 -
Flags: review?(dolske) → review+
Assignee | ||
Comment 1•12 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/06c0a1711186
Summary: remove unnecessary nested function in nsPrompter.js → inline a nested function in nsPrompter.js
Comment 2•12 years ago
|
||
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.
Description
•