Closed Bug 1253593 Opened 8 years ago Closed 8 years ago

[e10s/appcache] onchecking not called

Categories

(Core :: Networking: Cache, defect)

Product:
Core
Component:
Networking: Cache
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
e10s + ---
firefox47 --- affected
firefox48 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

Details

(Whiteboard: [necko-active])

Attachments

(1 file)

v1
1.27 KB, patch
jduell.mcbugs
: review+
Details | Diff | Splinter Review 
      No description provided.
tracking-e10s: --- → ?
Whiteboard: [necko-backlog]
This is weird!  Now I can't reproduce with exactly the same setup :)  That's fun.

Can you give me your STR/expected/actual?
Flags: needinfo?(gijskruitbosch+bugs)
Whiteboard: [necko-backlog] → [necko-active]
For me it seems that only onchecking is not called...
(In reply to Honza Bambas (:mayhemer) from comment #1)
> This is weird!  Now I can't reproduce with exactly the same setup :)  That's
> fun.
> 
> Can you give me your STR/expected/actual?

1. get fx-team on Win32, build;
2. apply WIP patch from bug 621158;
3. ./mach build faster
4. ./mach mochitest --e10s browser/base/content/test/general/browser_offlineQuotaNotification.js 

and it hangs indefinitely waiting for the oncached call.
Flags: needinfo?(gijskruitbosch+bugs)
After a correct verification only missing notification is onchecking.  oncached is always called.
Status: NEW → ASSIGNED
Summary: [e10s/appcache] onchecking and oncached are not called → [e10s/appcache] onchecking not called
Attached patch v1 — — Splinter Review 

        Attachment #8727076 -
        Flags: review?(jduell.mcbugs)

    
    

    
  
Tho, doesn't fix the original bug.
No longer blocks: 621158

    
  

        Attachment #8727076 -
        Flags: review?(jduell.mcbugs) → review+

    
  
Keywords: checkin-needed

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/13462856bcc5
Status: ASSIGNED → RESOLVED
Closed: 8 years ago

          status-firefox48:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: