Closed Bug 345319 Opened 19 years ago Closed 19 years ago

Installation of an engine with a very long name can fail

Categories

(Firefox :: Search, defect)

Product:
Firefox
Component:
Search
Version:
2.0 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 2 beta2

People

(Reporter: Gavin, Assigned: Gavin)

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

patch
1.49 KB, patch
mconnor
: review+
darin.moz
: approval1.8.1+
Details | Diff | Splinter Review
This is the search equivalent to bug 342212. The same patch applies since the two use the same function for generating getting unique files.
Attached patch patch — — Splinter Review
Assignee: nobody → gavin.sharp
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 2 beta2
Attachment #229956 - Flags: review?(mconnor)
mconnor: this is the same fix that already landed for the microsummary code (bug 342212)
Whiteboard: [patch-r?]
Attachment #229956 - Flags: review?(mconnor) → review+
Whiteboard: [patch-r?] → [checkin needed]
mozilla/browser/components/search/nsSearchService.js 1.58
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed] → [need-a]
Comment on attachment 229956 [details] [diff] [review] patch This is a trivial fix, and it's the same patch that already landed on the trunk for microsummaries.
Attachment #229956 - Flags: approval1.8.1?
Whiteboard: [need-a] → [a?]
Comment on attachment 229956 [details] [diff] [review] patch a=darin on behalf of drivers
Attachment #229956 - Flags: approval1.8.1? → approval1.8.1+
Whiteboard: [a?] → [checkin needed (1.8 branch)]
mozilla/browser/components/search/nsSearchService.js 1.1.2.48
Keywords: fixed1.8.1
Whiteboard: [checkin needed (1.8 branch)]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: