Bug 48023

Summary:

fast/transforms/scrollIntoView-transformed.html should have different expectations now that transforms affect overflow

Product:

WebKit

Reporter:

Beth Dakin <bdakin>

Component:

CSS

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bdakin, ossy

Priority:

Version:

528+ (Nightly build)

Hardware:

OS:

OS X 10.5

Attachments:

Description	Flags
Patch to change the test	none

Beth Dakin

Reported 2010-10-20 16:14:35 PDT

https://bugs.webkit.org/show_bug.cgi?id=47514 caused transforms to affect overflow. That makes some assumptions in fast/transforms/scrollIntoView-transformed.html obsolete. The test needs to be updated to test what it originally intended to test.

Attachments
Patch to change the test (19.48 KB, patch) 2010-10-20 16:33 PDT, Beth Dakin	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Beth Dakin

Comment 1 2010-10-20 16:33:17 PDT

Created attachment 71357 [details] Patch to change the test I went back and forth for a little while trying to decide if I should remove the transforms part of this test entirely or just change it somehow to pass in a meaningful way. After reading over https://bugs.webkit.org/show_bug.cgi?id=41259 (which is the change which originally added this test) and the HTML 5 specification for scrollIntoView, I ultimately decided that the transforms part of the test is really not relevant to the original fix. So I decided to remove that part of the test.

Csaba Osztrogonác

Comment 2 2010-10-20 23:34:04 PDT

Comment on attachment 71357 [details] Patch to change the test https://bugs.webkit.org/show_bug.cgi?id=47514 broke the test on Qt too. :( Let's see if this one fixed it.

Csaba Osztrogonác

Comment 3 2010-10-21 02:10:50 PDT

Comment on attachment 71357 [details] Patch to change the test It was landed it http://trac.webkit.org/changeset/70192 .

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