Closed
Bug 779283
Opened 12 years ago
Closed 12 years ago
Camera - ###!!! ASSERTION: Invalid image dimensions!: 'data.mCbCrStride & 0x7 == 0', file /Volumes/b2g/B2G/gecko/dom/camera/GonkCameraPreview.cpp, line 143
Categories
(Firefox OS Graveyard :: General, defect)
Firefox OS Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dzbarsky, Assigned: mikeh)
References
Details
Attachments
(1 file, 1 obsolete file)
2.50 KB,
patch
|
Details | Diff | Splinter Review |
I hit this assertion when opening the camera preview on Nexus S. Let me know if you can't reproduce and I can try to get stack traces for you.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #647755 -
Flags: review?(dzbarsky)
Assignee | ||
Updated•12 years ago
|
Summary: ###!!! ASSERTION: Invalid image dimensions!: 'data.mCbCrStride & 0x7 == 0', file /Volumes/b2g/B2G/gecko/dom/camera/GonkCameraPreview.cpp, line 143 → Camera - ###!!! ASSERTION: Invalid image dimensions!: 'data.mCbCrStride & 0x7 == 0', file /Volumes/b2g/B2G/gecko/dom/camera/GonkCameraPreview.cpp, line 143
Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 647755 [details] [diff] [review] Fix observed assertions r=me
Attachment #647755 -
Flags: review?(dzbarsky) → review+
Assignee | ||
Updated•12 years ago
|
blocking-basecamp: --- → ?
Reporter | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9263673d378c
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•12 years ago
|
||
Now with 100% more headers.
Attachment #647755 -
Attachment is obsolete: true
Reporter | ||
Comment 6•12 years ago
|
||
I already landed the patch, I was just telling you for future reference ;)
Comment 7•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9263673d378c
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
blocking-basecamp: ? → ---
Comment 8•12 years ago
|
||
Was this fix important enough that we'd block if it regressed? Please re-basecamp? if so. Thanks.
Assignee | ||
Comment 9•12 years ago
|
||
(In reply to Andrew Overholt [:overholt] from comment #8) > Was this fix important enough that we'd block if it regressed? Please > re-basecamp? if so. Thanks. Functionally, no change. But it clutters the logcat output with incorrect assertions.
You need to log in
before you can comment on or make changes to this bug.
Description
•