[Webkit-unassigned] [Bug 159138] [GTK][EFL] Build with threaded compositor enabled is broken

[bugzilla-daemon at webkit.org]

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

--- Comment #4 from Miguel Gomez <magomez at igalia.com> ---
(In reply to comment #3)
> Comment on attachment 282115 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=282115&action=review
> 
> > Source/WebCore/platform/graphics/cairo/ImageBufferCairo.cpp:-124
> > -    cairoSurfaceSetDeviceScale(m_compositorSurface.get(), m_resolutionScale, m_resolutionScale);
> 
> Are you sure we don't need this here? Shouldn't we pass the resolution scale
> to the data structure instead?

Sure, we don't need it. The compositor buffer is only used to copy the canvas content and take it to the composition stage when the accelerated canvas is enabled. And according to the comments in r202421, the HiDPI canvas feature was removed, so when using a canvas the resolution factor is always 1.

But we can store the resolutionScale in the data structure as well, yes. If you prefer that approach I can update the patch.

-- 
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/20160627/a4fe954b/attachment.html>