Closed Bug 351881 Opened 18 years ago Closed 18 years ago

nsIPrompt needs nsIPromptService::STD_YES_NO_BUTTONS

Categories

(Core :: General, defect)

Product:
Core
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: csthomas, Assigned: csthomas)

Details

Attachments

(1 file)

patch
1.48 KB, patch
Biesinger
: review+
darin.moz
: superreview+
Details | Diff | Splinter Review 
See bug 351374 comment 3 - nsIPrompt needs the yes/no constant to be defined.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
Keywords: helpwanted
Attached patch patch — — Splinter Review 
Assignee: nobody → cst
Status: NEW → ASSIGNED

        Attachment #249804 -
        Flags: superreview?(darin.moz)

        Attachment #249804 -
        Flags: review?

        Attachment #249804 -
        Flags: review? → review?(cbiesinger)

    
    

    
  
Comment on attachment 249804 [details] [diff] [review]
patch

will you fix callers of nsIPrompt to use this new constant too?

        Attachment #249804 -
        Flags: review?(cbiesinger) → review+

    
    

    
  
Comment on attachment 249804 [details] [diff] [review]
patch

sr=darin

        Attachment #249804 -
        Flags: superreview?(darin.moz) → superreview+

    
    

    
  
Fixed on trunk.  Filed bug 365529 to fix callers.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Keywords: helpwanted

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: