<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>202962</bug_id>
          
          <creation_ts>2019-10-14 16:34:16 -0700</creation_ts>
          <short_desc>Add some PencilKit extension points</short_desc>
          <delta_ts>2019-10-21 23:05:48 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>WebKit Local Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Daniel Bates">dbates</reporter>
          <assigned_to name="Daniel Bates">dbates</assigned_to>
          <cc>aestes</cc>
    
    <cc>jlewis3</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1579891</commentid>
    <comment_count>0</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-14 16:34:16 -0700</bug_when>
    <thetext>Add some PencilKit extension points.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1579892</commentid>
    <comment_count>1</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-14 16:34:31 -0700</bug_when>
    <thetext>This is the WebKit part corresponding to &lt;rdar://problem/56261392&gt;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1579893</commentid>
    <comment_count>2</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2019-10-14 16:34:51 -0700</bug_when>
    <thetext>&lt;rdar://problem/56269759&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1579895</commentid>
    <comment_count>3</comment_count>
      <attachid>380933</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-14 16:37:49 -0700</bug_when>
    <thetext>Created attachment 380933
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1579911</commentid>
    <comment_count>4</comment_count>
      <attachid>380933</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-14 17:09:43 -0700</bug_when>
    <thetext>Comment on attachment 380933
Patch

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

&gt; Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:153
&gt; +#if HAVE(PENCILKIT_ADDITIONS)

