Closed Bug 1210133 Opened 9 years ago Closed 4 years ago

Intermittent test_filehandle_getFile.html | indexedDB error, 'AbortError'

Categories

(Core :: Storage: IndexedDB, defect, P2)

defect

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox-esr68 --- disabled

People

(Reporter: KWierso, Unassigned)

Details

(Keywords: intermittent-failure, Whiteboard: [comment 30][stockwell disabled])

Attachments

(1 file)

Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
Priority: P3 → P5

Recent failure:

https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=235709330&repo=autoland&lineNumber=7195

[task 2019-03-24T08:26:58.244Z] 08:26:58     INFO - TEST-INFO | started process screentopng
[task 2019-03-24T08:26:59.280Z] 08:26:59     INFO - TEST-INFO | screentopng: exit 0
[task 2019-03-24T08:26:59.282Z] 08:26:59     INFO - Buffered messages logged at 08:26:58
[task 2019-03-24T08:26:59.283Z] 08:26:59     INFO - Running
[task 2019-03-24T08:26:59.284Z] 08:26:59     INFO - Pushing preferences
[task 2019-03-24T08:26:59.285Z] 08:26:59     INFO - Pushing permissions
[task 2019-03-24T08:26:59.286Z] 08:26:59     INFO - Clearing old databases
[task 2019-03-24T08:26:59.287Z] 08:26:59     INFO - TEST-FAIL | dom/indexedDB/test/test_filehandle_getFile.html | Skipping test in a worker because it's not structured properly 
[task 2019-03-24T08:26:59.289Z] 08:26:59     INFO - Running test in main thread
[task 2019-03-24T08:26:59.291Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Correct interface 
[task 2019-03-24T08:26:59.293Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Correct interface 
[task 2019-03-24T08:26:59.294Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist 
[task 2019-03-24T08:26:59.297Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist 
[task 2019-03-24T08:26:59.298Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist 
[task 2019-03-24T08:26:59.300Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist 
[task 2019-03-24T08:26:59.301Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist 
[task 2019-03-24T08:26:59.302Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist 
[task 2019-03-24T08:26:59.304Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Async/await tests shouldn't call finishTest() 
[task 2019-03-24T08:26:59.305Z] 08:26:59     INFO - Buffered messages finished
[task 2019-03-24T08:26:59.306Z] 08:26:59     INFO - TEST-UNEXPECTED-FAIL | dom/indexedDB/test/test_filehandle_getFile.html | indexedDB error, 'AbortError' 
[task 2019-03-24T08:26:59.308Z] 08:26:59     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:275:18
[task 2019-03-24T08:26:59.309Z] 08:26:59     INFO -     errorHandler@dom/indexedDB/test/helpers.js:206:3
[task 2019-03-24T08:26:59.310Z] 08:26:59     INFO -     EventHandlerNonNull*testSteps@dom/indexedDB/test/test_filehandle_getFile.html:31:5
[task 2019-03-24T08:26:59.311Z] 08:26:59     INFO -     grabEventAndContinueHandler@dom/indexedDB/test/helpers.js:189:17
[task 2019-03-24T08:26:59.313Z] 08:26:59     INFO -     EventHandlerNonNull*testSteps@dom/indexedDB/test/test_filehandle_getFile.html:27:5
[task 2019-03-24T08:26:59.314Z] 08:26:59     INFO -     grabEventAndContinueHandler@dom/indexedDB/test/helpers.js:189:17
[task 2019-03-24T08:26:59.315Z] 08:26:59     INFO -     EventHandlerNonNull*testSteps@dom/indexedDB/test/test_filehandle_getFile.html:19:5
[task 2019-03-24T08:26:59.316Z] 08:26:59     INFO -     testHarnessSteps@dom/indexedDB/test/helpers.js:155:19
[task 2019-03-24T08:26:59.318Z] 08:26:59     INFO -     nextTestHarnessStep@dom/indexedDB/test/helpers.js:49:26
[task 2019-03-24T08:26:59.319Z] 08:26:59     INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Async/await tests shouldn't call finishTest() 
[task 2019-03-24T08:26:59.320Z] 08:26:59     INFO - GECKO(3969) | MEMORY STAT | vsize 1484MB | residentFast 125MB | heapAllocated 14MB
[task 2019-03-24T08:26:59.323Z] 08:26:59    ERROR - [SimpleTest.finish()] this test already called finish!
[task 2019-03-24T08:26:59.325Z] 08:26:59     INFO - TEST-UNEXPECTED-ERROR | dom/indexedDB/test/test_filehandle_getFile.html | called finish() multiple times

:ttung -- I wonder if this might have been caused or triggered by bug 1498183? Can you have a look?

Flags: needinfo?(shes050117)

(In reply to Geoff Brown [:gbrown] from comment #23)

:ttung -- I wonder if this might have been caused or triggered by bug 1498183? Can you have a look?

It doesn't look like because the change in bug 1498183 is for supporting async/await on indexedDB tests. And, the error here are "indexedDB error, 'AbortError'", and "called finish() multiple times".

The second error is probably because there is a race between onerror event (it will call finish() in errorHandler [1]) and at the end of the test itself (calling finish).

Thus, if we can fix the first one than all the stuff should be fixed. (The 'AbortError' shouldn't be caused by bug 1498183)

[1] https://searchfox.org/mozilla-central/rev/ddd1679c0534f7ddf36cafddd17b710c4fefe3c4/dom/indexedDB/test/helpers.js#207

Flags: needinfo?(shes050117)

There are 22 total failures in the last 7 days on android-em-7-0-x86_64 opt, linux32 debug, linux32-shippable opt, linux64-qr and linux64-shippable-qr opt

Recent failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=238581967&repo=mozilla-inbound&lineNumber=7364

[task 2019-04-06T13:13:33.204Z] 13:13:33 INFO - TEST-START | dom/indexedDB/test/test_filehandle_getFile.html
[task 2019-04-06T13:13:33.329Z] 13:13:33 INFO - TEST-INFO | started process screentopng
[task 2019-04-06T13:13:34.451Z] 13:13:34 INFO - TEST-INFO | screentopng: exit 0
[task 2019-04-06T13:13:34.452Z] 13:13:34 INFO - Buffered messages logged at 13:13:33
[task 2019-04-06T13:13:34.452Z] 13:13:34 INFO - Running
[task 2019-04-06T13:13:34.452Z] 13:13:34 INFO - Pushing preferences
[task 2019-04-06T13:13:34.452Z] 13:13:34 INFO - Pushing permissions
[task 2019-04-06T13:13:34.453Z] 13:13:34 INFO - Clearing old databases
[task 2019-04-06T13:13:34.453Z] 13:13:34 INFO - TEST-FAIL | dom/indexedDB/test/test_filehandle_getFile.html | Skipping test in a worker because it's not structured properly
[task 2019-04-06T13:13:34.453Z] 13:13:34 INFO - Running test in main thread
[task 2019-04-06T13:13:34.453Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Correct interface
[task 2019-04-06T13:13:34.453Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Correct interface
[task 2019-04-06T13:13:34.456Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist
[task 2019-04-06T13:13:34.457Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist
[task 2019-04-06T13:13:34.459Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist
[task 2019-04-06T13:13:34.460Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist
[task 2019-04-06T13:13:34.462Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist
[task 2019-04-06T13:13:34.464Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Property should not exist
[task 2019-04-06T13:13:34.466Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Async/await tests shouldn't call finishTest()
[task 2019-04-06T13:13:34.468Z] 13:13:34 INFO - Buffered messages finished
[task 2019-04-06T13:13:34.470Z] 13:13:34 INFO - TEST-UNEXPECTED-FAIL | dom/indexedDB/test/test_filehandle_getFile.html | indexedDB error, 'AbortError'
[task 2019-04-06T13:13:34.472Z] 13:13:34 INFO - SimpleTest.ok@SimpleTest/SimpleTest.js:275:18
[task 2019-04-06T13:13:34.473Z] 13:13:34 INFO - errorHandler@dom/indexedDB/test/helpers.js:206:3
[task 2019-04-06T13:13:34.475Z] 13:13:34 INFO - EventHandlerNonNulltestSteps@dom/indexedDB/test/test_filehandle_getFile.html:31:5
[task 2019-04-06T13:13:34.476Z] 13:13:34 INFO - grabEventAndContinueHandler@dom/indexedDB/test/helpers.js:189:17
[task 2019-04-06T13:13:34.478Z] 13:13:34 INFO - EventHandlerNonNull
testSteps@dom/indexedDB/test/test_filehandle_getFile.html:27:5
[task 2019-04-06T13:13:34.479Z] 13:13:34 INFO - grabEventAndContinueHandler@dom/indexedDB/test/helpers.js:189:17
[task 2019-04-06T13:13:34.482Z] 13:13:34 INFO - EventHandlerNonNull*testSteps@dom/indexedDB/test/test_filehandle_getFile.html:19:5
[task 2019-04-06T13:13:34.484Z] 13:13:34 INFO - testHarnessSteps@dom/indexedDB/test/helpers.js:155:19
[task 2019-04-06T13:13:34.486Z] 13:13:34 INFO - nextTestHarnessStep@dom/indexedDB/test/helpers.js:49:26
[task 2019-04-06T13:13:34.487Z] 13:13:34 INFO - TEST-PASS | dom/indexedDB/test/test_filehandle_getFile.html | Async/await tests shouldn't call finishTest()
[task 2019-04-06T13:13:34.489Z] 13:13:34 INFO - GECKO(3972) | MEMORY STAT | vsize 1485MB | residentFast 125MB | heapAllocated 14MB
[task 2019-04-06T13:13:34.490Z] 13:13:34 ERROR - [SimpleTest.finish()] this test already called finish!
[task 2019-04-06T13:13:34.492Z] 13:13:34 INFO - TEST-UNEXPECTED-ERROR | dom/indexedDB/test/test_filehandle_getFile.html | called finish() multiple times
[task 2019-04-06T13:13:34.494Z] 13:13:34 INFO - TEST-INFO took 138ms
[task 2019-04-06T13:13:34.499Z] 13:13:34 INFO - TEST-OK | dom/indexedDB/test/test_filehandle_getFile.html

Geoff is this something you are looking into?

Flags: needinfo?(gbrown)
Whiteboard: [stockwell needswork:owner]

(In reply to Andreea Pavel [:apavel] from comment #26)

Geoff is this something you are looking into?

No, not at all.

Flags: needinfo?(gbrown)

Andrew can you assign someone here?

Flags: needinfo?(overholt)

I can't assign someone right now but we will soon have time on the team to take this on. Thanks for bringing it to my attention, Andreea!

Flags: needinfo?(overholt)
Priority: P5 → P2

Thanks Andrew!

Whiteboard: [stockwell needswork:owner] → [stockwell needswork:owner][comment 30]

I'll redirect this to Hsin-Yi who manages the team now.

Flags: needinfo?(overholt) → needinfo?(htsai)

Just talking with team, we can disable the test given we're going to remove FileHandle support from IDB.

Flags: needinfo?(htsai)
Pushed by rmaries@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/53cb6344eb97
test_filehandle_getFile.html on all platforms - FileHandle will be removed from IDB. r=jmaher
Keywords: leave-open
Whiteboard: [comment 30][stockwell disable-recommended] → [comment 30][stockwell disabled]

IIRC, getFile was removed from trunk. The test is still active on esr68, should it be disabled there at least?

Flags: needinfo?(jvarga)

Yeah, we can disable it (we don't expect uplifting changes that would break it there).

Flags: needinfo?(jvarga)

[Tracking Requested - why for this release]: Intermittent test failures. The test was previously disabled and ultimately removed with the implementation of getFile from trunk. We don't plan to uplift any production code changes relating to this to esr68.

We don't need to track this for ESR68, but sure, we can uplift the test disabling there. Sounds like this bug is a wontfix at this point too.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
Whiteboard: [comment 30][stockwell disabled] → [comment 30][stockwell disabled][checkin-needed-esr68]
Whiteboard: [comment 30][stockwell disabled][checkin-needed-esr68] → [comment 30][stockwell disabled]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: