Ubuntu
qtubuntu package

First scope not rendered until unlocked

Bug #1620297 reported by Paweł Stołowski
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Critical
Michał Sawicz
Canonical System Image 13 "OTA13"
qtubuntu (Ubuntu)
Fix Released
Critical
Michał Sawicz

Bug Description

The pre-populating of first scope on startup (before the screen is unlocked) doesn't seem to be fully working.

My early investigation shows that the shell plugin correctly pre-populates scope model on startup when the screen is still locked, but after unlocking scope view is white for 1-2 seconds and only then the page is rendered and all the images are requested, so the perception is as if pre-population didn't work. The perceived performance depends on whether the 1st scope is a local scope (such as Apps) or a scope which needs remote images (and whether the images are already cached by the thumbnailer in the latter case).

This might be caused by https://bugs.launchpad.net/qtmir/+bug/1475678

See original description

Related branches

lp:~saviq/qtubuntu/revert-occlude-lp1620297
Rejected for merging into lp:qtubuntu
Paweł Stołowski (community): Approve
Unity8 CI Bot: Needs Fixing (continuous-integration)
Diff: 69 lines (+2/-17)
1 file modified
src/ubuntumirclient/window.cpp (+2/-17)
lp:~dandrader/qtmir/dontOccludeFreshSurfaces
Gerry Boland (community): Approve
Unity8 CI Bot (community): Approve (continuous-integration)
Diff: 61 lines (+22/-4)
2 files modified
src/modules/Unity/Application/mirsurface.cpp (+16/-4)
src/modules/Unity/Application/mirsurface.h (+6/-0)
Revision history for this message
Paweł Stołowski (stolowski) wrote :

Attaching a snippet of unity8-dash.log. Apps scope is my 1st scope.

Note, how it's pre-populated @15:03:44 on startup and delivers 29 results:
[2016-09-05:15:03:44.547] Pre-populating first scope: "clickscope"

I waited ~2 minutes before unlocking the device - this is when I think rendering started (and the scope was set active after unlocking)

[2016-09-05:15:05:48.394] file:///usr/share/unity8/Dash/GenericScopeView.qml:390: TypeError: Cannot read property 'defaultMinimumHorizontalSpacing' of null
[2016-09-05:15:05:48.790] file:///usr/share/unity8/Dash/GenericScopeView.qml:390: TypeError: Cannot read property 'defaultMinimumHorizontalSpacing' of null
[2016-09-05:15:05:49.048] "clickscope" : setActive: true

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

I'm afraid the fix to bug #1475678 caused this... Dash isn't told to render until the screen is unlocked. Wonder if we should only tell apps to not render after a few seconds... Or make the dash an exception...

affects: unity8 (Ubuntu) → qtubuntu (Ubuntu)
Changed in canonical-devices-system-image:
status: New → Confirmed
Paweł Stołowski (stolowski)
description: updated
Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
status: Confirmed → In Progress
assignee: nobody → Michał Sawicz (saviq)
Changed in qtubuntu (Ubuntu):
status: New → In Progress
assignee: nobody → Michał Sawicz (saviq)
Changed in canonical-devices-system-image:
importance: Undecided → Low
importance: Low → High
Changed in qtubuntu (Ubuntu):
importance: Undecided → Critical
importance: Critical → High
Changed in canonical-devices-system-image:
milestone: none → 13
importance: High → Critical
Changed in qtubuntu (Ubuntu):
importance: High → Critical
Revision history for this message
Paweł Stołowski (stolowski) wrote :

Yep, the attached MP fixes the problem.

Jean-Baptiste Lallement (jibel)
Changed in canonical-devices-system-image:
status: In Progress → Fix Committed
Michał Sawicz (saviq)
Changed in qtubuntu (Ubuntu):
status: In Progress → Fix Released
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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