<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [Catalina] Enable WebKit build"
   href="https://bugs.webkit.org/show_bug.cgi?id=199209#c19">Comment # 19</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [Catalina] Enable WebKit build"
   href="https://bugs.webkit.org/show_bug.cgi?id=199209">bug 199209</a>
              from <span class="vcard"><a class="email" href="mailto:timothy@apple.com" title="Timothy Hatcher <timothy@apple.com>"> <span class="fn">Timothy Hatcher</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=372970&action=diff" name="attach_372970" title="patch-6">attachment 372970</a> <a href="attachment.cgi?id=372970&action=edit" title="patch-6">[details]</a></span>
patch-6

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=372970&action=review">https://bugs.webkit.org/attachment.cgi?id=372970&action=review</a>

<span class="quote">>>> Source/WebKitLegacy/mac/WebView/WebHTMLView.mm:215
>>> +#endif
>> 
>> Surprised a protected field is SPI. And surprised that defining it in a category does the right thing. Are you sure this technique is correct? Gets a binary that works, not just compiles?
>> 
>> Also, if we have something this tricky, I think it eventually needs to move into an SPI header, not just in a .mm file, and especially not in two different .mm files.

> As far as I can tell, the binaries work, but I only ran API tests.

> I would like to know if anyone else knows more about these protected NSView and NSWindow bits, because what's even more surprising is that these were available in the Mojave headers (if you look, usage of _subviews and _borderView isn't protected by Catalina macros). This seems to get everything working, but I agree that it doesn't seem quite right.</span >

The _subviews ivar is still there, just not published in the public header anymore. It is published in the internal SDK version of the header. However, putting this in a category likely builds but fails to do what we expect.

I forgot we asked AppKit to add a method to let us set this ivar (and they added it in Catalina). We should use that instead. I'll post a patch for that on <a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - Move WebKitLegacy off of a couple AppKit ivars"
   href="show_bug.cgi?id=199279">bug 199279</a>.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>