Bug 210226

Summary:	REGRESSION(r259705): [GTK][WPE] Several IndexedDB are failing
Product:	WebKit	Reporter:	Diego Pino <dpino>
Component:	WebKitGTK	Assignee:	Nobody <webkit-unassigned>
Status:	NEW ---
Severity:	Normal	CC:	bugs-noreply, cgarcia
Priority:	P2
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Diego Pino 2020-04-08 16:59:12 PDT

The following tests are failing after r259705:

   http/tests/appcache/history-test.html [ Failure ]
   imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors-small.html [ Failure ]
   imported/w3c/web-platform-tests/IndexedDB/large-requests-abort.html [ Failure ]
   imported/w3c/web-platform-tests/IndexedDB/request-event-ordering.html [ Failure ]

wk-testhunter for IndexedDB/interleaved-cursors-small.html:

r259704                       NOERROR
r259705                       UNKNOWN
r259706                       TEXT (Expected: PASS)

Other tests results are the same.

Diff:

--- /home/igalia/dpino/workspace/webkit/layout-test-results/retries/http/tests/appcache/history-test-expected.txt
+++ /home/igalia/dpino/workspace/webkit/layout-test-results/retries/http/tests/appcache/history-test-actual.txt
@@ -1,3 +1,4 @@
+CONSOLE MESSAGE: line 1: ApplicationCache is deprecated. Please use ServiceWorkers instead.
 CONSOLE MESSAGE: line 1: ApplicationCache is deprecated. Please use ServiceWorkers instead.
 Test whether window.history.back() is supported in application cache.


--- /home/igalia/dpino/workspace/webkit/layout-test-results/retries/imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors-small-expected.txt
+++ /home/igalia/dpino/workspace/webkit/layout-test-results/retries/imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors-small-actual.txt
@@ -1,5 +1,5 @@
 
 PASS 1 cursors 
-PASS 10 cursors 
-PASS 100 cursors 
+FAIL 10 cursors promise_test: Unhandled rejection with value: object "QuotaExceededError: Failed to PutOrAdd in database because not enough space for domain"
+FAIL 100 cursors promise_test: Unhandled rejection with value: object "QuotaExceededError: Failed to PutOrAdd in database because not enough space for domain"
 

--- /home/igalia/dpino/workspace/webkit/layout-test-results/retries/imported/w3c/web-platform-tests/IndexedDB/large-requests-abort-expected.txt
+++ /home/igalia/dpino/workspace/webkit/layout-test-results/retries/imported/w3c/web-platform-tests/IndexedDB/large-requests-abort-actual.txt
@@ -1,6 +1,6 @@
 
-PASS small values 
-PASS large values 
-PASS large value followed by small values 
-PASS large values mixed with small values 
+FAIL small values assert_equals: Expected success event, but got error event instead expected "success" but got "error"
+FAIL large values assert_equals: Expected success event, but got error event instead expected "success" but got "error"
+FAIL large value followed by small values assert_equals: Expected success event, but got error event instead expected "success" but got "error"
+FAIL large values mixed with small values assert_equals: Expected success event, but got error event instead expected "success" but got "error"


--- /home/igalia/dpino/workspace/webkit/layout-test-results/retries/imported/w3c/web-platform-tests/IndexedDB/request-event-ordering-expected.txt
+++ /home/igalia/dpino/workspace/webkit/layout-test-results/retries/imported/w3c/web-platform-tests/IndexedDB/request-event-ordering-actual.txt
@@ -1,6 +1,6 @@
 
-PASS small values 
-PASS large values 
-PASS large value followed by small values 
-PASS large values mixed with small values 
+FAIL small values assert_equals: Expected success event, but got error event instead expected "success" but got "error"
+FAIL large values assert_equals: Expected success event, but got error event instead expected "success" but got "error"
+FAIL large value followed by small values assert_equals: Expected success event, but got error event instead expected "success" but got "error"
+FAIL large values mixed with small values assert_equals: Expected success event, but got error event instead expected "success" but got "error"

