Closed Bug 394652 Opened 17 years ago Closed 17 years ago

Merge firefox-specific context-menu items "back" to tabbrowser.xml

Categories

(Firefox :: General, defect, P2)

Product:
Firefox
Component:
General
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 3 beta1

People

(Reporter: asaf, Assigned: asaf)

Details

(Keywords: perf)

Attachments

(1 file)

patch
12.39 KB, patch
mconnor
: review+
mconnor
: approval1.9+
Details | Diff | Splinter Review 
Merge firefox-specific context-menu items "back" to tabbrowser.xml.
This is about addBookmarkMenuitems and _addUndoCloseTabContextMenu in browser.js, right? Are you working on this? I just came across this and almost filed a bug, would be willing to post a patch.
Version: unspecified → Trunk
Attached patch patch — — Splinter Review 

        Attachment #283226 -
        Flags: review?(mconnor)

    
  

        Attachment #283226 -
        Flags: approval1.9?

    
  
Status: NEW → ASSIGNED
Keywords: perf
Priority: -- → P2
Target Milestone: --- → Firefox 3 M9

    
    

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

yay to stupid cruft removal

        Attachment #283226 -
        Flags: review?(mconnor) → review+

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

    
    

    
  
mozilla/browser/base/content/browser.js 1.859
mozilla/browser/base/content/tabbrowser.xml 1.245
mozilla/browser/locales/en-US/chrome/browser/browser.properties 1.49
mozilla/browser/locales/en-US/chrome/browser/tabbrowser.dtd 1.5
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: