Bug 48023 - fast/transforms/scrollIntoView-transformed.html should have different expectations now that transforms affect overflow
Summary: fast/transforms/scrollIntoView-transformed.html should have different expecta...
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2010-10-20 16:14 PDT by Beth Dakin
Modified: 2010-10-21 02:11 PDT (History)
2 users (show)

See Also:

Patch to change the test (19.48 KB, patch)
2010-10-20 16:33 PDT, Beth Dakin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Beth Dakin 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.
Comment 1 Beth Dakin 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.
Comment 2 Csaba Osztrogonác 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.
Comment 3 Csaba Osztrogonác 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 .