Comment 1 Diego Pino 2020-04-08 19:22:01 PDT

In WPE this additional test is failing:

  imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors-large.html [ Failure ]

Comment 2 Diego Pino 2020-04-09 01:01:35 PDT

Also in WPE this other test started failing:

  http/tests/cache/partitioned-cache.html 

In GTK interleaved-cursors-large.html wasn't reported as failure because it was already marked as flaky in TestExpectations.

Comment 3 Carlos Garcia Campos 2020-04-09 06:16:09 PDT

(In reply to Diego Pino from comment #0)
> The following tests are failing after r259705:
> 
>    http/tests/appcache/history-test.html [ Failure ]
>    imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors-small.html
> [ Failure ]
>    imported/w3c/web-platform-tests/IndexedDB/large-requests-abort.html [
> Failure ]
>    imported/w3c/web-platform-tests/IndexedDB/request-event-ordering.html [
> Failure ]
> 
> wk-testhunter for IndexedDB/interleaved-cursors-small.html:
> 
> r259704                       NOERROR
> r259705                       UNKNOWN
> r259706                       TEXT (Expected: PASS)
> 
> Other tests results are the same.
> 
> Diff:
> 
> ---
> /home/igalia/dpino/workspace/webkit/layout-test-results/retries/http/tests/
> appcache/history-test-expected.txt
> +++
> /home/igalia/dpino/workspace/webkit/layout-test-results/retries/http/tests/
> appcache/history-test-actual.txt
> @@ -1,3 +1,4 @@
> +CONSOLE MESSAGE: line 1: ApplicationCache is deprecated. Please use
> ServiceWorkers instead.
>  CONSOLE MESSAGE: line 1: ApplicationCache is deprecated. Please use
> ServiceWorkers instead.
>  Test whether window.history.back() is supported in application cache.

This is the cross-platform expected results, so this is a progression. We can just remove the gtk specific results.

Comment 4 Carlos Garcia Campos 2020-04-09 06:20:51 PDT

(In reply to Carlos Garcia Campos from comment #3)
> (In reply to Diego Pino from comment #0)
> > The following tests are failing after r259705:
> > 
> >    http/tests/appcache/history-test.html [ Failure ]
> >    imported/w3c/web-platform-tests/IndexedDB/interleaved-cursors-small.html
> > [ Failure ]
> >    imported/w3c/web-platform-tests/IndexedDB/large-requests-abort.html [
> > Failure ]
> >    imported/w3c/web-platform-tests/IndexedDB/request-event-ordering.html [
> > Failure ]
> > 
> > wk-testhunter for IndexedDB/interleaved-cursors-small.html:
> > 
> > r259704                       NOERROR
> > r259705                       UNKNOWN
> > r259706                       TEXT (Expected: PASS)
> > 
> > Other tests results are the same.
> > 
> > Diff:
> > 
> > ---
> > /home/igalia/dpino/workspace/webkit/layout-test-results/retries/http/tests/
> > appcache/history-test-expected.txt
> > +++
> > /home/igalia/dpino/workspace/webkit/layout-test-results/retries/http/tests/
> > appcache/history-test-actual.txt
> > @@ -1,3 +1,4 @@
> > +CONSOLE MESSAGE: line 1: ApplicationCache is deprecated. Please use
> > ServiceWorkers instead.
> >  CONSOLE MESSAGE: line 1: ApplicationCache is deprecated. Please use
> > ServiceWorkers instead.
> >  Test whether window.history.back() is supported in application cache.
> 
> This is the cross-platform expected results, so this is a progression. We
> can just remove the gtk specific results.

Committed r259794: <https://trac.webkit.org/changeset/259794>

Comment 5 Diego Pino 2020-05-11 22:09:28 PDT

imported/w3c/web-platform-tests/IndexedDB/large-requests-abort.html [ Failure ]
imported/w3c/web-platform-tests/IndexedDB/request-event-ordering.html [ Failure ]

fixed by r261533.