Bug 192762 – Reproducible ASSERTion failure when toggling layer borders with find-in-page up

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[zalan]

<rdar://problem/46676873>

[zalan]

Created attachment 357452 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 357452 [details]
Patch

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

> LayoutTests/editing/document-marker-null-check.html:6
> +div:before {
> +	content: "foobar";

Isn't this exactly what I said!

[zalan]

(In reply to Simon Fraser (smfr) from comment #2)
> Comment on attachment 357452 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=357452&action=review
> 
> > LayoutTests/editing/document-marker-null-check.html:6
> > +div:before {
> > +	content: "foobar";
> 
> Isn't this exactly what I said!
I don't remember you saying "foobar".

[WebKit Commit Bot]

Comment on attachment 357452 [details]
Patch

Clearing flags on attachment: 357452

Committed r239279: <https://trac.webkit.org/changeset/239279>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Ryan Haddad]

As EWS predicted, this has broken the iOS build:

In file included from /Volumes/Data/EWS/WebKit/WebKitBuild/Release-iphoneos/DerivedSources/WebCore/unified-sources/UnifiedSource11-mm.mm:7:
./page/ios/FrameIOS.mm:628:62: error: non-const lvalue reference to type 'WebCore::Node' cannot bind to a value of unrelated type 'WebCore::Node *'
        for (auto* marker : document()->markers().markersFor(node, DocumentMarker::DictationPhraseWithAlternatives)) {
                                                             ^~~~
In file included from /Volumes/Data/EWS/WebKit/WebKitBuild/Release-iphoneos/DerivedSources/WebCore/unified-sources/UnifiedSource11-mm.mm:7:
In file included from ./page/ios/FrameIOS.mm:34:
/Volumes/Data/EWS/WebKit/Source/WebCore/dom/DocumentMarkerController.h:85:68: note: passing argument to parameter here
    WEBCORE_EXPORT Vector<RenderedDocumentMarker*> markersFor(Node&, OptionSet<DocumentMarker::MarkerType> = DocumentMarker::allMarkers());
                                                                   ^
1 error generated.

[zalan]

(In reply to Ryan Haddad from comment #6)
> As EWS predicted, this has broken the iOS build:
> 
> In file included from
> /Volumes/Data/EWS/WebKit/WebKitBuild/Release-iphoneos/DerivedSources/WebCore/
> unified-sources/UnifiedSource11-mm.mm:7:
> ./page/ios/FrameIOS.mm:628:62: error: non-const lvalue reference to type
> 'WebCore::Node' cannot bind to a value of unrelated type 'WebCore::Node *'
>         for (auto* marker : document()->markers().markersFor(node,
> DocumentMarker::DictationPhraseWithAlternatives)) {
>                                                              ^~~~
> In file included from
> /Volumes/Data/EWS/WebKit/WebKitBuild/Release-iphoneos/DerivedSources/WebCore/
> unified-sources/UnifiedSource11-mm.mm:7:
> In file included from ./page/ios/FrameIOS.mm:34:
> /Volumes/Data/EWS/WebKit/Source/WebCore/dom/DocumentMarkerController.h:85:68:
> note: passing argument to parameter here
>     WEBCORE_EXPORT Vector<RenderedDocumentMarker*> markersFor(Node&,
> OptionSet<DocumentMarker::MarkerType> = DocumentMarker::allMarkers());
>                                                                    ^
> 1 error generated.

https://trac.webkit.org/changeset/239280