Bug 71536 - Layout Test storage/indexeddb/exception-in-event-aborts.html is failing
Summary: Layout Test storage/indexeddb/exception-in-event-aborts.html is failing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Erik Arvidsson
URL:
Keywords:
Depends on:
Blocks: 71533
  Show dependency tree / graph
 
Reported: 2011-11-03 20:30 PDT by Yuta Kitamura
Modified: 2011-11-06 18:50 PST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yuta Kitamura 2011-11-03 20:30:31 PDT 
The following layout test is failing on Chromium

storage/indexeddb/exception-in-event-aborts.html

Probable cause:
http://trac.webkit.org/changeset/99258

It wasn't obvious for me to fix this test correctly, as I don't know the background of the bulk removal of successfullyParsed. So I'm marking this as failing for now.

Diff:

--- e:\b\build\slave\Webkit_Win7\build\layout-test-results\storage/indexeddb/exception-in-event-aborts-expected.txt 
+++ e:\b\build\slave\Webkit_Win7\build\layout-test-results\storage/indexeddb/exception-in-event-aborts-actual.txt 
@@ -55,7 +55,7 @@
 
 PASS The transaction completed.
 
-PASS successfullyParsed is true
+FAIL successfullyParsed should be true (of type boolean). Was undefined (of type undefined).
 
 TEST COMPLETE
Comment 1 Erik Arvidsson 2011-11-03 22:34:09 PDT 
The code throws so it needs to call shouldHaveHadError. Fixing.
Comment 2 Erik Arvidsson 2011-11-04 13:59:33 PDT 
Sorry, procedure failure. This was committed as r99316.

I don't have a Chromium checkout at the moment. I'll have to do that on Monday. Is there a cr bug for this?
Comment 3 Yuta Kitamura 2011-11-06 18:50:06 PST 
Confirmed
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=storage%2Findexeddb%2Fexception-in-event-aborts.html

I'll remove the test expectation later today. Thanks!