Bug 235984 - Fixes for non-unified builds after r288868
Summary: Fixes for non-unified builds after r288868
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Pablo Saavedra
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-02-01 16:15 PST by Pablo Saavedra
Modified: 2022-02-02 12:43 PST (History)
12 users (show)

See Also:


Attachments
patch (2.23 KB, patch)
2022-02-01 16:16 PST, Pablo Saavedra
no flags Details | Formatted Diff | Diff
patch (2.34 KB, patch)
2022-02-02 11:43 PST, Pablo Saavedra
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pablo Saavedra 2022-02-01 16:15:01 PST
SSIA

Related-to: https://bugs.webkit.org/show_bug.cgi?id=235938
Comment 1 Pablo Saavedra 2022-02-01 16:16:49 PST
Created attachment 450583 [details]
patch
Comment 2 Chris Dumez 2022-02-01 16:18:43 PST
Comment on attachment 450583 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=450583&action=review

> Source/WebCore/rendering/RenderInline.h:26
> +#include "RenderBlockFlow.h"

What is this for? I don't see it being obviously used in this header?
Comment 3 Chris Dumez 2022-02-01 16:20:54 PST
Comment on attachment 450583 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=450583&action=review

>> Source/WebCore/rendering/RenderInline.h:26
>> +#include "RenderBlockFlow.h"
> 
> What is this for? I don't see it being obviously used in this header?

