[Webkit-unassigned] [Bug 159455] [GTK] Web view background colors don't work in accelerated compositing mode

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=159455

--- Comment #5 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to comment #4)
> (In reply to comment #3)
> > Comment on attachment 282882 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=282882&action=review
> > 
> > The compatibility issue should be mentioned when you prepare release notes
> > for 2.14.
> > 
> > > Source/WebKit2/ChangeLog:12
> > > +        window has an RGBA visual, even when setting a fully opaque brakcground, because we still need the web process
> > 
> > background
> > 
> > > Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp:3519
> > > + * #GtkWidget:app-paintable property set to %TRUE, for backgrounds colors to work.
> > 
> > Get rid of this comma.
> > 
> > > Source/WebKit2/UIProcess/gtk/RedirectedXCompositeWindow.cpp:192
> > > +    xDamageNotifier().add(m_window.get(), [this, damageNotify = WTFMove(damageNotify)] {
> > 
> > Is it normal WebKit style to put spaces around the assignment operator here?
> > I think it's easier to read without spaces.
> 
> Yes.

Yes, that it's normal webkit style, I don't think removing the spaces makes it easier to read ta all.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160721/d3c751e9/attachment.html>