Closed Bug 1121519 Opened 9 years ago Closed 9 years ago

Intermittent WorkerGlobalScope/close/sending-messages.html | close() and sending messages - assert_equals: expected 2 but got 0

Categories

(Core :: DOM: Workers, defect)

Product:
Core
Component:
DOM: Workers
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla38
Tracking Flags:
Tracking Status
firefox36 --- unaffected
firefox37 --- fixed
firefox38 --- fixed
firefox-esr31 --- unaffected
b2g-v2.2 --- fixed
b2g-master --- fixed

People

(Reporter: RyanVM, Assigned: bent.mozilla)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Fix
1.00 KB, patch
jgraham
: review+
Details | Diff | Splinter Review 
06:45:40 INFO - TEST-START | /workers/interfaces/WorkerGlobalScope/close/sending-messages.html
06:45:40 INFO - PROCESS | 192 |
06:45:40 INFO - PROCESS | 192 | ###!!! [Parent][OnMaybeDequeueOne] Error: Channel closing: too late to send/recv, messages will be lost
06:45:40 INFO - PROCESS | 192 |
06:45:40 INFO - PROCESS | 192 | WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate. Use isContentWindowPrivate instead (but only for frame scripts).
06:45:40 INFO - PROCESS | 192 | pbu_isWindowPrivate@resource://gre/modules/PrivateBrowsingUtils.jsm:25:14
06:45:40 INFO - PROCESS | 192 | nsBrowserAccess.prototype.openURI@chrome://browser/content/browser.js:14051:25
06:45:40 INFO - PROCESS | 192 | __marionetteFunc@dummy file:17:30
06:45:40 INFO - PROCESS | 192 | @dummy file:23:3
06:45:40 INFO - PROCESS | 192 | executeWithCallback@chrome://marionette/content/marionette-listener.js:684:5
06:45:40 INFO - PROCESS | 192 | executeAsyncScript@chrome://marionette/content/marionette-listener.js:577:3
06:45:40 INFO - PROCESS | 192 |
06:45:40 INFO - PROCESS | 192 | ###!!! [Parent][OnMaybeDequeueOne] Error: Channel closing: too late to send/recv, messages will be lost
06:45:40 INFO - PROCESS | 192 |
06:45:41 INFO - TEST-UNEXPECTED-FAIL | /workers/interfaces/WorkerGlobalScope/close/sending-messages.html | close() and sending messages - assert_equals: expected 2 but got 0
06:45:41 INFO - TEST-OK | /workers/interfaces/WorkerGlobalScope/close/sending-messages.html | took 750ms
This test is using a small setTimeout, so it's guaranteed to be racy/flaky.
Attached patch Fix — — Splinter Review 

        Attachment #8549085 -
        Flags: review?(james)

        Attachment #8549085 -
        Flags: review?(james) → review+

    
    

    
      
  

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/4f99aee889fa
Assignee: nobody → bent.mozilla
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38

    
    

    
      
  

    
    

    
      
  

    
    

    
  
Can we please uplift this to Aurora? :)

          status-b2g-v2.2:
          --- → affected

          status-b2g-master:
          --- → fixed

          status-firefox37:
          unaffected → affected
Flags: needinfo?(bent.mozilla)

    
    

    
  
(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #10)
> Can we please uplift this to Aurora? :)

Today was the uplift, so s/Aurora/Beta :)

    
  
Flags: needinfo?(bent.mozilla)

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: