1187634 - Various find as you type bustage from bug 1133981

Status: RESOLVED FIXED

(SeaMonkey :: Find In Page, defect)

Description

[[e-mail]]

Bug 1133981 caused several issues:
* Toggling the autostart preference no longer works
* The findbar does not work in the browser
* The findbar always steals type ahead find in view source

Comment 1

[[e-mail]]

Attachment: Proposed patch

* Fix autostart issue by a) using new variable name in constructor and pref observer b) updating the browser in the pref observer (the parent constructor already updates the browser)
* Rename the usefindbar variable because it's far too long
* Remove obsolete code from destroy method
* Remove obsolete _shouldFastFind method
* Fix browser issue by copying receiveMessage method and removing browser test (since SeaMonkey has one find bar for the entire tabbrowser)
* Fix view source issue by making the receiveMessage method also check that we're using the find bar before passing on the keypress

Note that the browser fix makes it steal type ahead find in the same way as the view source issue but the fix for that fixes the browser again too.

Comment 2

[Philip Chee]

Comment on attachment 8638953 [details] [diff] [review]
Proposed patch

Start SeaMonkey go to some page type "/" findbar opens.

Open more tabs. type "/" FAYT message shows up in the status bar for all tabs except the first one.

Comment 3

[[e-mail]]

Attachment: Updated for review comments

* Switched to using the window message manager in the tabbrowser so that the findbar receives messages from all tabs
* Provided an override of the _updateBrowserWithState function which uses the window message manager (the function name is different)

Comment 4

[[e-mail]]

Pushed comm-central changeset 5befa64efa62.

Comment 5

[[e-mail]]

Comment on attachment 8642057 [details] [diff] [review]
Updated for review comments

[Approval Request Comment]
Regression caused by (bug #): 1133981
User impact if declined: Erratic Find bar behaviour
Testing completed (on m-c, etc.): Landed on c-c
Risk to taking this patch (and alternatives if risky): Not risky
String changes made by this patch: None

Comment 6

[Ian Neal]

Comment on attachment 8642057 [details] [diff] [review]
Updated for review comments

[Triage Comment]
a=me for the relevant branches it is needed on.

Comment 7

[[e-mail]]

(In reply to Ian Neal from comment #6)
> a=me for the relevant branches it is needed on.

Whoa, I hadn't realised it was uplift time already.
In that case, yeah, I need c-b and c-r.

Comment 8

[Ian Neal]

Comment on attachment 8642057 [details] [diff] [review]
Updated for review comments

Removing the unneeded c-a a=

Comment 9

[[e-mail]]

https://hg.mozilla.org/releases/comm-beta/rev/4cb7d9c42615
https://hg.mozilla.org/releases/comm-release/rev/58111896d3d0