Bug 151870

Summary:

Modern IDB: storage/indexeddb/version-change-exclusive.html fails

Product:

WebKit

Reporter:

Brady Eidson <beidson>

Component:

WebCore Misc.

Assignee:

Brady Eidson <beidson>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, alecflett, buildbot, commit-queue, jsbell, rniwa

Priority:

Version:

Safari 9

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

149117, 150882

Attachments:

Description	Flags
Patch v1	achristensen: review+, buildbot: commit-queue-
Archive of layout-test-results from ews103 for mac-yosemite	none
Archive of layout-test-results from ews106 for mac-yosemite-wk2	none
Patch again for EWS (now that r193428 has landed)	none

Brady Eidson

Reported 2015-12-04 10:18:30 PST

Modern IDB: storage/indexeddb/version-change-exclusive.html fails Trying to open the same database twice in quick succession, while the first open operation is creating the database, fails.

Attachments
Patch v1 (6.37 KB, patch) 2015-12-04 10:20 PST, Brady Eidson	achristensen: review+ buildbot: commit-queue-	Details Formatted Diff Diff
Archive of layout-test-results from ews103 for mac-yosemite (954.74 KB, application/zip) 2015-12-04 10:46 PST, Build Bot	no flags	Details
Archive of layout-test-results from ews106 for mac-yosemite-wk2 (789.58 KB, application/zip) 2015-12-04 11:10 PST, Build Bot	no flags	Details
Patch again for EWS (now that r193428 has landed) (6.37 KB, patch) 2015-12-04 11:11 PST, Brady Eidson	no flags	Details Formatted Diff Diff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.

Brady Eidson

Comment 1 2015-12-04 10:20:41 PST

Created attachment 266637 [details] Patch v1

Alex Christensen

Comment 2 2015-12-04 10:27:57 PST

Comment on attachment 266637 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=266637&action=review > LayoutTests/storage/indexeddb/modern/double-open-expected.txt:2 > +Second request done Why don't we print out First request done, too?

Build Bot

Comment 3 2015-12-04 10:46:12 PST

Comment on attachment 266637 [details] Patch v1 Attachment 266637 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/516144 New failing tests: storage/indexeddb/version-change-exclusive.html

Build Bot

Comment 4 2015-12-04 10:46:15 PST

Created attachment 266642 [details] Archive of layout-test-results from ews103 for mac-yosemite The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-yosemite Platform: Mac OS X 10.10.5

Build Bot

Comment 5 2015-12-04 11:10:06 PST

Comment on attachment 266637 [details] Patch v1 Attachment 266637 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.webkit.org/results/516190 New failing tests: storage/indexeddb/version-change-exclusive.html

Build Bot

Comment 6 2015-12-04 11:10:08 PST

Created attachment 266643 [details] Archive of layout-test-results from ews106 for mac-yosemite-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: ews106 Port: mac-yosemite-wk2 Platform: Mac OS X 10.10.5

Brady Eidson

Comment 7 2015-12-04 11:11:01 PST

Created attachment 266644 [details] Patch again for EWS (now that r193428 has landed)

Brady Eidson

Comment 8 2015-12-04 11:42:30 PST

http://trac.webkit.org/changeset/193431

Brady Eidson

Comment 9 2015-12-04 11:43:02 PST

(In reply to comment #2) > Comment on attachment 266637 [details] > Patch v1 > > View in context: > https://bugs.webkit.org/attachment.cgi?id=266637&action=review > > > LayoutTests/storage/indexeddb/modern/double-open-expected.txt:2 > > +Second request done > > Why don't we print out First request done, too? Doh, sorry I missed this comment. Not relevant for the test, really, but I'll keep it in mind if I have to revisit this test.

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