Bug 20499 - REGRESSION(r35203 - r35231): Find fails to scroll to matches in embedded iframes
Summary: REGRESSION(r35203 - r35231): Find fails to scroll to matches in embedded iframes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Nobody
URL: data:text/html,<iframe style="positio...
Keywords: HasReduction, Regression
Depends on:
Blocks:
 
Reported: 2008-08-24 11:03 PDT by Gavin Sherlock
Modified: 2009-07-03 07:53 PDT (History)
1 user (show)

See Also:


Attachments
reduction (1.56 KB, text/html)
2008-08-24 16:23 PDT, Gavin Sherlock
no flags Details
better reduction based on last comment (438 bytes, text/html)
2008-08-24 17:21 PDT, Gavin Sherlock
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Gavin Sherlock 2008-08-24 11:03:18 PDT
Steps to reproduce:

1. Go to http://www.urbanmusicblog.net/category/new-music/
2.  Search for "down" on the page (without quotes) - there should be 5 or 6
3.  Cycle through the matches using apple-G

Expected results:

All matches should be shown as you cycle through them

Actual results:

Several within an iFrame are not shown.  This is a regression from 3.1.2, tested with r35900 on 10.5.4 on a MacPro
Comment 1 Gavin Sherlock 2008-08-24 11:21:45 PDT
Regressed between r35203 and r35231.

Most likely suspect is:

http://trac.webkit.org/changeset/35223
Comment 2 Gavin Sherlock 2008-08-24 16:23:11 PDT
Created attachment 22965 [details]
reduction

reduction, though note, this depends on google ads being generated
Comment 3 Gavin Sherlock 2008-08-24 16:25:03 PDT
Changing title - when I was creating the reduction, I realized that it fails to scroll to the iframe matches, rather than failing to see or highlight them (delete the br tags to see).  I don't know how to further reduce the iframe to that doesn't require on the external resource of the google ads though, so this was the best I could do.
Comment 4 Matt Lilek 2008-08-24 17:15:34 PDT
(In reply to comment #3)
> Changing title - when I was creating the reduction, I realized that it fails to
> scroll to the iframe matches, rather than failing to see or highlight them
> (delete the br tags to see).  I don't know how to further reduce the iframe to
> that doesn't require on the external resource of the google ads though, so this
> was the best I could do.
> 

Load data:text/html,<iframe style="position: absolute; top: 125%;" src="data:text/html, pudding"> and search for "pudding".
Comment 5 Gavin Sherlock 2008-08-24 17:21:36 PDT
Created attachment 22966 [details]
better reduction based on last comment

Not sure if you need this given the data url, but though I'd add it anyway.
Comment 6 Gavin Sherlock 2009-07-03 07:53:20 PDT
Not sure when this was fixed, and it would be nice if there were a test, but closing as this now works (in Safari 4.0x, and r45463)