Bug 20499 – REGRESSION(r35203 - r35231): Find fails to scroll to matches in embedded iframes

• Component: New Bugs
• Status: RESOLVED FIXED

[Gavin Sherlock]

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

[Gavin Sherlock]

Regressed between r35203 and r35231.

Most likely suspect is:

http://trac.webkit.org/changeset/35223

[Gavin Sherlock]

Created attachment 22965 [details]
reduction

reduction, though note, this depends on google ads being generated

[Gavin Sherlock]

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.

[Matt Lilek]

(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".

[Gavin Sherlock]

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.

[Gavin Sherlock]

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)