WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
49611
REGRESSION (
r72115
): 5 layout tests are broken
https://bugs.webkit.org/show_bug.cgi?id=49611
Summary
REGRESSION (r72115): 5 layout tests are broken
Csaba Osztrogonác
Reported
2010-11-16 12:11:42 PST
It broke 3 tests on all bot: fast/dom/frame-loading-via-document-write.html fast/dom/HTMLHeadElement/head-check.html fast/frames/iframe-no-src-set-location.html
http://build.webkit.org/results/SnowLeopard%20Intel%20Release%20%28Tests%29/r72115%20%2820918%29/results.html
Attachments
Add attachment
proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1
2010-11-16 14:35:35 PST
***
Bug 49620
has been marked as a duplicate of this bug. ***
Csaba Osztrogonác
Comment 2
2010-11-16 14:38:14 PST
The tests are still fail and the commit queue is block because of this bug. :( Any progression with the fix? Or should we rollout the patch?
David Levin
Comment 3
2010-11-16 14:40:16 PST
Actually it is 5 tests: fast/dom/frame-loading-via-document-write.html fast/dom/HTMLHeadElement/head-check.html fast/dom/Window/open-existing-pop-up-blocking.html fast/frames/iframe-no-src-set-location.html fast/frames/location-change.html 3 failed immediately and 2 more in later runs (which was odd but they have the same error, so I believe they are the same).
Darin Adler
Comment 4
2010-11-16 14:48:29 PST
It’s fine to roll out the patch for now. I should have run those tests before checking in. I’ll resubmit the patch with fixes for those tests. Sorry, I am in the middle of some other work and can’t take care of it myself.
Csaba Osztrogonác
Comment 5
2010-11-16 14:50:58 PST
(In reply to
comment #4
)
> It’s fine to roll out the patch for now. I should have run those tests before checking in. I’ll resubmit the patch with fixes for those tests. > > Sorry, I am in the middle of some other work and can’t take care of it myself.
I'm going to do the rollout. But please reopen the original bug, and comment it, because we don't have the power to do it.
Darin Adler
Comment 6
2010-11-16 14:51:32 PST
Committed
r72137
.
Darin Adler
Comment 7
2010-11-16 14:51:50 PST
(In reply to
comment #5
)
> I'm going to do the rollout. But please reopen the original bug, > and comment it, because we don't have the power to do it.
Sorry, I did it!
Csaba Osztrogonác
Comment 8
2010-11-16 14:53:11 PST
(In reply to
comment #7
)
> Sorry, I did it!
Not problem, you was the faster. :) Thx.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug