Lyrics fetching blocks UI

Bug #224731 reported by Jamie Bennett
2
Affects Status Importance Assigned to Milestone
Entertainer Media Center
Triaged
High
Unassigned

Bug Description

Reported by charclo.michael, Apr 11, 2008

What steps will reproduce the problem?
1. Play a song
2. See the progress bar window
3. Screen freezes after 2 seconds

What is the expected output? What do you see instead?
-The regular window with the progress bar
-Frozen window

What version of the product are you using? On what operating system?
rev255 on ubuntu

Please provide any additional information below.
This will probably only happen when leoslyrics.com is really slow. When I
go to the site right now it takes 10 seconds to load.

Tags: defect
Revision history for this message
Jamie Bennett (jamiebennett) wrote :

** IMPORTED FROM GOOGLE CODE **

Changed in entertainer:
importance: Undecided → Medium
Paul Hummer (rockstar)
Changed in entertainer:
assignee: nobody → charclo-michael
importance: Medium → High
status: New → Triaged
Paul Hummer (rockstar)
Changed in entertainer:
milestone: none → 0.1-release
Changed in entertainer:
assignee: charclo-michael → nobody
Revision history for this message
Joshua Scotton (joshuascotton) wrote : Re: Leoslyrics blocks UI

Lyrics fetching needs to be threaded and have a timeout.
If that has been done in Charclo's changes can we mark this as done?

Revision history for this message
Charclo (charclo-michael) wrote :

I added a timeout of 5 seconds for fetching the url in my code. gobject timeout is also used with a callback but it doesn't seem to be working correctly so this bug can't be closed yet.

Revision history for this message
Paul Hummer (rockstar) wrote :

Adding the timeout the request is already added in charclo's branch, so if it takes too long, it times and and moves on (thus not giving you lyrics, but not blocking for too long). It's a fair workaround, and so this can be moved off the 0.1 release.

Changed in entertainer:
milestone: 0.1-release → none
Revision history for this message
Jamie Bennett (jamiebennett) wrote :

Is it a complete workaround or is a more elegant solution required?

Paul Hummer (rockstar)
Changed in entertainer:
importance: High → Critical
milestone: none → entertainer-0.2
Paul Hummer (rockstar)
Changed in entertainer:
milestone: entertainer-0.2 → none
Paul Hummer (rockstar)
Changed in entertainer:
importance: Critical → High
Revision history for this message
Matt Layman (mblayman) wrote :

The lyrics tab isn't using gobject timeout's at all. I think that this should be fairly easy to fix.

Changed in entertainer:
assignee: nobody → laymansterms
status: Triaged → In Progress
Revision history for this message
Matt Layman (mblayman) wrote :

I've not actually been working on this. I gave a quick stab at it a while ago, but it didn't work. I didn't do any further investigation and I haven't picked this back up.

Changed in entertainer:
assignee: laymansterms → nobody
status: In Progress → Triaged
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.