Closed
Bug 1057487
Opened 10 years ago
Closed 10 years ago
No need to call stop() in chromecast code
Categories
(Firefox for Android Graveyard :: Screencasting, defect)
Tracking
(firefox33 fixed, firefox34 fixed)
RESOLVED
FIXED
People
(Reporter: wesj, Unassigned)
References
Details
Attachments
(1 file)
1.09 KB,
patch
|
wesj
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
This already landed on nightly in a different patch (bug 1055338). Needs to go to aurora. Sorry for the mixup.
Reporter | ||
Comment 1•10 years ago
|
||
Comment on attachment 8477545 [details] [diff] [review] Patch Approval Request Comment [Feature/regressing bug #]: casting [User impact if declined]: crash when video finishes casting. [Describe test coverage new/current, TBPL]: working on em [Risks and why]: Low risk. stop() does nothing but call the callback passed to it. Its a stub. Passing null just makes it crash. [String/UUID change made/needed]: none.
Attachment #8477545 -
Flags: review+
Attachment #8477545 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
Attachment #8477545 -
Attachment is patch: true
Updated•10 years ago
|
status-firefox33:
--- → affected
status-firefox34:
--- → fixed
Updated•10 years ago
|
Attachment #8477545 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Reporter | ||
Comment 2•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/9251ac84b3e4
Updated•10 years ago
|
Reporter | ||
Comment 3•10 years ago
|
||
backed out for bustage (left over mirroring code): https://hg.mozilla.org/releases/mozilla-aurora/rev/52f54e8b5414 will reland shortly.
Reporter | ||
Comment 4•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/439ceee39acb
Reporter | ||
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•