[Spice-devel] [spice-HTML5][PATCH try 2] Handling non-topdown lz_rgb

[Pavel Grunt]

Hi Fabio,

On Mon, 2015-09-07 at 13:35 +0200, Fabio Fantoni wrote:
> Il 03/09/2015 15:28, Pavel Grunt ha scritto:
> > Hi,
> > 
> > On Thu, 2015-09-03 at 11:52 +0200, Fabio Fantoni wrote:
> > > Hi, based also on js console output:
> > > http://pastebin.com/L9Rr3Y25
> > > I tried this patch for fix the non-topdown problem (remove one of the
> > > FIXME type):
> > > http://lists.freedesktop.org/archives/spice-devel/2013-March/012590.html
> > > 
> > > I applied it clean on latest spice-html5 from source (official git
> > > repository) but with it show nothing on screen at all
> > yeah, there is a typo in the patch. You can get the 'fixed' patch from my
> > git
> > repo in the 'topdown' branch: http://cgit.freedesktop.org/~pgrunt/spice-html
> > 5/ 
> > 
> > Thanks,
> > Pavel
> 
> Thanks for your help, with the 3 additional patches of your topdown branch
> the topdown problem is solved but remain the other image problems.
> Here a small screen recording: fantu.it/vari/spice-html5.webm
> Here the js console output: http://fantu.it/vari/spice-html5-js.txt
> 
> Probably about topdown patch itself that seems ok now is good reply to this
> report about it:
> https://bugzilla.redhat.com/show_bug.cgi?id=1014066
> And apply it upstream if is ok after other users test.
>  
> If you need more information and/or tests tell me and I'll post them.
> 
> Thanks for any reply and sorry for my bad english.
> 
You can try it without my last patch. There are about 60 FIXMEs and a half of
them is in display.js. Fixing DrawCopy / DrawFill could help with your problem
:)

Pavel