After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 701764 - Startup time takes too long and initial view freezes
Startup time takes too long and initial view freezes
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-music-maint
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2013-06-06 23:02 UTC by Seif Lotfy
Modified: 2013-06-07 09:05 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to lazy load views (2.09 KB, patch)
2013-06-06 23:03 UTC, Seif Lotfy 		needs-work Details | Review
Improve initial startup time by lazy loading views (3.15 KB, patch)
2013-06-07 08:58 UTC, Vadim Rutkovsky 		none Details | Review

Description Seif Lotfy 2013-06-06 23:02:43 UTC 
Upon startup of the application it takes time until the views are usable.
This is due all views loading their first 50 items and trying to run discover and lookup (for the thumbnails)
The solution is to lazy load the views (upon request)
Comment 1 Seif Lotfy 2013-06-06 23:03:36 UTC 
Created attachment 246200 [details] [review]
Patch to lazy load views
Comment 2 Vadim Rutkovsky 2013-06-07 08:30:17 UTC 
Review of attachment 246200 [details] [review]:

Doesn't pass tests:
Running suite tests_albumPlayback.js
Gjs-Message: JS LOG: running test testAlbumViewPlayback
Gtk-Message: Failed to load module "pk-gtk-module"
Gtk-Message: Failed to load module "canberra-gtk-module"
Detected new source availabe: 'Tracker' and it supports search

(gjs:12283): Gjs-WARNING **: JS ERROR: Exception in callback for signal: ready: TypeError: this.header_bar.getStack(...) is null
Comment 3 Vadim Rutkovsky 2013-06-07 08:58:35 UTC 
Created attachment 246231 [details] [review]
Improve initial startup time by lazy loading views

Updated tests to create a stack and set visible child there. Removed duplicated function
Comment 4 Seif Lotfy 2013-06-07 08:59:12 UTC 
NICE
Please push
Comment 5 Vadim Rutkovsky 2013-06-07 09:05:46 UTC 
pushed as 952588aec1cf7e6dd86552847c358db7aa99d444