Bug 73999

Summary:

[Qt][WK2] REGRESSION(102228): lot of test failures after enabled page cache

Product:

WebKit

Reporter:

Balazs Kelemen <kbalazs>

Component:

WebKit2

Assignee:

Caio Marcelo de Oliveira Filho <cmarcelo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

cmarcelo, hausmann, ossy, zalan

Priority:

Keywords:

LayoutTestFailure, Qt, QtTriaged

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

77033

Bug Blocks:

73938

Attachments:

Description	Flags
Patch	kenneth: review+

Balazs Kelemen

Reported 2011-12-07 07:00:27 PST

9 tests broke. Before: http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r102227%20(16628)/results.html After: http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r102228%20(16629)/results.html

Attachments
Patch (2.10 KB, patch) 2012-01-26 10:43 PST, Caio Marcelo de Oliveira Filho	kenneth: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Balazs Kelemen

Comment 1 2011-12-07 07:50:32 PST

Skipped tests in http://trac.webkit.org/changeset/102236

Caio Marcelo de Oliveira Filho

Comment 2 2012-01-25 11:59:53 PST

These regressions are being caused because tests are expected to run WITHOUT PageCache enabled. The tests that use page cache enable it by overridePreference(), which isn't implemented in WTR yet. I've made a patch for bug 77033 that makes WTR disable page cache usage and implement enough of overridePreference() so we don't regress the tests we were passing "by accident" (those which test page cache).

Caio Marcelo de Oliveira Filho

Comment 3 2012-01-26 10:43:21 PST

Created attachment 124143 [details] Patch

Caio Marcelo de Oliveira Filho

Comment 4 2012-01-26 11:58:06 PST

Committed r106023: <http://trac.webkit.org/changeset/106023>

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