<?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>165123</bug_id>
          
          <creation_ts>2016-11-28 16:16:36 -0800</creation_ts>
          <short_desc>[Mac][WK2] Enable HTML interactive form validation</short_desc>
          <delta_ts>2016-12-08 14:13:12 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>DOM</component>
          <version>WebKit Nightly 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>
          
          <blocked>164382</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>barraclough</cc>
    
    <cc>beidson</cc>
    
    <cc>cdumez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>darin</cc>
    
    <cc>jonlee</cc>
    
    <cc>m.kurz+webkitbugs</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1254087</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-11-28 16:16:36 -0800</bug_when>
    <thetext>Enable HTML interactive form validation now that we have a native UI for it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254088</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-11-28 16:16:50 -0800</bug_when>
    <thetext>&lt;rdar://problem/28944652&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254089</commentid>
    <comment_count>2</comment_count>
      <attachid>295549</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-11-28 16:18:31 -0800</bug_when>
    <thetext>Created attachment 295549
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254097</commentid>
    <comment_count>3</comment_count>
      <attachid>295549</attachid>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2016-11-28 16:42:13 -0800</bug_when>
    <thetext>Comment on attachment 295549
Patch

Making InteractiveFormValidationEnabled a non-experimental feature means that it won&apos;t get enabled by default in WTR, so maybe iOS-WK2 tests will be affected?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254098</commentid>
    <comment_count>4</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-11-28 16:43:31 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; Comment on attachment 295549 [details]
&gt; Patch
&gt; 
&gt; Making InteractiveFormValidationEnabled a non-experimental feature means
&gt; that it won&apos;t get enabled by default in WTR, so maybe iOS-WK2 tests will be
&gt; affected?

Ok, I&apos;ll wait to see if iOS EWS is happy then.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254202</commentid>
    <comment_count>5</comment_count>
      <attachid>295549</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-11-28 21:37:30 -0800</bug_when>
    <thetext>Comment on attachment 295549
Patch

Clearing flags on attachment: 295549

Committed r209060: &lt;http://trac.webkit.org/changeset/209060&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254203</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2016-11-28 21:37:35 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1254289</commentid>
    <comment_count>7</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-11-29 09:05:20 -0800</bug_when>
    <thetext>&lt;rdar://problem/29421342&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1257862</commentid>
    <comment_count>8</comment_count>
    <who name="">m.kurz+webkitbugs</who>
    <bug_when>2016-12-08 13:49:05 -0800</bug_when>
    <thetext>Shouldn&apos;t bug 28649 also get marked as resolved now? Isn&apos;t it a duplicate?

