RESOLVED DUPLICATE of bug 107882 107179
REGRESSION (r134166-134170?): Fixed position div unexpectedly moves after table row removed
https://bugs.webkit.org/show_bug.cgi?id=107179
Summary REGRESSION (r134166-134170?): Fixed position div unexpectedly moves after tab...
Kevin M. Dean
Reported 2013-01-17 14:41:47 PST
Created attachment 183282 [details] Before A table row is removed via javascript and the fixed div jumps down the page until you click. This in WordPress. I've set up a wordpress test area if you want to login and take a look. Just respond here or send me an email directly for the access information and instructions to trigger the bug.
Attachments
Before (211.64 KB, image/png)
2013-01-17 14:41 PST, Kevin M. Dean
no flags
After (246.50 KB, image/png)
2013-01-17 14:42 PST, Kevin M. Dean
no flags
Kevin M. Dean
Comment 1 2013-01-17 14:42:09 PST
Radar WebKit Bug Importer
Comment 2 2013-01-17 15:04:23 PST
Alexey Proskuryakov
Comment 3 2013-01-18 10:59:49 PST
The regression range is very surprising. There weren't any even remotely related changes in r139554-r139561. > I've set up a wordpress test area if you want to login and take a look. Just respond here Yes, please.
Kevin M. Dean
Comment 4 2013-01-18 11:31:12 PST
(In reply to comment #3) > > I've set up a wordpress test area if you want to login and take a look. Just respond here > > Yes, please. Access info emailed to you.
Kevin M. Dean
Comment 5 2013-01-18 11:33:29 PST
(In reply to comment #3) > The regression range is very surprising. There weren't any even remotely related changes in r139554-r139561. Re-checking the range. I may have listed the wrong releases.
Kevin M. Dean
Comment 6 2013-01-18 11:39:28 PST
I see what happened. If I didn't scroll the page down a little first it didn't always show the bug causing me to see the wrong range. New range forthcoming.
Alexey Proskuryakov
Comment 7 2013-01-18 12:18:11 PST
This is not 100% reliably reproducible, so take this with a grain of salt, but my range is <http://trac.webkit.org/log/trunk/?rev=136455&stop_rev=136447>, with <http://trac.webkit.org/r136452> being the only suspicious revision in that range.
Kevin M. Dean
Comment 8 2013-01-18 12:40:20 PST
(In reply to comment #7) > This is not 100% reliably reproducible, so take this with a grain of salt, but my range is <http://trac.webkit.org/log/trunk/?rev=136455&stop_rev=136447>, with <http://trac.webkit.org/r136452> being the only suspicious revision in that range. In my testing so far, even though it gets really hard to produce, the issue is at least on or before r135805. Still checking...
Kevin M. Dean
Comment 9 2013-01-18 12:52:05 PST
Anything of interest between r135421-r135472 I've been able to reproduce with r135472, but struggling to get it to happen with r135421
Kevin M. Dean
Comment 10 2013-01-18 12:52:59 PST
(In reply to comment #9) > Anything of interest between r135421-r135472 > > I've been able to reproduce with r135472, but struggling to get it to happen with r135421 ...and no sooner than I said that, r135421 showed the bug.
Kevin M. Dean
Comment 11 2013-01-18 13:32:23 PST
My latest range is r134166-134170. Anything possible there? It's definitely easy to reproduce in the most recent builds, but much harder in the old builds. Something must have changed recently that exposes the bug more.
Alexey Proskuryakov
Comment 12 2013-01-18 13:39:13 PST
Yes, there are two patches that could be related. There is also a chance that this got broken multiple times, because it certainly at least became much easier to reproduce around my range.
Kevin M. Dean
Comment 13 2013-01-18 13:44:07 PST
(In reply to comment #12) > Yes, there are two patches that could be related. > > There is also a chance that this got broken multiple times, because it certainly at least became much easier to reproduce around my range. Yes, I just confirmed that the range for me where it get's easier to reproduce is r136402-r136460, which your previous suspicious revision r136452 falls within.
Robert Hogan
Comment 14 2013-01-28 11:30:45 PST
Kevin: can you add instructions for reproducing the bug please? Or even better, a reduced test case?
Kevin M. Dean
Comment 15 2013-01-28 11:37:17 PST
In WordPress on the Wordfence page, you'll see a New Issues section. Click on the "I have fixed the issue" links to remove rows and trigger the bug. Once you've removed all the rows, you'll need to click "Start a Wordfence Scan", to bring the Issues back for further testing.
Kevin M. Dean
Comment 16 2013-01-28 11:41:26 PST
Looks like the problem is no longer occurring in the recent nightly builds or it has gone back to being more difficult to reproduce like in builds prior to r136447
Beth Dakin
Comment 17 2013-01-28 14:41:54 PST
(In reply to comment #16) > Looks like the problem is no longer occurring in the recent nightly builds or it has gone back to being more difficult to reproduce like in builds prior to r136447 Kevin, could you send me the access information as well? I will try to narrow down when it became less reproducible or was fixed.
Kevin M. Dean
Comment 18 2013-01-28 15:06:49 PST
(In reply to comment #17) > Kevin, could you send me the access information as well? I will try to narrow down when it became less reproducible or was fixed. Emailed.
Beth Dakin
Comment 19 2013-01-28 15:27:52 PST
My testing indicates that this was fixed (or made less reproducible) by http://trac.webkit.org/changeset/140758 But that is pretty surprising! Since I am not sure how to explain that, I think we should keep this open for now in case this bug does still exist. Kevin, please let me know if you see the bug again.
Beth Dakin
Comment 20 2013-02-18 15:02:16 PST
Okay, I'm going to assume this was fixed since we haven't seen this again lately. *** This bug has been marked as a duplicate of bug 107882 ***
Note You need to log in before you can comment on or make changes to this bug.