Closed Bug 576179 Opened 10 years ago Closed 10 years ago

Remove return {} hack used for sync messages

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mfinkle, Assigned: mfinkle)

Details

Attachments

(1 file)

patch
4.89 KB, patch
vingtetun
: review+
Details | Diff | Splinter Review
Attached patch patch — — Splinter Review 
Since we removed our use of sendSyncMessage, we can remove the hack we used to stop JSON.parse errors (see bug 568818 which should still be fixed)
Attachment #455358 - Flags: review?(21)
(In reply to comment #0)
> Created an attachment (id=455358) [details]
> patch
> 
> Since we removed our use of sendSyncMessage, we can remove the hack we used to
> stop JSON.parse errors (see bug 568818 which should still be fixed)

Yep, I will work to reland it asap.
pushed:
http://hg.mozilla.org/mobile-browser/rev/32ccee634739
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
bugspam
Assignee: nobody → mark.finkle
You need to log in before you can comment on or make changes to this bug.