Closed Bug 403036 Opened 16 years ago Closed 16 years ago

nsAutoCompleteController::GetPopupWidget() is dead code

Categories

(Toolkit :: Autocomplete, defect, P1)

Product:
Toolkit
Component:
Autocomplete
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9beta2

People

(Reporter: Gavin, Assigned: Gavin)

Details

(Keywords: memory-footprint)

Attachments

(1 file)

patch
4.82 KB, patch
enndeakin
: review+
mtschrep
: approval1.9+
Details | Diff | Splinter Review 
I noticed this while reviewing Seth's patch for bug 402668. nsAutoCompleteController::GetPopupWidget() was added by masayuki in bug 187772, but it's only callers (AttachRollupListener/DetachRollupListener) were removed by Neil's patch for bug 279703.
Attached patch patch — — Splinter Review 
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED

        Attachment #288028 -
        Flags: review?(enndeakin)

    
  

        Attachment #288028 -
        Flags: review?(enndeakin) → review+

        Attachment #288028 -
        Flags: approval1.9?
Priority: -- → P1
Target Milestone: --- → mozilla1.9 M10

    
  

        Attachment #288028 -
        Flags: approval1.9? → approval1.9+

    
    

    
  
mozilla/toolkit/components/autocomplete/src/Makefile.in 	1.13
mozilla/toolkit/components/autocomplete/src/nsAutoCompleteController.cpp 1.68
mozilla/toolkit/components/autocomplete/src/nsAutoCompleteController.h 	1.11
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Version: unspecified → Trunk

          You need to log in
          before you can comment on or make changes to this bug.