250198 - "find as you type" doesn't work

Status: RESOLVED FIXED

(SeaMonkey :: Find In Page, defect)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

cvs build from today, linux/gtk2

typeaheadfind doesn't work: just typing letters doesn't do anything - nothing
appears in statusbar, links aren't being selected

Typing / does show a "starting" message in the statusbar, but typing letters
does not have any effect.

Comment 1

[Aaron Leventhal]

Mozilla 1.8a2
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a2) Gecko/20040707

Comment 2

[Aaron Leventhal]

Attachment: Partial backout #1

Comment 3

[Aaron Leventhal]

Attachment: Partial backout #2

Comment 4

[Aaron Leventhal]

Not sure if either of these work, but worth trying.

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

these patches make no difference for me (compiling in content/ and layout/ (make
libs))

Comment 6

[Christian :Biesinger (don't email me, ping me on IRC)]

backing out Bug 171366 does NOT fix this bug

Comment 7

[Patrick]

*** Bug 250341 has been marked as a duplicate of this bug. ***

Comment 8

[Christian :Biesinger (don't email me, ping me on IRC)]

hm, I thought this was trunk-only.... (and thus can't block aviary1.0, since
that's on a branch and unaffected)

Comment 9

[Christian :Biesinger (don't email me, ping me on IRC)]

caused by bug 246447

Comment 10

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: patch

readd line from that bug that apparently shouldn't have been deleted

Comment 11

[Ilya Konstantinov]

Please r and sr the last patch as soon as possible. It's restoring a line which 
should've never been removed (by me). I erroneously removed this line in my 
patch for bug	246447. 

Comment 12

[Johnny Stenback (:jst)]

Comment on attachment 152618 [details] [diff] [review]
patch

r=jst

Comment 13

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in widget/src/gtk2/nsWindow.cpp;
/cvsroot/mozilla/widget/src/gtk2/nsWindow.cpp,v  <--  nsWindow.cpp
new revision: 1.112; previous revision: 1.111
done

Comment 14

[Benjamin Smedberg]

Is this needed on the aviary branch?

Comment 15

[Asa Dotzler [:asa]]

This was a trunk only regression so I don't think we need this for the branch.