Closed Bug 1059052 Opened 5 years ago Closed 5 years ago
Decoder State Machine state transition - centralize state transitions
Spin off of bug 1054831 comment 1 and bug 1054831 comment 2. Centralizing state transitions will allow us to track state changes more easily and it is also a prerequisite for adding entry/exit actions to state transition to enhance our state machine.
Move all |mState = XXX| to SetState() and add logs to track state transition. Try: https://tbpl.mozilla.org/?tree=Try&rev=67a69ceb7138 The status of B2G ICS Emulator Opt is bad. Most of them are bug 1059120. So I guess my patch is fine then.
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8480460 - Flags: review?(cpearce)
Attachment #8480460 - Flags: review?(cpearce) → review+
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.