<?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>188579</bug_id>
          
          <creation_ts>2018-08-14 14:17:16 -0700</creation_ts>
          <short_desc>Remove api misuse check so that custom webcontent service identifier can be set at runtime</short_desc>
          <delta_ts>2018-08-14 19:33:52 -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>New Bugs</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>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ben Richards">benton_richards</reporter>
          <assigned_to name="Ben Richards">benton_richards</assigned_to>
          <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>rniwa</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1450097</commentid>
    <comment_count>0</comment_count>
    <who name="Ben Richards">benton_richards</who>
    <bug_when>2018-08-14 14:17:16 -0700</bug_when>
    <thetext>Remove api misuse check so that custom webcontent service identifier can be set at runtime</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450098</commentid>
    <comment_count>1</comment_count>
      <attachid>347109</attachid>
    <who name="Ben Richards">benton_richards</who>
    <bug_when>2018-08-14 14:18:31 -0700</bug_when>
    <thetext>Created attachment 347109
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450152</commentid>
    <comment_count>2</comment_count>
      <attachid>347109</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-08-14 16:22:15 -0700</bug_when>
    <thetext>Comment on attachment 347109
Patch

Clearing flags on attachment: 347109

Committed r234865: &lt;https://trac.webkit.org/changeset/234865&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450153</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-08-14 16:22:17 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450154</commentid>
    <comment_count>4</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-08-14 16:24:30 -0700</bug_when>
    <thetext>&lt;rdar://problem/43308992&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450215</commentid>
    <comment_count>5</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2018-08-14 19:31:03 -0700</bug_when>
    <thetext>This opens up possibilities for terrible misuse, what is the super important reason for this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1450216</commentid>
    <comment_count>6</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2018-08-14 19:33:52 -0700</bug_when>
    <thetext>(In reply to Alexey Proskuryakov from comment #5)
&gt; This opens up possibilities for terrible misuse, what is the super important
&gt; reason for this?

It&apos;s because we&apos;d like to make this configurable at runtime. We would eventually bring back this API misuse check once we&apos;ve solidified the feature.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>347109</attachid>
            <date>2018-08-14 14:18:31 -0700</date>
            <delta_ts>2018-08-14 16:22:15 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-188579-20180814141830.patch</filename>
            <type>text/plain</type>
            <size>2685</size>
            <attacher name="Ben Richards">benton_richards</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjM0ODU5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCmluZGV4IDMzYjExZWIwMTY0MWFhNzM5
YTEyNzJlN2IxMDg2NTlkMDc1NDAzNjYuLjQ1MTJkNTYwZDJmYmYzM2IwNTY2MmZiYzQ1NzZjOTYx
ZmQwOTg2OWQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCisrKyBiL1NvdXJj
ZS9XZWJLaXQvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTgtMDgtMTQgIEJlbiBSaWNo
YXJkcyAgPGJlbnRvbl9yaWNoYXJkc0BhcHBsZS5jb20+CisKKyAgICAgICAgUmVtb3ZlIGFwaSBt
aXN1c2UgY2hlY2sgc28gdGhhdCBjdXN0b20gd2ViY29udGVudCBzZXJ2aWNlIGlkZW50aWZpZXIg
Y2FuIGJlIHNldCBhdCBydW50aW1lCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD0xODg1NzkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMh
KS4KKworICAgICAgICBDaGFuZ2VkIEFQSSBtaXN1c2UgY2hlY2sgc28gdGhhdCBhIGN1c3RvbSBi
dW5kbGUgaWRlbnRpZmllciBjYW4gYmUgc2V0IGF0IHJ1bnRpbWUgd2l0aCBhIGRlYnVnIGZsYWcK
KworICAgICAgICAqIFVJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHA6CisgICAgICAgIChXZWJL
aXQ6OldlYlByb2Nlc3NQb29sOjpzZXRDdXN0b21XZWJDb250ZW50U2VydmljZUJ1bmRsZUlkZW50
aWZpZXIpOgorICAgICAgICAqIFVJUHJvY2Vzcy9XZWJQcm9jZXNzUHJveHkuY3BwOgorICAgICAg
ICAoV2ViS2l0OjpXZWJQcm9jZXNzUHJveHk6OmdldExhdW5jaE9wdGlvbnMpOgorCiAyMDE4LTA4
LTE0ICBBbnR0aSBLb2l2aXN0byAgPGFudHRpQGFwcGxlLmNvbT4KIAogICAgICAgICBSZW1vdGVM
YXllclRyZWVUcmFuc2FjdGlvbiBzaG91bGQgdXNlIE9wdGlvblNldCBmb3IgY2hhbmdlIGZsYWdz
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAg
Yi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vzcy9XZWJQcm9jZXNzUG9vbC5jcHAKaW5kZXggYzcyMWEz
MGE3NmVjNTU0YzMwNWI2NDg0NDgyYzAxNDZmZDM5OWIzYi4uN2E5M2JmN2E3MmEyYmI2ODJhODQz
NWEwNDJjNmQ4MDU4ZDU2YzllOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdC9VSVByb2Nlc3Mv
V2ViUHJvY2Vzc1Bvb2wuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nl
c3NQb29sLmNwcApAQCAtNDIxLDcgKzQyMSw3IEBAIHZvaWQgV2ViUHJvY2Vzc1Bvb2w6OnNldE1h
eGltdW1OdW1iZXJPZlByZXdhcm1lZFByb2Nlc3Nlcyh1bnNpZ25lZCBtYXhpbXVtTnVtYmVyCiB2
b2lkIFdlYlByb2Nlc3NQb29sOjpzZXRDdXN0b21XZWJDb250ZW50U2VydmljZUJ1bmRsZUlkZW50
aWZpZXIoY29uc3QgU3RyaW5nJiBjdXN0b21XZWJDb250ZW50U2VydmljZUJ1bmRsZUlkZW50aWZp
ZXIpCiB7CiAgICAgLy8gR3VhcmQgYWdhaW5zdCBBUEkgbWlzdXNlLgotICAgIGlmIChtX3Byb2Nl
c3Nlcy5zaXplKCkgfHwgIWN1c3RvbVdlYkNvbnRlbnRTZXJ2aWNlQnVuZGxlSWRlbnRpZmllci5p
c0FsbEFTQ0lJKCkpCisgICAgaWYgKCFjdXN0b21XZWJDb250ZW50U2VydmljZUJ1bmRsZUlkZW50
aWZpZXIuaXNBbGxBU0NJSSgpKQogICAgICAgICBDUkFTSCgpOwogCiAgICAgbV9jb25maWd1cmF0
aW9uLT5zZXRDdXN0b21XZWJDb250ZW50U2VydmljZUJ1bmRsZUlkZW50aWZpZXIoY3VzdG9tV2Vi
Q29udGVudFNlcnZpY2VCdW5kbGVJZGVudGlmaWVyKTsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJL
aXQvVUlQcm9jZXNzL1dlYlByb2Nlc3NQcm94eS5jcHAgYi9Tb3VyY2UvV2ViS2l0L1VJUHJvY2Vz
cy9XZWJQcm9jZXNzUHJveHkuY3BwCmluZGV4IDVlMGJjNjBiNTA1ZjZiMWM3MGRiY2Q0ZDMyZmI4
ZjlhYTgxN2Y2MTcuLjFhYmVlNTZmODZjYTVlMDU4NDU5ZTNmN2EwM2Y1MWM3N2NjOTViYzggMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvVUlQcm9jZXNzL1dlYlByb2Nlc3NQcm94eS5jcHAKKysr
IGIvU291cmNlL1dlYktpdC9VSVByb2Nlc3MvV2ViUHJvY2Vzc1Byb3h5LmNwcApAQCAtMTYyLDcg
KzE2Miw3IEBAIHZvaWQgV2ViUHJvY2Vzc1Byb3h5OjpnZXRMYXVuY2hPcHRpb25zKFByb2Nlc3NM
YXVuY2hlcjo6TGF1bmNoT3B0aW9ucyYgbGF1bmNoT3B0CiAKICAgICBDaGlsZFByb2Nlc3NQcm94
eTo6Z2V0TGF1bmNoT3B0aW9ucyhsYXVuY2hPcHRpb25zKTsKIAotICAgIGlmICghbV9wcm9jZXNz
UG9vbC0+Y3VzdG9tV2ViQ29udGVudFNlcnZpY2VCdW5kbGVJZGVudGlmaWVyKCkuaXNOdWxsKCkp
CisgICAgaWYgKCFtX3Byb2Nlc3NQb29sLT5jdXN0b21XZWJDb250ZW50U2VydmljZUJ1bmRsZUlk
ZW50aWZpZXIoKS5pc0VtcHR5KCkpCiAgICAgICAgIGxhdW5jaE9wdGlvbnMuY3VzdG9tV2ViQ29u
dGVudFNlcnZpY2VCdW5kbGVJZGVudGlmaWVyID0gbV9wcm9jZXNzUG9vbC0+Y3VzdG9tV2ViQ29u
dGVudFNlcnZpY2VCdW5kbGVJZGVudGlmaWVyKCkuYXNjaWkoKTsKICAgICBpZiAoV2ViS2l0Ojpp
c0luc3BlY3RvclByb2Nlc3NQb29sKHByb2Nlc3NQb29sKCkpKQogICAgICAgICBsYXVuY2hPcHRp
b25zLmV4dHJhSW5pdGlhbGl6YXRpb25EYXRhLmFkZCgiaW5zcGVjdG9yLXByb2Nlc3MiX3MsICIx
Il9zKTsK
</data>

          </attachment>
      

    </bug>

</bugzilla>