Summary: | REGRESSION (r82185): iframe not displayed after navigation back to cached page | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Beth Dakin <bdakin> | ||||
Component: | Layout and Rendering | Assignee: | Beth Dakin <bdakin> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, bdakin, eric, webkit.review.bot | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | OS X 10.5 | ||||||
Attachments: |
|
Description
Beth Dakin
2011-04-22 12:51:57 PDT
Created attachment 90745 [details]
Patch
I made a test that works for this in Safari, but it is not a valid DRT test since it passes in DRT even without my patch.
Comment on attachment 90745 [details]
Patch
What's different about DRT? IS it the fact that DRT doesn't use overlay scrollers? I'd suggest at least adding it as a manual test so we don't forget about it. r=me
(In reply to comment #2) > (From update of attachment 90745 [details]) > What's different about DRT? IS it the fact that DRT doesn't use overlay scrollers? I'd suggest at least adding it as a manual test so we don't forget about it. r=me No, the bug occurs with all types of scrollbars. For some reason DRT just lays out and paints the iframe after going back to the page, which is not true in Safar/WebKit without this patch. I thought it would at least work as a pixel test, but it does not. There must be something sufficiently different about about DRT's back mechanism and "back" in the browser that causes the necessary layout to happen. I can add it as a manual test though. Do we have manual tests checked in anymore? I can't seem to find them. Committed the change with revision 84688. Not sure if there's a place for my still-manual test, so I will just hold onto it for now. > Do we have manual tests checked in anymore? I can't seem to find them.
Source/WebCore/manual-tests. Not that anyone ever runs those.
(In reply to comment #6) > > Do we have manual tests checked in anymore? I can't seem to find them. > > Source/WebCore/manual-tests. Not that anyone ever runs those. WebCore! That's where they are hiding. Yeah, it's not super useful, but I will check it in there anyway. Manual test added with revision 84694. http://trac.webkit.org/changeset/84688 might have broken SnowLeopard Intel Release (Build) http://trac.webkit.org/changeset/84694 might have broken SnowLeopard Intel Release (Build) http://trac.webkit.org/changeset/84695 might have broken SnowLeopard Intel Release (Build) |