Preview generation should not rely on glReadPixels

Bug #1407640 reported by Thomas Voß
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Confirmed
High
Unassigned
qtvideo-node (Ubuntu)
Confirmed
High
Jim Hodapp

Bug Description

Right now, snapshot/preview generation relies on a call to QGlFramebufferObject::toImage to obtain a QImage in http://bazaar.launchpad.net/~phablet-team/qtvideo-node/trunk/view/head:/src/snapshotgenerator.cpp#L118. Ultimately, this results in a very expensive call to glReadPixels, blocking the buffer processing of the camera app. Instead, we should avoid mapping the pixels to main memory and just copy the texture.

Changed in qtvideo-node (Ubuntu):
importance: Undecided → High
Changed in canonical-devices-system-image:
assignee: nobody → John McAleely (john.mcaleely)
importance: Undecided → High
milestone: none → backlog
Changed in canonical-devices-system-image:
status: New → Confirmed
Changed in canonical-devices-system-image:
assignee: John McAleely (john.mcaleely) → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in qtvideo-node (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.