Closed
      
        Bug 1189536
      
      
        Opened 10 years ago
          Closed 10 years ago
      
        
    
  
Make service-workers/service-worker/fetch-request-xhr.https.html pass  
    Categories
(Core :: DOM: Service Workers, defect)
        Core
          
        
        
      
        
    
        DOM: Service Workers
          
        
        
      
        
    Tracking
()
        RESOLVED
        FIXED
        
    
  
        
            mozilla44
        
    
  
People
(Reporter: jdm, Assigned: ehsan.akhgari)
References
Details
Attachments
(2 files)
| 9.88 KB,
          patch         | Details | Diff | Splinter Review | |
| 8.78 KB,
          patch         | bkelly
:
              
              review+ | Details | Diff | Splinter Review | 
Dumping ground for my various test changes required to get it running in Firefox.
| Reporter | ||
| Comment 1•10 years ago
           | ||
| Reporter | ||
| Updated•10 years ago
           | 
Assignee: nobody → josh
Status: NEW → ASSIGNED
|   | ||
| Comment 2•10 years ago
           | ||
Harness status: Timeout
Found 1 tests
1 Timeout
* Verify the body of FetchEvent using XMLHttpRequest -> timed out
Traces: https://pastebin.mozilla.org/8841201
since the test status is "Timeout" blocking the meta gathering this (Bug 1180622)
| Reporter | ||
| Comment 3•10 years ago
           | ||
FWIW the timeout is caused by the use of HTTP_ORIGIN in the postMessage call, as well as various other places the the test's iframe. Once those are changed an actual failure is reported, but the test is designed so that either everything passes or the whole test fails. It's quite frustrating.
| Comment 4•10 years ago
           | ||
Also, I'm not sure its worth grouping stuff under a "timeout" meta bug.  We're fixing them regardless of the error type.
|   | ||
| Comment 5•10 years ago
           | ||
(In reply to Ben Kelly [:bkelly] from comment #4)
> Also, I'm not sure its worth grouping stuff under a "timeout" meta bug. 
> We're fixing them regardless of the error type.
ok, please feel free to remove all the meta bugs created to gather the different scenarios ("timeout"("file not found"), "error", "ok+failing tests"), and just use Bug 1189023 as the meta bug, in case they don't add any value to track wpt test suite issues.
Depends on: 1108181
| Assignee | ||
| Updated•10 years ago
           | 
Assignee: josh → ehsan
| Assignee | ||
| Comment 6•10 years ago
           | ||
        Attachment #8675896 -
        Flags: review?(bkelly)
| Updated•10 years ago
           | 
        Attachment #8675896 -
        Flags: review?(bkelly) → review+
| Comment 8•10 years ago
           | ||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4ed118d4faff0282d6783182aba8fd96b50f583a
Backed out changeset d8e39b884737 (bug 1189536) for test failure ON A CLOSED TREE
| Comment 9•10 years ago
           | ||
Backed out for bustages like this:
 23:50:36     INFO - TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/fetch-request-xhr.https.html | Verify the body of FetchEvent using XMLHttpRequest - Test timed out
 23:50:36     INFO - TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/fetch-request-xhr.https.html | expected OK
https://treeherder.mozilla.org/logviewer.html#?job_id=15924928&repo=mozilla-inbound
| Comment 10•10 years ago
           | ||
| Comment 11•10 years ago
           | ||
| backout bugherder merge | ||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
          status-firefox44:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
          You need to log in
          before you can comment on or make changes to this bug.
        
Description
•