Closed Bug 601466 Opened 14 years ago Closed 14 years ago

navigatorOverlay.xul still has a reference to BrowserSearchInternet (Tools > Search the Web doesn't work)

Categories

(SeaMonkey :: Search, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1b1

People

(Reporter: iannbugzilla, Assigned: kairo)

References

()

Details

Attachments

(1 file)

See https://bugzilla.mozilla.org/show_bug.cgi?id=410613#c58
Comment on attachment 479086 [details] [diff] [review]
v2.1: address Neil's comments

>-      BrowserSearchInternet();
>+      BrowserSearch.webSearch();
Missed a call in navigatorOverlay.xul :-(
Summary: navigatorOverlay.xul still has a reference to BrowserSearchInternet → navigatorOverlay.xul still has a reference to BrowserSearchInternet (Tools > Search the Web doesn't work)
With this patch, "Tools > Search the Web" should work again.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #480753 - Flags: review?(neil)
Comment on attachment 480753 [details] [diff] [review]
Fix this instance as well

[I now hit the null checks as previously mentioned.]
Attachment #480753 - Flags: review?(neil) → review+
Landed as http://hg.mozilla.org/comm-central/rev/d6f93d5b6f45
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1b1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: