Bug 226227 - [macOS] Support painting data detector highlights inside image overlays
Summary: [macOS] Support painting data detector highlights inside image overlays
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Wenson Hsieh
URL:
Keywords: InRadar
Depends on: 209788
Blocks: 226267
  Show dependency treegraph
 
Reported: 2021-05-25 10:59 PDT by Wenson Hsieh
Modified: 2021-06-04 16:34 PDT (History)
13 users (show)

See Also:


Attachments
Patch (27.45 KB, patch)
2021-05-25 11:58 PDT, Wenson Hsieh
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Fix pre-macOS 11 build (27.77 KB, patch)
2021-05-25 12:15 PDT, Wenson Hsieh
no flags Details | Formatted Diff | Diff
For EWS (26.75 KB, patch)
2021-05-26 11:59 PDT, Wenson Hsieh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Wenson Hsieh 2021-05-25 10:59:42 PDT
.
Comment 1 Wenson Hsieh 2021-05-25 11:58:21 PDT Comment hidden (obsolete)
Comment 2 Wenson Hsieh 2021-05-25 12:15:22 PDT
Created attachment 429674 [details]
Fix pre-macOS 11 build
Comment 3 EWS 2021-05-26 11:54:07 PDT
Tools/Scripts/svn-apply failed to apply attachment 429674 [details] to trunk.
Please resolve the conflicts and upload a new patch.
Comment 4 Wenson Hsieh 2021-05-26 11:59:30 PDT
Created attachment 429779 [details]
For EWS
Comment 5 EWS 2021-05-26 13:06:40 PDT
Committed r278121 (238171@main): <https://commits.webkit.org/238171@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 429779 [details].
Comment 6 Radar WebKit Bug Importer 2021-05-26 13:07:19 PDT
<rdar://problem/78528218>
Comment 7 David Kilzer (:ddkilzer) 2021-06-04 08:53:41 PDT
(In reply to EWS from comment #5)
> Committed r278121 (238171@main): <https://commits.webkit.org/238171@main>
> 
> All reviewed patches have been landed. Closing bug and clearing flags on
> attachment 429779 [details].

Follow-up build fix for coverage+release builds due to weak external symbol:

    REGRESSION (r278121): Fix build failure due to weak external symbol

Committed r278461 (278461@main): <https://commits.webkit.org/278461@main>

I believe this happened because the weak symbol (DDHighlightCreateWithRectsInVisibleRectWithStyleScaleAndDirection) was added to a second source file:

    Source/WebCore/page/mac/ImageOverlayControllerMac.mm
Comment 8 Wenson Hsieh 2021-06-04 08:59:58 PDT
(In reply to David Kilzer (:ddkilzer) from comment #7)
> (In reply to EWS from comment #5)
> > Committed r278121 (238171@main): <https://commits.webkit.org/238171@main>
> > 
> > All reviewed patches have been landed. Closing bug and clearing flags on
> > attachment 429779 [details].
> 
> Follow-up build fix for coverage+release builds due to weak external symbol:
> 
>     REGRESSION (r278121): Fix build failure due to weak external symbol
> 
> Committed r278461 (278461@main): <https://commits.webkit.org/278461@main>
> 
> I believe this happened because the weak symbol
> (DDHighlightCreateWithRectsInVisibleRectWithStyleScaleAndDirection) was
> added to a second source file:
> 
>     Source/WebCore/page/mac/ImageOverlayControllerMac.mm

Ideally, these would be declared a softlinking header in PAL. I can do this refactoring in a followup.
Comment 9 Wenson Hsieh 2021-06-04 16:34:31 PDT
(In reply to Wenson Hsieh from comment #8)
> (In reply to David Kilzer (:ddkilzer) from comment #7)
> > (In reply to EWS from comment #5)
> > > Committed r278121 (238171@main): <https://commits.webkit.org/238171@main>
> > > 
> > > All reviewed patches have been landed. Closing bug and clearing flags on
> > > attachment 429779 [details].
> > 
> > Follow-up build fix for coverage+release builds due to weak external symbol:
> > 
> >     REGRESSION (r278121): Fix build failure due to weak external symbol
> > 
> > Committed r278461 (278461@main): <https://commits.webkit.org/278461@main>
> > 
> > I believe this happened because the weak symbol
> > (DDHighlightCreateWithRectsInVisibleRectWithStyleScaleAndDirection) was
> > added to a second source file:
> > 
> >     Source/WebCore/page/mac/ImageOverlayControllerMac.mm
> 
> Ideally, these would be declared a softlinking header in PAL. I can do this
> refactoring in a followup.

Doing this in https://bugs.webkit.org/show_bug.cgi?id=226669.