Closed Bug 281001 Opened 20 years ago Closed 18 years ago

nsComboboxControlFrame.cpp, make GetOption() return already_AddRefed

Categories

(Core :: Layout: Form Controls, enhancement)

Product:
Core
Component:
Layout: Form Controls
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)

References

Details

(spawned off from bug 279868 comment 32)

Boris says:
Mind doing a separate patch on making this method return an already_AddRefed
and removing dont_AddRef/already_AddRefed from callers?

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/forms/nsComboboxControlFrame.cpp&rev=1.303&root=/cvsroot&mark=715-728#711
Boris also said in bug 279868 comment 32:
It may also be worth creating a GetListFrame() method and refactoring all these
callers to use it....


This code went away with the reflow branch.
Status: NEW → RESOLVED
Closed: 18 years ago
Depends on: reflow-refactor
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.