RESOLVED FIXED 13375
REGRESSION (r20901): failing fast/overflow/scrollRevealButton.html 
https://bugs.webkit.org/show_bug.cgi?id=13375
Summary REGRESSION (r20901): failing fast/overflow/scrollRevealButton.html
mitz
Reported 2007-04-17 12:22:03 PDT
fast/overflow/scrollRevealButton.html fails to scroll the button into view when it's focused. Regressed in <http://trac.webkit.org/projects/webkit/changeset/20901>.
Attachments
Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2007-04-17 13:17:55 PDT
Wow, did this change really cause the regression? I have no idea how!
mitz
Comment 2 2007-04-17 13:32:11 PDT
(In reply to comment #1) > Wow, did this change really cause the regression? Yes. > I have no idea how! I suspect it's the removal of the updateLayout() call from setIsActive(), that just happened to be at right place at the right time to ensure that everything is set up for scroll-to-reveal to do its thing. Without it, some of the dimensions are (0, 0) which messes up the scrolling. Even prior to r20901, opening the test in a background tab, for example, would make it fail. I've readded an updateLayout() back in order to test my theory. If it's right, then that call should probably be added in a more appropriate place to ensure that scroll-to-reveal has all the information it needs.
mitz
Comment 3 2007-04-17 13:57:04 PDT
(In reply to comment #2) > I suspect it's the removal of the updateLayout() call from setIsActive(). Confirmed.
Darin Adler
Comment 4 2007-04-17 14:56:30 PDT
(In reply to comment #3) > Confirmed. To me it seems fine to just add an updateLayout to setIsActive, check that in, and then later try to come up with a better way to fix this problem.
mitz
Comment 5 2007-04-18 22:48:56 PDT
Fixed by Darin in r20936.
Note You need to log in before you can comment on or make changes to this bug.