Closed
Bug 1294634
Opened 8 years ago
Closed 8 years ago
Use a switch statement to convert State to string
Categories
(Core :: Audio/Video: Playback, defect, P5)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
FIXED
mozilla51
Tracking | Status | |
---|---|---|
firefox51 | --- | fixed |
People
(Reporter: jwwang, Assigned: jwwang)
Details
Attachments
(1 file)
https://hg.mozilla.org/mozilla-central/file/0502bd9e025edde29777ba1de4280f9b52af4663/dom/media/MediaDecoderStateMachine.cpp#l1045 We got screwed whenever adding a field without changing this string table. A switch statement can warn us when the fields are not exhausted.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → jwwang
Priority: -- → P5
Comment hidden (mozreview-request) |
Assignee | ||
Updated•8 years ago
|
Attachment #8780408 -
Flags: review?(bechen)
Comment 2•8 years ago
|
||
mozreview-review |
Comment on attachment 8780408 [details] Bug 1294634 - Use a switch statement to convert State to string. https://reviewboard.mozilla.org/r/71148/#review68628
Attachment #8780408 -
Flags: review?(bechen) → review+
Assignee | ||
Comment 3•8 years ago
|
||
Thanks!
Comment hidden (mozreview-request) |
Pushed by jwwang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7b74ee1d97db Use a switch statement to convert State to string. r=bechen
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/7b74ee1d97db
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox51:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in
before you can comment on or make changes to this bug.
Description
•