[window management] add an 'opaque' flag for the shell so that other surfaces are not composited

Bug #1089962 reported by Florian Boucault
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
touch-preview-images
Invalid
High
Unassigned
unity8 (Ubuntu)
Expired
High
Unassigned

Bug Description

Each QtQuick window that does not need to be transparent (e.g. dash) used by the shell needs to be set the 'opaque' flag supported by qtubuntu:
  view = new QQuickView();
  view->setProperty("opaque", opaque);

This will result in increased rendering performance.

Changed in manhattan:
status: New → Confirmed
importance: Undecided → Critical
Changed in manhattan:
assignee: nobody → Florian Boucault (fboucault)
description: updated
Bill Filler (bfiller)
Changed in manhattan:
milestone: none → demo-final
Revision history for this message
Florian Boucault (fboucault) wrote :

Won't work until the shell is split up into multiple windows (one for indicators, one for launcher, one for dash).

Bill Filler (bfiller)
Changed in manhattan:
importance: Critical → High
milestone: final → update-1
description: updated
tags: added: performance
Changed in manhattan:
assignee: Florian Boucault (fboucault) → nobody
milestone: update-1 → none
Andrea Cimitan (cimi)
tags: added: unity-public
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

is this still valid?

information type: Proprietary → Public
affects: manhattan → touch-preview-images
Changed in touch-preview-images:
assignee: nobody → Florian Boucault (fboucault)
status: Confirmed → Incomplete
Revision history for this message
Florian Boucault (fboucault) wrote :

Still valid.

Changed in unity8:
status: New → Confirmed
Changed in touch-preview-images:
status: Incomplete → Confirmed
Changed in unity8:
importance: Undecided → High
Changed in touch-preview-images:
assignee: Florian Boucault (fboucault) → nobody
Revision history for this message
Michał Sawicz (saviq) wrote :

It's not clear whether we're going to split the shell into separate components in the end.

Changed in unity8:
status: Confirmed → Incomplete
Revision history for this message
Florian Boucault (fboucault) wrote :

If you don't do it you will be using more memory than necessary. For example when in an app, you only need a surface as big as the launcher to show the launcher. Right now the surface is the size of the screen.

Revision history for this message
Florian Boucault (fboucault) wrote :

Also, it's not about splitting the shell into separate components but in separate QQuickWindows.

Revision history for this message
Florian Boucault (fboucault) wrote :

And it's not just memory we are wasting but also a ton of GPU cycles given the massive overdraw it incurs.
I imagine fixing this bug would increase performance very very significantly.

Revision history for this message
Michał Sawicz (saviq) wrote :

This will be out of scope once QtCompositor gets in.

Changed in touch-preview-images:
status: Confirmed → Invalid
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for Unity 8 because there has been no activity for 60 days.]

Changed in unity8:
status: Incomplete → Expired
Michał Sawicz (saviq)
affects: unity8 → unity8 (Ubuntu)
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.