Closed Bug 917647 Opened 11 years ago Closed 11 years ago

JSONUtils is never used.

Categories

(Firefox for Android Graveyard :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: ckitching, Unassigned)

Details

Attachments

(1 obsolete file)

Attached patch Remove JSONUtils (obsolete) — Splinter Review
mobile/android/base/util/JSONUtils is dead code. Let's prune it.

At least, I'm fairly confident it is. Just in case I'm losing my mind:
https://tbpl.mozilla.org/?tree=Try&rev=d066eab7a781
Attachment #806412 - Flags: review?(margaret.leibovic)
JSONUtils was landed as part of the Push Notifications work, which is itself not 100% landed yet. See bug 834033. I don't think we can remove this code, even though it is not really used yet.
Comment on attachment 806412 [details] [diff] [review]
Remove JSONUtils

Ah. So I've run into it before it's finished landing. Sorry about that.
Attachment #806412 - Attachment is obsolete: true
Attachment #806412 - Flags: review?(margaret.leibovic)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: