Re: RFR: 8271398: GTK3 drag view image swaps red and blue color channels [v3]

2021-09-17 Thread Kevin Rushforth
On Tue, 17 Aug 2021 13:12:41 GMT, Pankaj Bansal wrote: >> Thiago Milczarek Sayao has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Change test to manual > > The fix works fine and the test passes on all platforms. I have given minor >

Re: RFR: 8271398: GTK3 drag view image swaps red and blue color channels [v3]

2021-08-20 Thread Thiago Milczarek Sayao
On Tue, 17 Aug 2021 13:10:53 GMT, Pankaj Bansal wrote: >> Thiago Milczarek Sayao has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Change test to manual > > tests/manual/dnd/DndTestDragViewRawImage.java line 59: > >> 57: }); >>

Re: RFR: 8271398: GTK3 drag view image swaps red and blue color channels [v3]

2021-08-17 Thread Pankaj Bansal
On Sun, 15 Aug 2021 21:18:51 GMT, Thiago Milczarek Sayao wrote: >> It seems raw images need to be converted BRGA -> RGBA. >> >> It was being converted on gtk2 code path, but gtk3 only uses >> `gtk_drag_set_icon_pixbuf`. >> >> I have simplified the gtk2 `DragView::View::expose` to paint with

Re: RFR: 8271398: GTK3 drag view image swaps red and blue color channels [v3]

2021-08-15 Thread Thiago Milczarek Sayao
On Sun, 15 Aug 2021 21:18:51 GMT, Thiago Milczarek Sayao wrote: >> It seems raw images need to be converted BRGA -> RGBA. >> >> It was being converted on gtk2 code path, but gtk3 only uses >> `gtk_drag_set_icon_pixbuf`. >> >> I have simplified the gtk2 `DragView::View::expose` to paint with

Re: RFR: 8271398: GTK3 drag view image swaps red and blue color channels [v3]

2021-08-15 Thread Thiago Milczarek Sayao
> It seems raw images need to be converted BRGA -> RGBA. > > It was being converted on gtk2 code path, but gtk3 only uses > `gtk_drag_set_icon_pixbuf`. > > I have simplified the gtk2 `DragView::View::expose` to paint with > `gdk_cairo_set_source_pixbuf` (that is available since Gtk 2.8)