Bug 230480 - REGRESSION(r282735): build failing on gtk/gcc
Summary: REGRESSION(r282735): build failing on gtk/gcc
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-20 07:37 PDT by Jim Mason
Modified: 2021-09-22 08:45 PDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Mason 2021-09-20 07:37:17 PDT 
Building with gcc 10, gtk 3.  Still breaking as of r282742.

Reversing the commit r282735 makes the build complete normally.

Build failure is as follows:

In file included from /build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/LegacyInlineFlowBox.cpp:44,
                 from WebCore/DerivedSources/unified-sources/UnifiedSource-043dd90b-4.cpp:7:
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:45: error: 'InlineContent' in namespace 'WebCore::LayoutIntegration' does not name a type
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                             ^~~~~~~~~~~~~
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:67: error: 'Layout' does not name a type; did you mean 'TextLayout'?
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                                                   ^~~~~~
      |                                                                   TextLayout
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:78: error: expected unqualified-id before '&' token
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                                                              ^
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:78: error: expected ')' before '&' token
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                   ~                                                          ^
      |                                                                              )
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:78: error: constructors may not be ref-qualified
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:81: warning: ISO C++ forbids declaration of 'PaintInfo' with no type [-fpermissive]
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                                                                 ^~~~~~~~~
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:81: error: expected ';' at end of member declaration
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                                                                 ^~~~~~~~~
      |                                                                                          ;
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:50:91: error: expected unqualified-id before ',' token
   50 |     TextBoxPainter(const LayoutIntegration::InlineContent&, const Layout::Run&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                                                                           ^
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:58:58: error: 'PaintInfo' is not a type
   58 |     TextBoxPainter(LayoutIntegration::TextRunIterator&&, PaintInfo&, const LayoutPoint& paintOffset);
      |                                                          ^~~~~~~~~
/build/rocketman/components/desktop/webkitgtk4-dev/webkit/Source/WebCore/rendering/TextBoxPainter.h:92:5: error: 'PaintInfo' does not name a type
   92 |     PaintInfo& m_paintInfo;
      |     ^~~~~~~~~
[2758/5099] Building CXX object Source...sources/UnifiedSource-043dd90b-6.cpp.o
ninja: build stopped: subcommand failed.
gmake: *** [/build/rocketman/make-rules/justmake.mk:65: /build/rocketman/components/desktop/webkitgtk4-dev/build/amd64/.built] Error 1
Comment 1 Jim Mason 2021-09-22 08:45:29 PDT 
This has been fixed by r282788 <ross.kirsling@sony.com>: "Unreviewed build fix for WinCairo with ENABLE_EXPERIMENTAL_FEATURES off"

Thank you, Ross!