Closed Bug 1365861 Opened 7 years ago Closed 7 years ago

Minor gonk cleanup in nsUpdateService.js

Categories

(Toolkit :: Application Update, enhancement)

Product:
Toolkit
Component:
Application Update
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox55 --- fixed

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

Details

Attachments

(1 file)

patch rev1
8.77 KB, patch
molly
: review+
Details | Diff | Splinter Review 
Missed some of the cleanup. Also cleans up a very small amount of code and replaces Array.prototype.indexOf with Array.prototype.includes now that it is stable.
Attached patch patch rev1 — — Splinter Review 
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED

        Attachment #8868909 -
        Flags: review?(mhowell)

        Attachment #8868909 -
        Flags: review?(mhowell) → review+

    
    

    
  
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/59566eae9404
Minor gonk cleanup in nsUpdateService.js. r=mhowell

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/59566eae9404
Status: ASSIGNED → RESOLVED
Closed: 7 years ago

          status-firefox55:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: