Closed
Bug 1179162
Opened 9 years ago
Closed 9 years ago
[FLAME-L][Video] When opening Video app, the video crashes
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: hcheng, Assigned: mchiang)
References
Details
(Whiteboard: [FLAME-L-LOCALBUILD])
Attachments
(1 file)
+++ This bug was initially created as a clone of Bug #1171836 +++
### STR
1. Open Video App
### Actual
Video app crashes and even adb logcat is terminated...
### Expected
Video app open correctly
### Crash Report
https://crash-stats.mozilla.com/report/index/249a2e5b-b420-4f00-9ca6-7e57a2150701
### Version
Local build
Build ID 20150630184049
Gaia Revision 5997b406e77ea726fbd9047057a1c3504f6cd6d4
Gaia Date 2015-06-30 01:48:01
Gecko Revision n/a
Gecko Version 42.0a1
Device Name flame
Firmware(Release) 5.0
Firmware(Incremental) eng.hermesc.20150630.182707
Firmware Date 二 6月 30 18:27:28 CST 2015
Bootloader L1TC00011L60
|
Reporter | |
Updated•9 years ago
|
Summary: [FLAME-L][Camera] When opening Video app, the video crashes → [FLAME-L][Video] When opening Video app, the video crashes
|
Assignee | |
Updated•9 years ago
|
Assignee: nobody → mchiang
|
|
Assignee | |
Comment 3•9 years ago
|
||
Attachment #8631480 -
Flags: review?(sotaro.ikeda.g)
|
||
Comment 4•9 years ago
|
||
Comment on attachment 8631480 [details]
cherry pick QCT patch for YUV Venus format color convertor
Looks good!
FYI, but there are more preferred way to this problem from gecko point of view. It is Bug 1120780 and gralloc hal's fix. Actual change is the below.
> https://github.com/mozilla-b2g/hardware_qcom_display/commit/7c83e14bfad823c66d643bf4e81b862ad22eab7b
It is used by nexus-5-l, it fixed the problem almost all cases, but it caused the regression( bug 1147763). The regression is caused by POVB problem :-( flame-kk already use the color convertor side fix and Bug 1120780 fix has a regression risk. Therefore, the color convertor side fix is OK for now.
Attachment #8631480 -
Flags: review?(sotaro.ikeda.g) → review+
|
|
||
Updated•9 years ago
|
Component: Gaia::Video → Graphics
Product: Firefox OS → Core
|
|
||
Updated•9 years ago
|
|
|
||
Updated•9 years ago
|
|
|
||
Comment 5•9 years ago
|
||
|
|
Assignee | |
Updated•9 years ago
|
Keywords: checkin-needed
|
||
Comment 6•9 years ago
|
||
seems sotaro did this already the checkin, is there more to do here ?
Flags: needinfo?(mchiang)
Keywords: checkin-needed
|
|
Assignee | |
Comment 7•9 years ago
|
||
Yes, Sotaro has finished merging the patch.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•9 years ago
|
Flags: needinfo?(mchiang)
You need to log in
before you can comment on or make changes to this bug.
Description
•