Last modified: 2007-09-09 06:58:33 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 11245 - OggHandler Cortado stop button doesn't work right, makes java go out to lunch.
OggHandler Cortado stop button doesn't work right, makes java go out to lunch.
Product: MediaWiki extensions
Classification: Unclassified
OggHandler (Other open bugs)
All All
: Normal major (vote)
: ---
Assigned To: Tim Starling
Depends on:
  Show dependency treegraph
Reported: 2007-09-09 00:35 UTC by Gregory Maxwell
Modified: 2007-09-09 06:58 UTC (History)
0 users

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Description Gregory Maxwell 2007-09-09 00:35:48 UTC
The stop button does not appear to work correctly and causes misbehavior in Firefox with the cortado player using Sun JVM 1.5 or 1.6. (confirmed on Linux with JVM 1.5 and windows with JVM 1.6)

I'll give detailed reproduction instructions but you can reproduce this with pretty much anything:

Go to  Play the first example. When it finishes. Hit stop. Notice that java goes out to lunch and no player work for 10 seconds or so. Scrolling the screen makes the player iframes go blank.

Now go to

Observe that the same behavior doesn't happen.

I think this is also related to the player intermittently going out to lunch before it starts but thats harder to reproduce.
Comment 1 Tim Starling 2007-09-09 01:00:43 UTC
Can't reproduce. What sort of lunch? Sandwich?
Comment 2 Gregory Maxwell 2007-09-09 01:19:17 UTC
Mmm.. What kind of lunch? Something stale no doubt....

The issue is 100% reproducible for me, and apparently some other folks... Shall I send you a VMware player image with a Linux install that experiences this? ;) I'm not sure what to say.. if I use the patched jar that OggHander uses on my external player it gives the same behavior.

I also get peroidic clicking in the output audio when I use the OggHandler jar file. The clicking doesn't happen with the jar I've been using.

What did you compile that jar with? I used Sun javac 1.6.0 with -source and -target set to um.. I think 1.2.. whichever was the lowest version number that didn't have issues compiling the code.
Comment 3 Gregory Maxwell 2007-09-09 01:21:16 UTC
My primary test case is w/ Java(TM) Plug-in 1.5.0_09-b03 on "Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv: Gecko/20070510 Fedora/ Firefox/". Also had someone else test with FF2 on Windows XP with Java(TM) Platform SE 6 U1.
Comment 4 Gregory Maxwell 2007-09-09 02:15:06 UTC
I recompiled the plugin with our patch using sun javac 1.6 and ant.  It doesn't have the problem. I've asked tim to replace the copy on our site.
Comment 5 Gregory Maxwell 2007-09-09 03:17:59 UTC
The new jar was not displaying the status bar when status height = applet height.  This was as coded and should have never worked. I'm guessing it worked at all because slight differences between swing and awt behavior exposed by building against the wrong target version.

I changed the code to allow them to be equal.. since thats actually harmless. The jar compiled with this change is

--- ./src/com/fluendo/player/       2007-09-09 03:05:45.693764000 +0000
+++ ./src/com/fluendo/player/      2007-09-09 03:05:45.678859000 +0000
@@ -302,7 +302,7 @@
         int dheight = getSize().height;
         /* sometimes dimension is wrong */
-        if (dwidth <= 0 || dheight < statusHeight) {
+        if (dwidth <= 0 || dheight <= statusHeight) {
          appletDimension = null;

Comment 6 Gregory Maxwell 2007-09-09 03:20:11 UTC
(above patch is backwards ;) )
Comment 7 Gregory Maxwell 2007-09-09 06:58:33 UTC
Domas pushed out the new jar. All is good.

Note You need to log in before you can comment on or make changes to this bug.