Closed Bug 506563 Opened 15 years ago Closed 15 years ago

Use native trim() in FeedProcessor.js

Categories

(Firefox Graveyard :: RSS Discovery and Preview, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 3.6a1

People

(Reporter: rockmfr, Assigned: rockmfr)

References

Details

Attachments

(1 file)

Replace trimString with native trim in FeedProcessor.js.
yeah, good idea.
Attached patch patchSplinter Review
There's at least one place where trim() might be called on null, but this patch doesn't make that issue better or worse. I'll file a follow-up for that.
Assignee: nobody → rockmfr
Status: NEW → ASSIGNED
Attachment #390763 - Flags: review?(sayrer)
No longer blocks: 305064
Depends on: 305064
Attachment #390763 - Flags: review?(sayrer) → review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/2a3953a1606c
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.6a1
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: