Bug 159138 – [GTK][EFL] Build with threaded compositor enabled is broken

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Miguel Gomez]

The changes in r202421 and r202439 broke the build.

[Miguel Gomez]

Created attachment 282115 [details]
Patch

[WebKit Commit Bot]

Attachment 282115 [details] did not pass style-queue:


ERROR: Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.h:49:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.h:50:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.cpp:43:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebKit2/Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.cpp:49:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 4 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Carlos Garcia Campos]

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?

[Miguel Gomez]

(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.

[Carlos Garcia Campos]

Comment on attachment 282115 [details]
Patch

Ok, then

[WebKit Commit Bot]

Comment on attachment 282115 [details]
Patch

Clearing flags on attachment: 282115

Committed r202484: <http://trac.webkit.org/changeset/202484>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.