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)
SeaMonkey
Search
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.1b1
People
(Reporter: iannbugzilla, Assigned: kairo)
References
()
Details
Attachments
(1 file)
(deleted),
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
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 :-(
Assignee | ||
Updated•14 years ago
|
Summary: navigatorOverlay.xul still has a reference to BrowserSearchInternet → navigatorOverlay.xul still has a reference to BrowserSearchInternet (Tools > Search the Web doesn't work)
Assignee | ||
Comment 1•14 years ago
|
||
With this patch, "Tools > Search the Web" should work again.
Comment 2•14 years ago
|
||
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+
Assignee | ||
Comment 3•14 years ago
|
||
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.
Description
•