1615239 - Clear search history in SeaMonkey search box is not working

Status: RESOLVED FIXED

(SeaMonkey :: Search, task)

Description

[Frank-Rainer Grahl (:frg)]

Reported over irc:
TypeError: this.FormHistory is undefined[Learn More] search.xml:600:15

NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS: [JavaScript Error: "this.FormHistory is undefined" {file: "chrome://communicator/content/search/search.xml" line: 600}]'[JavaScript Error: "this.FormHistory is undefined" {file: "chrome://communicator/content/search/search.xml" line: 600}]' when calling method: [nsIController::doCommand]

Bug 1442925 moved the import to the searchbar but I am still puzzled why 2.57 does not throw. Probably it does not because there are still bugs in it and the code is not called. The autocomplete is only semi working (blank line at the start of the popup among other problems).

Comment 1

[Frank-Rainer Grahl (:frg)]

Attachment: 1615239-searchclear.patch

Does not work for clearing search from the sidebar yet.

Comment 2

[Frank-Rainer Grahl (:frg)]

Attachment: 1615239-searchclear.patch

[Approval Request Comment]
Regression caused by (bug #): 1442925
User impact if declined: clearing search histroy does not work. Needs to be cleared in the data manager
Testing completed (on m-c, etc.): 2.53 2.57(has issues)
Risk to taking this patch (and alternatives if risky): none already broken.
String changes made by this patch: --

The safest approach seems to be an additional FormHistory import in the text box. It was done differently with just an import before Bug 1442925 but seems to be cleaner using a field now.
At least the sidebar search field does not habe a searchbar and also not a parent Binding (second problem discovered and fixed) so we can't just use the Formhistory from the searchbar.

Does not throw in 2.57 but also not removing history because of other problems which need to be fixed first. The patch should then work fine for it too.

Btw. just opening a customize toolbar box constructs a search box but never calls the destructor. Something to fix for later and not related to the existing bugs here.

Comment 3

[Ian Neal]

Comment on attachment 9126436 [details] [diff] [review]
1615239-searchclear.patch

[Triage Comment]
LGTM r/a=me

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/915e5658d0b9
Define FormHistory in the search textbox and don't assume that a search bar exists. r=IanN

Comment 5

[Frank-Rainer Grahl (:frg)]

Fixed 2.53.1
https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/43950a1b2d51ff0311022b87fd1f52f5eedf477c