queue2: percentage is relative to high-percent
authorThiago Santos <thiagoss@osg.samsung.com>
Sun, 23 Nov 2014 08:45:24 +0000 (05:45 -0300)
committerThiago Santos <thiagoss@osg.samsung.com>
Sun, 23 Nov 2014 08:51:12 +0000 (05:51 -0300)
When comparing percentage values, compare with 0-100 scale as it
has already been made relative to 0-high_percent, otherwise we mark
the queue as not buffering and report a 50% to the user. This leads to
a buffering stall as the user assumes the queue is still buffering but
it thinks it isn't.

https://bugzilla.gnome.org/show_bug.cgi?id=736969

plugins/elements/gstqueue2.c

index 3f887f0bd2c19e1950dcf30954dc222e22dffd16..dd862a315b3aca7221ed29665619955d2babedfe 100644 (file)
@@ -953,7 +953,7 @@ update_buffering (GstQueue2 * queue)
 
   if (queue->is_buffering) {
     /* if we were buffering see if we reached the high watermark */
-    if (percent >= queue->high_percent)
+    if (percent >= 100)
       queue->is_buffering = FALSE;
 
     SET_PERCENT (queue, percent);