What&apos;s unclear for me is if this gets rolled out both for Safari on macOS and iOS or just one of these plattforms?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1257867</commentid>
    <comment_count>9</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-12-08 13:52:06 -0800</bug_when>
    <thetext>(In reply to comment #8)
&gt; Shouldn&apos;t bug 28649 also get marked as resolved now? Isn&apos;t it a duplicate?
&gt; 
&gt; What&apos;s unclear for me is if this gets rolled out both for Safari on macOS
&gt; and iOS or just one of these plattforms?

I duped it to https://bugs.webkit.org/show_bug.cgi?id=164382. The reason https://bugs.webkit.org/show_bug.cgi?id=164382 is not closed yet because we have not yet enabled interactive validation on iOS WebKit1.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1257880</commentid>
    <comment_count>10</comment_count>
    <who name="">m.kurz+webkitbugs</who>
    <bug_when>2016-12-08 14:08:08 -0800</bug_when>
    <thetext>Isn&apos;t iOS using WebKit2 since version 8 anyway? Who would be affected by fixing bug 164382?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1257881</commentid>
    <comment_count>11</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2016-12-08 14:09:35 -0800</bug_when>
    <thetext>(In reply to comment #10)
&gt; Isn&apos;t iOS using WebKit2 since version 8 anyway? Who would be affected by
&gt; fixing bug 164382?

Apps using UIWebView instead of WKWebView.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1257882</commentid>
    <comment_count>12</comment_count>
    <who name="">m.kurz+webkitbugs</who>
    <bug_when>2016-12-08 14:13:12 -0800</bug_when>
    <thetext>Got it, thanks!</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>295549</attachid>
            <date>2016-11-28 16:18:31 -0800</date>
            <delta_ts>2016-11-28 21:37:30 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-165123-20161128161807.patch</filename>
            <type>text/plain</type>
            <size>3107</size>
            <attacher name="Chris Dumez">cdumez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjA5MDE2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggMGI0MTQ4NTgyYjdjNDNh
MmJiNGU0NDliZmY0YTUzZjA0OTdiOWUzOS4uMTIwNzI2ZWYzZTYzNjEwY2E5YTBiMGI1NDMyMDRi
ZDVmYjRiMmZkYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDE2LTExLTI4ICBDaHJp
cyBEdW1leiAgPGNkdW1lekBhcHBsZS5jb20+CisKKyAgICAgICAgW01hY11bV0syXSBFbmFibGUg
SFRNTCBpbnRlcmFjdGl2ZSBmb3JtIHZhbGlkYXRpb24KKyAgICAgICAgaHR0cHM6Ly9idWdzLndl
YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE2NTEyMworICAgICAgICA8cmRhcjovL3Byb2JsZW0v
Mjg5NDQ2NTI+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgRW5hYmxlIEhUTUwgaW50ZXJhY3RpdmUgZm9ybSB2YWxpZGF0aW9uIG5vdyB0aGF0IHdlIGhh
dmUgYSBuYXRpdmUgVUkKKyAgICAgICAgZm9yIGl0LgorCisgICAgICAgICogU2hhcmVkL1dlYlBy
ZWZlcmVuY2VzRGVmaW5pdGlvbnMuaDoKKwogMjAxNi0xMS0yOCAgQW5kcmV3IEdvbGQgIDxhZ29s
ZEBhcHBsZS5jb20+CiAKICAgICAgICAgU3VwcG9ydCBmb3IgSFRNTCBNZWRpYSBDYXB0dXJlIEFQ
SQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvU2hhcmVkL1dlYlByZWZlcmVuY2VzRGVmaW5p
dGlvbnMuaCBiL1NvdXJjZS9XZWJLaXQyL1NoYXJlZC9XZWJQcmVmZXJlbmNlc0RlZmluaXRpb25z
LmgKaW5kZXggZTAwZDJlZWUwNWEyNzAxNDEzYWEzNGQyZmQ0YjYxYmQ2NDBkNDBjNS4uZDZkMDdl
NzEwN2FkYjQyZGZhMGNjMDRjOGRmNmI1YzJhZmM4MDQ0ZiAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YktpdDIvU2hhcmVkL1dlYlByZWZlcmVuY2VzRGVmaW5pdGlvbnMuaAorKysgYi9Tb3VyY2UvV2Vi
S2l0Mi9TaGFyZWQvV2ViUHJlZmVyZW5jZXNEZWZpbml0aW9ucy5oCkBAIC01NCw2ICs1NCwxMiBA
QAogI2RlZmluZSBERUZBVUxUX1BERlBMVUdJTl9FTkFCTEVEIGZhbHNlCiAjZW5kaWYKIAorI2lm
IFBMQVRGT1JNKE1BQykKKyNkZWZpbmUgREVGQVVMVF9IVE1MX0lOVEVSQUNUSVZFX0ZPUk1fVkFM
SURBVElPTl9FTkFCTEVEIHRydWUKKyNlbHNlCisjZGVmaW5lIERFRkFVTFRfSFRNTF9JTlRFUkFD
VElWRV9GT1JNX1ZBTElEQVRJT05fRU5BQkxFRCBmYWxzZQorI2VuZGlmCisKICNpZiBQTEFURk9S
TShJT1MpCiAjZGVmaW5lIERFRkFVTFRfQUxMT1dTX1BJQ1RVUkVfSU5fUElDVFVSRV9NRURJQV9Q
TEFZQkFDSyB0cnVlCiAjZGVmaW5lIERFRkFVTFRfQkFDS1NQQUNFX0tFWV9OQVZJR0FUSU9OX0VO
QUJMRUQgZmFsc2UKQEAgLTIzOCw2ICsyNDQsNyBAQAogICAgIG1hY3JvKEN1c3RvbUVsZW1lbnRz
RW5hYmxlZCwgY3VzdG9tRWxlbWVudHNFbmFibGVkLCBCb29sLCBib29sLCB0cnVlLCAiIiwgIiIp
IFwKICAgICBtYWNybyhFbmNyeXB0ZWRNZWRpYUFQSUVuYWJsZWQsIGVuY3J5cHRlZE1lZGlhQVBJ
RW5hYmxlZCwgQm9vbCwgYm9vbCwgZmFsc2UsICIiLCAiIikgXAogICAgIG1hY3JvKEludGVyc2Vj
dGlvbk9ic2VydmVyRW5hYmxlZCwgaW50ZXJzZWN0aW9uT2JzZXJ2ZXJFbmFibGVkLCBCb29sLCBi
b29sLCBmYWxzZSwgIkludGVyc2VjdGlvbiBPYnNlcnZlciIsICJFbmFibGUgSW50ZXJzZWN0aW9u
IE9ic2VydmVyIHN1cHBvcnQiKSBcCisgICAgbWFjcm8oSW50ZXJhY3RpdmVGb3JtVmFsaWRhdGlv
bkVuYWJsZWQsIGludGVyYWN0aXZlRm9ybVZhbGlkYXRpb25FbmFibGVkLCBCb29sLCBib29sLCBE
RUZBVUxUX0hUTUxfSU5URVJBQ1RJVkVfRk9STV9WQUxJREFUSU9OX0VOQUJMRUQsICJIVE1MIElu
dGVyYWN0aXZlIEZvcm0gVmFsaWRhdGlvbiIsICJIVE1MIGludGVyYWN0aXZlIGZvcm0gdmFsaWRh
dGlvbiIpIFwKICAgICBcCiAKICNkZWZpbmUgRk9SX0VBQ0hfV0VCS0lUX0RPVUJMRV9QUkVGRVJF
TkNFKG1hY3JvKSBcCkBAIC0zMDUsNyArMzEyLDYgQEAKICNkZWZpbmUgRk9SX0VBQ0hfV0VCS0lU
X0VYUEVSSU1FTlRBTF9GRUFUVVJFX1BSRUZFUkVOQ0UobWFjcm8pIFwKICAgICBtYWNybyhDU1NH
cmlkTGF5b3V0RW5hYmxlZCwgY3NzR3JpZExheW91dEVuYWJsZWQsIEJvb2wsIGJvb2wsIERFRkFV
TFRfRVhQRVJJTUVOVEFMX0ZFQVRVUkVTX0VOQUJMRUQsICJDU1MgR3JpZCIsICJDU1MgR3JpZCBM
YXlvdXQgTW9kdWxlIHN1cHBvcnQiKSBcCiAgICAgbWFjcm8oU3ByaW5nVGltaW5nRnVuY3Rpb25F
bmFibGVkLCBzcHJpbmdUaW1pbmdGdW5jdGlvbkVuYWJsZWQsIEJvb2wsIGJvb2wsIERFRkFVTFRf
RVhQRVJJTUVOVEFMX0ZFQVRVUkVTX0VOQUJMRUQsICJDU1MgU3ByaW5nIEFuaW1hdGlvbnMiLCAi
Q1NTIFNwcmluZyBBbmltYXRpb24gcHJvdG90eXBlIikgXAotICAgIG1hY3JvKEludGVyYWN0aXZl
Rm9ybVZhbGlkYXRpb25FbmFibGVkLCBpbnRlcmFjdGl2ZUZvcm1WYWxpZGF0aW9uRW5hYmxlZCwg
Qm9vbCwgYm9vbCwgZmFsc2UsICJIVE1MIEludGVyYWN0aXZlIEZvcm0gVmFsaWRhdGlvbiIsICJI
VE1MIGludGVyYWN0aXZlIGZvcm0gdmFsaWRhdGlvbiBwcm90b3R5cGUiKSBcCiAgICAgbWFjcm8o
R2FtZXBhZHNFbmFibGVkLCBnYW1lcGFkc0VuYWJsZWQsIEJvb2wsIGJvb2wsIERFRkFVTFRfRVhQ
RVJJTUVOVEFMX0ZFQVRVUkVTX0VOQUJMRUQsICJHYW1lcGFkcyIsICJXZWIgR2FtZXBhZCBBUEkg
c3VwcG9ydCIpIFwKICAgICBtYWNybyhNb2Rlcm5NZWRpYUNvbnRyb2xzRW5hYmxlZCwgbW9kZXJu
TWVkaWFDb250cm9sc0VuYWJsZWQsIEJvb2wsIGJvb2wsIGZhbHNlLCAiTW9kZXJuIE1lZGlhIENv
bnRyb2xzIiwgIlVzZSBtb2Rlcm4gbWVkaWEgY29udHJvbHMgbG9vayIpIFwKICAgICBtYWNybyhW
YXJpYXRpb25Gb250c0VuYWJsZWQsIHZhcmlhdGlvbkZvbnRzRW5hYmxlZCwgQm9vbCwgYm9vbCwg
REVGQVVMVF9FWFBFUklNRU5UQUxfRkVBVFVSRVNfRU5BQkxFRCwgIlZhcmlhdGlvbiBGb250cyIs
ICJFbmFibGUgdmFyaWF0aW9uIGZvbnRzIikgXAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>