Bug 101996 - IndexedDB: storage/indexeddb/mozilla/add-twice-failure.html is flaky following r134252
Summary: IndexedDB: storage/indexeddb/mozilla/add-twice-failure.html is flaky followin...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Joshua Bell
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-12 14:53 PST by Joshua Bell
Modified: 2012-11-13 10:46 PST (History)
4 users (show)

See Also:


Attachments
Patch (1.49 KB, patch)
2012-11-12 14:59 PST, Joshua Bell
no flags Details | Formatted Diff | Diff
Patch (1.51 KB, patch)
2012-11-13 08:49 PST, Joshua Bell
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joshua Bell 2012-11-12 14:53:25 PST
Flakiness dashboard link:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=storage%2Findexeddb%2Fmozilla%2Fadd-twice-failure.html

Flaky diff:

 PASS request.result is key
 request = objectStore.add({}, key);
 PASS event.target.errorCode is IDBDatabaseException.CONSTRAINT_ERR
+FAIL Abort function called unexpectedly!
+FAIL Error function called unexpectedly: (AbortError) Version change transaction was aborted in upgradeneeded event handler.
 PASS successfullyParsed is true

Cause:

This was added to the test:

  event.target.transaction.onabort = unexpectedAbortCallback

The test induces an error but doesn't prevent the default, so it will bubble up and be caught by this. It's flaky because this may or may not occur before the test output is captured.

Trivial fix is to prevent the default.
Comment 1 Joshua Bell 2012-11-12 14:59:36 PST
Created attachment 173735 [details]
Patch
Comment 2 Joshua Bell 2012-11-12 14:59:52 PST
dgrogan@ - please take a look?
Comment 3 David Grogan 2012-11-12 15:05:08 PST
Comment on attachment 173735 [details]
Patch

LGTM
Comment 4 Joshua Bell 2012-11-12 15:05:45 PST
tony@ - r? cq?
Comment 5 WebKit Review Bot 2012-11-12 19:39:12 PST
Comment on attachment 173735 [details]
Patch

Rejecting attachment 173735 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
webkit-commit-queue/Source/WebKit/chromium/webkit --revision 167263 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
51>At revision 167263.

________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'

________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/14813601
Comment 6 Joshua Bell 2012-11-13 08:49:16 PST
Created attachment 173904 [details]
Patch
Comment 7 WebKit Review Bot 2012-11-13 10:46:08 PST
Comment on attachment 173904 [details]
Patch

Clearing flags on attachment: 173904

Committed r134439: <http://trac.webkit.org/changeset/134439>
Comment 8 WebKit Review Bot 2012-11-13 10:46:11 PST
All reviewed patches have been landed.  Closing bug.