Error: "Unable to play video. No URL." + "Aborting background thread." in MediaPortal.log

[GoogleCodeExporter]

What steps will reproduce the problem?
1. find video with long buffering (or set timeout to lower values)
2. browse to video and try to play

What is the expected output? What do you see instead?
Expected: No error.
Instead: Sometimes "Unable to play video. No URL." error occurs.

What version of the product are you using? On what operating system?
MediaPortal 1.1.1, OnlineVideos 0.26.0.0, Windows 7 Ultimate

Please provide any additional information below.

The problem is probably in Gui2UtilConnector class, 
ExecuteInBackgroundAndCallback() method. The problem can be reproduced each 
time when you insert "Thread.Sleep(1000);" immediately bellow  
"backgroundThread.Start();".

Reason: When you start background thread with "backgroundThread.Start();" and 
this thread ends before "if (timeout && 
!System.Diagnostics.Debugger.IsAttached) timeoutTimer.Start();" then 
timeoutTimer is stopped before it was started. If before playing video is 
executed any action through ExecuteInBackgroundAndCallback() method and this 
happens then timeoutTimer is running!

Resolution: I recommend to place "backgroundThread.Start();" immediately bellow 
"if (timeout && !System.Diagnostics.Debugger.IsAttached) timeoutTimer.Start();" 
and place "timeoutTimer.Stop();" into catch section.

Original issue reported on code.google.com by juraj.ho...@inmail.sk on 20 Nov 2010 at 8:48

[GoogleCodeExporter]

The current trunk already contains exactly your proposal :-) 
(http://code.google.com/p/mp-onlinevideos2/source/detail?r=1001)
Can you build a version from svn to confirm the issue is fixed?

Original comment by bborgsd...@gmail.com on 22 Nov 2010 at 7:43

• Changed state: Started

[GoogleCodeExporter]

I built a version (0.27.0.0) from svn and this issue is fixed.

Original comment by juraj.ho...@inmail.sk on 22 Nov 2010 at 8:38

[GoogleCodeExporter]

Thanks for confirming and reporting back!

Original comment by bborgsd...@gmail.com on 22 Nov 2010 at 9:01

• Changed state: Verified