Closed Bug 1021765 Opened 5 years ago Closed 5 years ago

[flame] camera preview fps is very low

Categories

(Firefox OS Graveyard :: Gaia::Camera, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

(tracking-b2g:backlog)

RESOLVED FIXED
tracking-b2g backlog

People

(Reporter: sotaro, Assigned: vliu)

Details

Attachments

(1 file, 1 obsolete file)

+++ This bug was initially created as a clone of Bug #988704 +++

Camera preview is very slow on flame device. It is around 14 fps. Bug 988704 cover the gonk KK devices. But flame still uses JB, we need to apply same change also to JB.
(In reply to Sotaro Ikeda [:sotaro] from comment #0)
> +++ This bug was initially created as a clone of Bug #988704 +++
> 
> Camera preview is very slow on flame device. It is around 14 fps. Bug 988704
> cover the gonk KK devices. But flame still uses JB, we need to apply same
> change also to JB.

I think I can take this job to apply the same change to JB.
Assignee: nobody → vliu
(In reply to Vincent Liu[:vliu] from comment #1)
> (In reply to Sotaro Ikeda [:sotaro] from comment #0)
> > +++ This bug was initially created as a clone of Bug #988704 +++
> > 
> > Camera preview is very slow on flame device. It is around 14 fps. Bug 988704
> > cover the gonk KK devices. But flame still uses JB, we need to apply same
> > change also to JB.
> 
> I think I can take this job to apply the same change to JB.

Yeah!
User Story: (updated)
Nominate to "b2g-v1.4+". This degrade user experience of the camera app.
blocking-b2g: --- → 1.4?
Attached patch bug-1021765-fix.patch (obsolete) — Splinter Review
Hi Sotaro,

Can you please have a review? Thanks.
Attachment #8436502 - Flags: review?(sotaro.ikeda.g)
Comment on attachment 8436502 [details] [diff] [review]
bug-1021765-fix.patch

Review of attachment 8436502 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good :-)
Attachment #8436502 - Flags: review?(sotaro.ikeda.g) → review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/b2g-inbound/rev/344e71c36f9f

Please make sure your patch includes a good commit message when requesting checkin. I gave it my best shot at one, but I much prefer when the patch author does it instead :)
https://developer.mozilla.org/en-US/docs/Developer_Guide/Committing_Rules_and_Responsibilities#Checkin_comment
Keywords: checkin-needed
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #7)
> https://hg.mozilla.org/integration/b2g-inbound/rev/344e71c36f9f
> 
> Please make sure your patch includes a good commit message when requesting
> checkin. I gave it my best shot at one, but I much prefer when the patch
> author does it instead :)
> https://developer.mozilla.org/en-US/docs/Developer_Guide/
> Committing_Rules_and_Responsibilities#Checkin_comment

Hi Ryan,

Thanks for the suggestion.
Attachment #8436502 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/344e71c36f9f
Status: NEW → RESOLVED
Closed: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
blocking-b2g: 1.4? → backlog
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.