Closed Bug 1209914 Opened 8 years ago Closed 8 years ago

Use toJSON methods instead of explicit _serializeToJSON calls

Categories

(Firefox :: Search, defect, P4)

Product:
Firefox
Component:
Search
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 44
Tracking Flags:
Tracking Status
firefox44 --- fixed

People

(Reporter: florian, Assigned: florian)

Details

(Whiteboard: [fxsearch])

Attachments

(1 file)

Patch
3.77 KB, patch
adw
: review+
Details | Diff | Splinter Review
Attached patch Patch — — Splinter Review 
- The aFilter parameter of SRCH_ENG__serializeToJSON(aFilter) is never 'used' (it's always true), and makes the code harder to read.
- The explicit _serializeToJSON calls make the code just more complicated, when naming the methods toJSON achieves the same behavior automatically.
Attachment #8667818 - Flags: review?(adw)
Attachment #8667818 - Flags: review?(adw) → review+
https://hg.mozilla.org/mozilla-central/rev/7ac681cd9e4f
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox44: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
You need to log in before you can comment on or make changes to this bug.