Actually, maybe it would be better to check for the header itself to avoid breaking builds internally at least temporarily. Then change it back after the dust settles.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1582022</commentid>
    <comment_count>5</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-21 14:06:15 -0700</bug_when>
    <thetext>(In reply to Daniel Bates from comment #4)
&gt; Comment on attachment 380933 [details]
&gt; Patch
&gt; 
&gt; View in context:
&gt; https://bugs.webkit.org/attachment.cgi?id=380933&amp;action=review
&gt; 
&gt; &gt; Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm:153
&gt; &gt; +#if HAVE(PENCILKIT_ADDITIONS)
&gt; 
&gt; Actually, maybe it would be better to check for the header itself to avoid
&gt; breaking builds internally at least temporarily. Then change it back after
&gt; the dust settles.

No need to do this. Everything will work out™ since HAVE_PENCILKIT_ADDITIONS is a new guard.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1582024</commentid>
    <comment_count>6</comment_count>
      <attachid>380933</attachid>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-21 14:07:44 -0700</bug_when>
    <thetext>Comment on attachment 380933
Patch

Clearing flags on attachment: 380933

Committed r251381: &lt;https://trac.webkit.org/changeset/251381&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1582025</commentid>
    <comment_count>7</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-21 14:07:45 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1582244</commentid>
    <comment_count>8</comment_count>
    <who name="Matt Lewis">jlewis3</who>
    <bug_when>2019-10-21 21:46:05 -0700</bug_when>
    <thetext>Reverted r251381 for reason:

This broke an internal build.

Committed r251415: &lt;https://trac.webkit.org/changeset/251415&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1582258</commentid>
    <comment_count>9</comment_count>
    <who name="Daniel Bates">dbates</who>
    <bug_when>2019-10-21 23:05:48 -0700</bug_when>
    <thetext>Committed r251419: &lt;https://trac.webkit.org/changeset/251419&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>380933</attachid>
            <date>2019-10-14 16:37:49 -0700</date>
            <delta_ts>2019-10-21 14:07:44 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-202962-20191014163748.patch</filename>
            <type>text/plain</type>
            <size>2307</size>
            <attacher name="Daniel Bates">dbates</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjUxMDg0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDlhZWJkMTU4ZjdlZmVkNjAy
NThjMGM2ZWQzM2FhY2Y1NDg5OTY5Y2UuLjRiNjJiZjFmZDcyYWIyNzdmZGFmZWVmZDRlZmM0YWVk
NjA4NDIyMzkgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTktMTAtMTQgIERhbmllbCBC
YXRlcyAgPGRhYmF0ZXNAYXBwbGUuY29tPgorCisgICAgICAgIEFkZCBzb21lIFBlbmNpbEtpdCBl
eHRlbnNpb24gcG9pbnRzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD0yMDI5NjIKKyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzU2MjY5NzU5PgorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoaXMgaXMgdGhlIFdl
YktpdCBwYXJ0IGNvcnJlc3BvbmRpbmcgdG8gPHJkYXI6Ly9wcm9ibGVtLzU2MjYxMzkyPi4KKwor
ICAgICAgICAqIFVJUHJvY2Vzcy9pb3MvV0tDb250ZW50Vmlld0ludGVyYWN0aW9uLm1tOgorICAg
ICAgICAoLVtXS0NvbnRlbnRWaWV3IHNldHVwSW50ZXJhY3Rpb25dKTogQ2FsbCBleHRlbnNpb24g
cG9pbnQuCisgICAgICAgICgtW1dLQ29udGVudFZpZXcgY2xlYW51cEludGVyYWN0aW9uXSk6IERp
dHRvLgorCiAyMDE5LTA5LTEzICBEYW5pZWwgQmF0ZXMgIDxkYWJhdGVzQGFwcGxlLmNvbT4KIAog
ICAgICAgICBTdXBwb3J0IGJ1aWxkaW5nIFdlYktpdCBmb3IgTWFjIENhdGFseXN0IGZyb20gWGNv
ZGUKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL2lvcy9XS0NvbnRlbnRWaWV3
SW50ZXJhY3Rpb24ubW0gYi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9pb3MvV0tDb250ZW50Vmll
d0ludGVyYWN0aW9uLm1tCmluZGV4IGY0ZTRlYTk5YzI2YjQzOTA4MDZjNWFkYWU5NmZiYjYyMjE3
OGYzZGMuLmQ2ZjY4ZTE3NWE0MzFiNzM5Y2M2NTQ3ZTAxYzg3MzEyYjNhODkyNTEgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL2lvcy9XS0NvbnRlbnRWaWV3SW50ZXJhY3Rpb24u
bW0KKysrIGIvU291cmNlL1dlYktpdC9VSVByb2Nlc3MvaW9zL1dLQ29udGVudFZpZXdJbnRlcmFj
dGlvbi5tbQpAQCAtMTUwLDYgKzE1MCwxMCBAQAogI2ltcG9ydCA8V2ViS2l0QWRkaXRpb25zL1dL
UGxhdGZvcm1GaWxlVXBsb2FkUGFuZWwubW0+CiAjZW5kaWYKIAorI2lmIEhBVkUoUEVOQ0lMS0lU
X0FERElUSU9OUykKKyNpbXBvcnQgPFdlYktpdEFkZGl0aW9ucy9XZWJLaXRQZW5jaWxBZGRpdGlv
bnMuaD4KKyNlbmRpZgorCiAjaWYgRU5BQkxFKFBPSU5URVJfRVZFTlRTKQogI2ltcG9ydCAiUmVt
b3RlU2Nyb2xsaW5nQ29vcmRpbmF0b3JQcm94eS5oIgogI2ltcG9ydCA8V2ViQ29yZS9Ub3VjaEFj
dGlvbi5oPgpAQCAtODAwLDYgKzgwNCwxMCBAQCAtICh2b2lkKXNldHVwSW50ZXJhY3Rpb24KICAg
ICBbc2VsZiBzZXR1cERyYWdBbmREcm9wSW50ZXJhY3Rpb25zXTsKICNlbmRpZgogCisjaWYgSEFW
RShQRU5DSUxLSVRfQURESVRJT05TKQorICAgIFtzZWxmIHNldHVwUGVuY2lsSW50ZXJhY3Rpb25d
OworI2VuZGlmCisKICAgICBfdHdvRmluZ2VyU2luZ2xlVGFwR2VzdHVyZVJlY29nbml6ZXIgPSBh
ZG9wdE5TKFtbVUlUYXBHZXN0dXJlUmVjb2duaXplciBhbGxvY10gaW5pdFdpdGhUYXJnZXQ6c2Vs
ZiBhY3Rpb246QHNlbGVjdG9yKF90d29GaW5nZXJTaW5nbGVUYXBHZXN0dXJlUmVjb2duaXplZDop
XSk7CiAgICAgW190d29GaW5nZXJTaW5nbGVUYXBHZXN0dXJlUmVjb2duaXplciBzZXRBbGxvd2Fi
bGVNb3ZlbWVudDo2MF07CiAgICAgW190d29GaW5nZXJTaW5nbGVUYXBHZXN0dXJlUmVjb2duaXpl
ciBfc2V0QWxsb3dhYmxlU2VwYXJhdGlvbjoxNTBdOwpAQCAtOTUyLDYgKzk2MCwxMCBAQCAtICh2
b2lkKWNsZWFudXBJbnRlcmFjdGlvbgogICAgIFtzZWxmIHRlYXJkb3duRHJhZ0FuZERyb3BJbnRl
cmFjdGlvbnNdOwogI2VuZGlmCiAKKyNpZiBIQVZFKFBFTkNJTEtJVF9BRERJVElPTlMpCisgICAg
W3NlbGYgY2xlYW51cFBlbmNpbEludGVyYWN0aW9uXTsKKyNlbmRpZgorCiAgICAgX2luc3BlY3Rv
ck5vZGVTZWFyY2hFbmFibGVkID0gTk87CiAgICAgaWYgKF9pbnNwZWN0b3JOb2RlU2VhcmNoR2Vz
dHVyZVJlY29nbml6ZXIpIHsKICAgICAgICAgW19pbnNwZWN0b3JOb2RlU2VhcmNoR2VzdHVyZVJl
Y29nbml6ZXIgc2V0RGVsZWdhdGU6bmlsXTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>