Closed Bug 1216180 Opened 9 years ago Closed 9 years ago

[TV] SyncManager does not set the error state properly

Categories

(Firefox OS Graveyard :: Gaia::TV::System, defect, P1)

Product:
Firefox OS Graveyard
Component:
Gaia::TV::System
Platform:
ARM
Gonk (Firefox OS)
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
FxOS-S10 (30Oct)

People

(Reporter: ferjm, Assigned: ferjm)

References

Details

(Whiteboard: [partner-cherry-pick])

Attachments

(1 file)

[gaia] ferjm:bug1216180.syncmanager.tv.error > mozilla-b2g:master
46 bytes, text/x-github-pull-request
yifan
: review+
Details | Review 
We are setting the state name instead of the error.
Assignee: nobody → ferjmoreno
Blocks: fxos-sync
Priority: -- → P1
Target Milestone: --- → FxOS-S10 (30Oct)
Attachment #8675754 - Flags: review?(yliao)
Component: Sync → Gaia::TV::System
Comment on attachment 8675754 [details] [review]
[gaia] ferjm:bug1216180.syncmanager.tv.error > mozilla-b2g:master

LGTM. Thank you!
Attachment #8675754 - Flags: review?(yliao) → review+
https://github.com/mozilla-b2g/gaia/commit/f95dbdf32ef1d9520022e11415947d4beff08b0f
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [partner-cherry-pick]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: