Closed Bug 907206 Opened 11 years ago Closed 11 years ago

data store for local apps

Categories

(DevTools Graveyard :: WebIDE, defect)

Product:
DevTools Graveyard
Component:
WebIDE
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 26

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file, 3 obsolete files)

data store for local apps
7.56 KB, patch
paul
: review+
Details | Diff | Splinter Review 
      No description provided.
Attached patch data store for local apps (obsolete) — — Splinter Review 
Needs tests
Attached patch data store for local apps (obsolete) — — Splinter Review 

        Attachment #792867 -
        Attachment is obsolete: true

    
    

    
  

      
      
      
      

        Attached patch
          
          
        data store for local apps (obsolete)
        — 
          — Splinter Review
      

    


  

        Attachment #792907 -
        Attachment is obsolete: true

    
  

        Attachment #792911 -
        Attachment is obsolete: true

    
    

    
  
Comment on attachment 792915 [details] [diff] [review]
data store for local apps

Now with test.

        Attachment #792915 -
        Flags: review?(paul)

    
  
Depends on: 907268

    
    

    
  
Comment on attachment 792915 [details] [diff] [review]
data store for local apps

How do we know that we should listen to "ready"?
Maybe AppProjects has already been initialized, and the "ready" event has been fired earlier.
The test assumes AppProjects has never been initialized.

        Attachment #792915 -
        Flags: review?(paul) → review+

    
    

    
  
I'll address that ready issue in a followup by using a "ready" promise.
Whiteboard: [land-in-fx-team]

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/0494d0220b3f
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26

    
  
Product: Firefox → DevTools

    
  
Product: DevTools → DevTools Graveyard

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: