195120 - FAYT fails after switching to compose window, typing a key, and returning

Status: VERIFIED FIXED

(SeaMonkey :: Find In Page, defect)

Description

[sairuh (rarely reading bugmail)]

found using 2003.02.26 comm trunk builds. FAYT fails after copy something from
browser and pasting in composer.

0. use the [default] FAYT setting to search links only.
1. open a page (or tab) with links, such as http://mozilla.org
2. make sure that FAYT works --try typing "net" and verify it's selected/gets focus.
3. copy something from the page --either highlight (drag select) some text and
hit accel+C, or select "copy link location" from a link's context menu.
4. open a new composer window (accel+4 or accel+shift+N).
5. paste what you copied --i used accel+V.
6. switch back to the browser window --i just mouseclicked back there.
7. try FAYT again --i just started typing the first chars of a link, eg, "moz".

results: nothing is found (highlighted), FAYT no longer works.

workaround: if i reload the page, FAYT starts working again.

Comment 1

[sairuh (rarely reading bugmail)]

this regressed sometime btwn the 2/13 (works fine) and 2/17 (broken) builds.
narrowing some more...

Comment 2

[sairuh (rarely reading bugmail)]

using linux comm bits, this regressed btwn the 2003.02.13.22 (works) and
2003.02.14.05 (broken) builds.

Comment 3

[Aaron Leventhal]

Attachment: Make sure that when window changes, we cancel the current find, so that it knows we've changed windows

Comment 4

[Aaron Leventhal]

Changing summary, to more accurately reflect problem.

Comment 5

[Aaron Leventhal]

As I said, the new summary is more accurate.

You don't have to paste, all you have to do is switch to the compose window and
type a key. Then find as you type won't start again.

Comment 6

[Aaron Leventhal]

checked in

Comment 7

[Asa Dotzler [:asa]]

Comment on attachment 116166 [details] [diff] [review]
Make sure that when window changes, we cancel the current find, so that it knows we've changed windows 

a=asa for checkin to the 1.3 branch. not going to hold for this but if it makes
it into the branch before we get final bits, that'd be nice.

Comment 8

[Asa Dotzler [:asa]]

Comment on attachment 116166 [details] [diff] [review]
Make sure that when window changes, we cancel the current find, so that it knows we've changed windows 

aaron says not for 1.3. removing 1.3+

Comment 9

[sairuh (rarely reading bugmail)]

vrfy'd fixed with 2003.03.13 comm bits, all platforms.