r288868 didn't actually touch this header.
Comment 4 zalan 2022-02-01 16:48:39 PST
(In reply to Chris Dumez from comment #2)
> Comment on attachment 450583 [details]
> patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=450583&action=review
> 
> > Source/WebCore/rendering/RenderInline.h:26
> > +#include "RenderBlockFlow.h"
> 
> What is this for? I don't see it being obviously used in this header?
RenderInline should have no business of referencing RenderBlockFlow in the header.
Comment 5 Darin Adler 2022-02-01 17:29:10 PST
Comment on attachment 450583 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=450583&action=review

>>>> Source/WebCore/rendering/RenderInline.h:26
>>>> +#include "RenderBlockFlow.h"
>>> 
>>> What is this for? I don't see it being obviously used in this header?
>> 
>> r288868 didn't actually touch this header.
> 
> RenderInline should have no business of referencing RenderBlockFlow in the header.

We need to see the error you are fixing with this include, so we can suggest a better solution.
Comment 6 Pablo Saavedra 2022-02-02 05:44:27 PST
Yeah, sorry for the noise, I think I've anticipated to much the r? flag. Let me work a bit in the problem. I will upload the error traces ASAP.
Comment 7 Pablo Saavedra 2022-02-02 11:43:06 PST

Related errors:

```
/home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorInlineBox.cpp: In function �~@~XWebCore::InlineIterator::InlineBoxIterator WebCore::InlineIterator::firstRootInlineBoxFor(const WebCore::RenderBlockFlow&)�~@~Y:
/home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorInlineBox.cpp:125:30: error: invalid use of incomplete type �~@~Xconst class WebCore::RenderBlockFlow�~@~Y
  125 |     return { BoxLegacyPath { block.firstRootBox() } };
      |                              ^~~~~
In file included from /home/bot/webkit/Source/WebCore/rendering/RenderElement.h:27,
                 from /home/bot/webkit/Source/WebCore/rendering/RenderLayerModelObject.h:25,
                 from /home/bot/webkit/Source/WebCore/rendering/RenderBoxModelObject.h:29,
                 from /home/bot/webkit/Source/WebCore/rendering/LegacyInlineBox.h:24,
                 from /home/bot/webkit/Source/WebCore/rendering/LegacyInlineTextBox.h:25,
                 from /home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorBoxLegacyPath.h:28,
                 from /home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorBox.h:28,
                 from /home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorInlineBox.h:28,
                 from /home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorInlineBox.cpp:27:
/home/bot/webkit/Source/WebCore/rendering/RenderObject.h:65:7: note: forward declaration of �~@~Xclass WebCore::RenderBlockFlow�~@~Y
   65 | class RenderBlockFlow;
      |       ^~~~~~~~~~~~~~~
```


```
/home/bot/webkit/Source/WebCore/rendering/RenderBlockFlow.cpp: In member function �~@~Xvirtual void WebCore::RenderBlockFlow::addFocusRingRectsForInlineChildren(WTF::Vector<WebCore::LayoutRect>&, const WebCore::LayoutPoint&, const WebCore::RenderLayerModelObject*)�~@~Y:
/home/bot/webkit/Source/WebCore/rendering/RenderBlockFlow.cpp:3460:37: error: �~@~XfirstRootInlineBoxFor�~@~Y is not a member of �~@~XWebCore::InlineIterator�~@~Y
 3460 |     for (auto box = InlineIterator::firstRootInlineBoxFor(*this); box; box.traverseNextInlineBox()) {
      |                                     ^~~~~~~~~~~~~~~~~~~~~
/home/bot/webkit/Source/WebCore/rendering/RenderBlockFlow.cpp:3465:153: error: no matching function for call to �~@~XWebCore::LayoutRect::LayoutRect(<brace-enclosed initializer list>)�~@~Y
 3465 |         LayoutRect rect { LayoutUnit(additionalOffset.x() + box->rect().x()), additionalOffset.y() + top, LayoutUnit(box->rect().width()), bottom - top };
      |                                                                                                                                                         ^
In file included from /home/bot/webkit/Source/WebCore/rendering/RenderBoxModelObject.h:27,
                 from /home/bot/webkit/Source/WebCore/rendering/LegacyInlineBox.h:24,
                 from /home/bot/webkit/Source/WebCore/rendering/LegacyInlineFlowBox.h:23,
                 from /home/bot/webkit/Source/WebCore/rendering/LegacyRootInlineBox.h:24,
                 from /home/bot/webkit/Source/WebCore/rendering/FloatingObjects.h:26,
                 from /home/bot/webkit/Source/WebCore/rendering/RenderBlockFlow.h:25,
                 from /home/bot/webkit/Source/WebCore/rendering/RenderBlockFlow.cpp:25:
/home/bot/webkit/Source/WebCore/platform/graphics/LayoutRect.h:51:5: note: candidate: �~@~Xtemplate<class T1, class T2, class U1, class U2> WebCore::LayoutRect::LayoutRect(T1, T2, U1, U2)�~@~Y
   51 |     LayoutRect(T1 x, T2 y, U1 width, U2 height)
      |     ^~~~~~~~~~
```


```
/home/bot/webkit/Source/WebCore/rendering/RenderInline.cpp: In member function �~@~Xvoid WebCore::RenderInline::paintOutline(WebCore::PaintInfo&, const WebCore::LayoutPoint&)�~@~Y:
/home/bot/webkit/Source/WebCore/rendering/RenderInline.cpp:963:30: error: invalid use of incomplete type �~@~Xclass WebCore::InlineIterator::LineIterator�~@~Y
  963 |         auto line = box->line();
      |                     ~~~~~~~~~^~
In file included from /home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorInlineBox.h:28,
                 from /home/bot/webkit/Source/WebCore/rendering/RenderInline.cpp:31:
/home/bot/webkit/Source/WebCore/layout/integration/InlineIteratorBox.h:41:7: note: forward declaration of �~@~Xclass WebCore::InlineIterator::LineIterator�~@~Y
   41 | class LineIterator;
      |       ^~~~~~~~~~~~
In file included from /home/bot/webkit/WebKitBuild/Release-alternative/bmalloc/Headers/bmalloc/IsoHeapInlines.h:38,
                 from /home/bot/webkit/WebKitBuild/Release-alternative/WTF/Headers/wtf/IsoMallocInlines.h:40,
                 from /home/bot/webkit/Source/WebCore/rendering/RenderInline.cpp:53:
```
Comment 8 Pablo Saavedra 2022-02-02 11:43:35 PST
Created attachment 450682 [details]
patch
Comment 9 Chris Dumez 2022-02-02 11:44:50 PST
Comment on attachment 450682 [details]
patch

This looks more reasonable, thank you.
Comment 10 EWS 2022-02-02 12:42:12 PST
Committed r288985 (246706@main): <https://commits.webkit.org/246706@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 450682 [details].
Comment 11 Radar WebKit Bug Importer 2022-02-02 12:43:17 PST
<rdar://problem/88396138>