Re: Good news in dillo's rendering!

On Wed, Jul 13, 2016 at 01:37:12PM -0400, Jorge Arellano Cid wrote:
> [...]

>   @Sebastian:  Problem  #10 is not related to GROWS2, but a three
> year  old  bug  discovered during the tests. So let's get it done
> first. How to reproduce it: Go to [2], wait a bit for most images
> to  load, then use "find text" to find the "re-re" string, scroll
> a  bit  downwards  to  find the image and center it. Go back then
> forward;  no  image!  But  dillo thinks it *is* there so move the
> cursor  to the center of the page and you'll see the hand cursor,
> wait  for  the  tooltip and drag it a bit downwards to the right.
> Dillo will repaint from the image data.
>   Bisect  shows  the  bug  was introduced by #2863. Attached is a
> simple concept patch for it. Please review/fix/commit, as I don't
> know the updated invariants of resize idle.
> 
> 
> [1] http://lists.dillo.org/pipermail/dillo-dev/2016-June/010887.html
> [2] http://tinyurl.com/huvf6pn

  As usual, the attachment arrives later...  ;)

-- 
  Cheers
  Jorge.-

_______________________________________________
Dillo-dev mailing list
[email protected]
http://lists.dillo.org/cgi-bin/mailman/listinfo/dillo-dev

--- layout.cc	2016-07-13 13:21:34.000000000 -0400
+++ layout.cc.ok1	2016-07-07 19:57:30.000000000 -0400
@@ -867,6 +867,8 @@
    //static int calls = 0;
    //printf ("Layout::resizeIdle calls = %d\n", ++calls);
 
+while (resizeIdleId != -1) {
+
    assert (resizeIdleId != -1);
 
    for (typed::Iterator <Widget> it = queueResizeList->iterator();
@@ -951,7 +953,7 @@
 
       // views are redrawn via Widget::resizeDrawImpl ()
    }
-
+}
    updateAnchor ();
 
    DBG_OBJ_MSGF ("resize", 1,