<?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>183073</bug_id>
          
          <creation_ts>2018-02-22 20:42:02 -0800</creation_ts>
          <short_desc>[RenderTreeBuilder] Add WARN_UNUSED_RETURN to detach()</short_desc>
          <delta_ts>2018-02-23 06:52:52 -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>Layout and Rendering</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="alan">zalan</reporter>
          <assigned_to name="alan">zalan</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>commit-queue</cc>
    
    <cc>koivisto</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>webkit-bug-importer</cc>
    
    <cc>zalan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1401641</commentid>
    <comment_count>0</comment_count>
    <who name="alan">zalan</who>
    <bug_when>2018-02-22 20:42:02 -0800</bug_when>
    <thetext>to ensure we don&apos;t accidentally destroy a renderer</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1401642</commentid>
    <comment_count>1</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-02-22 20:42:36 -0800</bug_when>
    <thetext>&lt;rdar://problem/37814585&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1401643</commentid>
    <comment_count>2</comment_count>
      <attachid>334503</attachid>
    <who name="alan">zalan</who>
    <bug_when>2018-02-22 20:43:49 -0800</bug_when>
    <thetext>Created attachment 334503
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1401711</commentid>
    <comment_count>3</comment_count>
      <attachid>334503</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-02-23 06:52:51 -0800</bug_when>
    <thetext>Comment on attachment 334503
Patch

Clearing flags on attachment: 334503

Committed r228948: &lt;https://trac.webkit.org/changeset/228948&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1401712</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2018-02-23 06:52:52 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>334503</attachid>
            <date>2018-02-22 20:43:49 -0800</date>
            <delta_ts>2018-02-23 06:52:51 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-183073-20180222204348.patch</filename>
            <type>text/plain</type>
            <size>4537</size>
            <attacher name="alan">zalan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjI4OTM4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMDg1NTQ1ZDAzMzczZTkz
YjMwOWFmYzY1NDYzNmY0NGYxOGE3ODNmNy4uYjY2YTU4NWY4Y2MwYTg0MTBjMGNmODhmZDE2YTgx
YTc0ODY4MDU1ZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE4LTAyLTIyICBaYWxh
biBCdWp0YXMgIDx6YWxhbkBhcHBsZS5jb20+CisKKyAgICAgICAgW1JlbmRlclRyZWVCdWlsZGVy
XSBBZGQgV0FSTl9VTlVTRURfUkVUVVJOIHRvIGRldGFjaCgpCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODMwNzMKKyAgICAgICAgPHJkYXI6Ly9wcm9i
bGVtLzM3ODE0NTg1PgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg
ICAgICAgIFNvIHRoYXQgd2UgZG9uJ3QgZGVzdHJveSBhIHJlbmRlcmVyIGFjY2lkZW50YWxseS4K
KworICAgICAgICAqIHJlbmRlcmluZy91cGRhdGluZy9SZW5kZXJUcmVlQnVpbGRlckJsb2NrLmg6
CisgICAgICAgICogcmVuZGVyaW5nL3VwZGF0aW5nL1JlbmRlclRyZWVCdWlsZGVyUnVieS5oOgor
ICAgICAgICAqIHJlbmRlcmluZy91cGRhdGluZy9SZW5kZXJUcmVlQnVpbGRlclNWRy5oOgorCiAy
MDE4LTAyLTIyICBaYWxhbiBCdWp0YXMgIDx6YWxhbkBhcHBsZS5jb20+CiAKICAgICAgICAgW1Jl
bmRlclRyZWVCdWlsZGVyXSBSZW5hbWUgaW5zZXJ0Q2hpbGQoKSAtPiBhdHRhY2goKSwgdGFrZUNo
aWxkKCkgLT4gZGV0YWNoKCkgYW5kIHJlbW92ZUFuZERlc3Ryb3koKSAtPiBkZXN0cm95KCkKZGlm
ZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy91cGRhdGluZy9SZW5kZXJUcmVlQnVp
bGRlckJsb2NrLmggYi9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvdXBkYXRpbmcvUmVuZGVyVHJl
ZUJ1aWxkZXJCbG9jay5oCmluZGV4IDI2MjA0MmQ5ZWQxNDY2YTM4OGE2MjIwYzNlMTA3NjAyNTBl
MDUzNTkuLjY2NmNkYzUxMjVlM2ExOTAyZjEwYzgzODJlZGYxMmZiYTMxNWIwNTYgMTAwNjQ0Ci0t
LSBhL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy91cGRhdGluZy9SZW5kZXJUcmVlQnVpbGRlckJs
b2NrLmgKKysrIGIvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL3VwZGF0aW5nL1JlbmRlclRyZWVC
dWlsZGVyQmxvY2suaApAQCAtMzYsOCArMzYsOCBAQCBwdWJsaWM6CiAgICAgdm9pZCBhdHRhY2go
UmVuZGVyQmxvY2smIHBhcmVudCwgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gY2hpbGQsIFJlbmRl
ck9iamVjdCogYmVmb3JlQ2hpbGQpOwogICAgIHZvaWQgYXR0YWNoSWdub3JpbmdDb250aW51YXRp
b24oUmVuZGVyQmxvY2smIHBhcmVudCwgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gY2hpbGQsIFJl
bmRlck9iamVjdCogYmVmb3JlQ2hpbGQpOwogCi0gICAgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4g
ZGV0YWNoKFJlbmRlckJsb2NrJiBwYXJlbnQsIFJlbmRlck9iamVjdCYgb2xkQ2hpbGQpOwotICAg
IFJlbmRlclB0cjxSZW5kZXJPYmplY3Q+IGRldGFjaChSZW5kZXJCbG9ja0Zsb3cmIHBhcmVudCwg
UmVuZGVyT2JqZWN0JiBjaGlsZCk7CisgICAgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gZGV0YWNo
KFJlbmRlckJsb2NrJiBwYXJlbnQsIFJlbmRlck9iamVjdCYgb2xkQ2hpbGQpIFdBUk5fVU5VU0VE
X1JFVFVSTjsKKyAgICBSZW5kZXJQdHI8UmVuZGVyT2JqZWN0PiBkZXRhY2goUmVuZGVyQmxvY2tG
bG93JiBwYXJlbnQsIFJlbmRlck9iamVjdCYgY2hpbGQpIFdBUk5fVU5VU0VEX1JFVFVSTjsKIAog
ICAgIHZvaWQgZHJvcEFub255bW91c0JveENoaWxkKFJlbmRlckJsb2NrJiBwYXJlbnQsIFJlbmRl
ckJsb2NrJiBjaGlsZCk7CiAgICAgdm9pZCBjaGlsZEJlY2FtZU5vbklubGluZShSZW5kZXJCbG9j
ayYgcGFyZW50LCBSZW5kZXJFbGVtZW50JiBjaGlsZCk7CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2Vi
Q29yZS9yZW5kZXJpbmcvdXBkYXRpbmcvUmVuZGVyVHJlZUJ1aWxkZXJSdWJ5LmggYi9Tb3VyY2Uv
V2ViQ29yZS9yZW5kZXJpbmcvdXBkYXRpbmcvUmVuZGVyVHJlZUJ1aWxkZXJSdWJ5LmgKaW5kZXgg
N2VjOWJmZTM1NzRjNmE0YmM1NDZlMDQ3YTcxNTg5ODIzZTVhYzU0MC4uYzE0ZjE0ZDg4MDM5NTZh
ZDJiZWIwNjA2ODY3NjE5MzI5ODE3MmMxYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcmVu
ZGVyaW5nL3VwZGF0aW5nL1JlbmRlclRyZWVCdWlsZGVyUnVieS5oCisrKyBiL1NvdXJjZS9XZWJD
b3JlL3JlbmRlcmluZy91cGRhdGluZy9SZW5kZXJUcmVlQnVpbGRlclJ1YnkuaApAQCAtNDIsOSAr
NDIsOSBAQCBwdWJsaWM6CiAgICAgUnVieShSZW5kZXJUcmVlQnVpbGRlciYpOwogCiAgICAgdm9p
ZCBhdHRhY2goUmVuZGVyUnVieVJ1biYgcGFyZW50LCBSZW5kZXJQdHI8UmVuZGVyT2JqZWN0PiBj
aGlsZCwgUmVuZGVyT2JqZWN0KiBiZWZvcmVDaGlsZCk7Ci0gICAgUmVuZGVyUHRyPFJlbmRlck9i
amVjdD4gZGV0YWNoKFJlbmRlclJ1YnlBc0lubGluZSYgcGFyZW50LCBSZW5kZXJPYmplY3QmIGNo
aWxkKTsKLSAgICBSZW5kZXJQdHI8UmVuZGVyT2JqZWN0PiBkZXRhY2goUmVuZGVyUnVieUFzQmxv
Y2smIHBhcmVudCwgUmVuZGVyT2JqZWN0JiBjaGlsZCk7Ci0gICAgUmVuZGVyUHRyPFJlbmRlck9i
amVjdD4gZGV0YWNoKFJlbmRlclJ1YnlSdW4mIHBhcmVudCwgUmVuZGVyT2JqZWN0JiBjaGlsZCk7
CisgICAgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gZGV0YWNoKFJlbmRlclJ1YnlBc0lubGluZSYg
cGFyZW50LCBSZW5kZXJPYmplY3QmIGNoaWxkKSBXQVJOX1VOVVNFRF9SRVRVUk47CisgICAgUmVu
ZGVyUHRyPFJlbmRlck9iamVjdD4gZGV0YWNoKFJlbmRlclJ1YnlBc0Jsb2NrJiBwYXJlbnQsIFJl
bmRlck9iamVjdCYgY2hpbGQpIFdBUk5fVU5VU0VEX1JFVFVSTjsKKyAgICBSZW5kZXJQdHI8UmVu
ZGVyT2JqZWN0PiBkZXRhY2goUmVuZGVyUnVieVJ1biYgcGFyZW50LCBSZW5kZXJPYmplY3QmIGNo
aWxkKSBXQVJOX1VOVVNFRF9SRVRVUk47CiAKICAgICB2b2lkIG1vdmVDaGlsZHJlbihSZW5kZXJS
dWJ5QmFzZSYgZnJvbSwgUmVuZGVyUnVieUJhc2UmIHRvKTsKIApkaWZmIC0tZ2l0IGEvU291cmNl
L1dlYkNvcmUvcmVuZGVyaW5nL3VwZGF0aW5nL1JlbmRlclRyZWVCdWlsZGVyU1ZHLmggYi9Tb3Vy
Y2UvV2ViQ29yZS9yZW5kZXJpbmcvdXBkYXRpbmcvUmVuZGVyVHJlZUJ1aWxkZXJTVkcuaAppbmRl
eCAxZGM5ZTMwZjM2ODA4ZDRkOTA0ZDQyYmRiNGU3YTJkMTE4MWEwY2M2Li5kOTMwY2FlZGU0YzVj
YTdjZGIwYTM1YzY3OGMyNmEzYjMxOWUxODUyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9y
ZW5kZXJpbmcvdXBkYXRpbmcvUmVuZGVyVHJlZUJ1aWxkZXJTVkcuaAorKysgYi9Tb3VyY2UvV2Vi
Q29yZS9yZW5kZXJpbmcvdXBkYXRpbmcvUmVuZGVyVHJlZUJ1aWxkZXJTVkcuaApAQCAtNDMsMTAg
KzQzLDEwIEBAIHB1YmxpYzoKICAgICB2b2lkIGF0dGFjaChSZW5kZXJTVkdSb290JiBwYXJlbnQs
IFJlbmRlclB0cjxSZW5kZXJPYmplY3Q+IGNoaWxkLCBSZW5kZXJPYmplY3QqIGJlZm9yZUNoaWxk
KTsKICAgICB2b2lkIGF0dGFjaChSZW5kZXJTVkdUZXh0JiBwYXJlbnQsIFJlbmRlclB0cjxSZW5k
ZXJPYmplY3Q+IGNoaWxkLCBSZW5kZXJPYmplY3QqIGJlZm9yZUNoaWxkKTsKIAotICAgIFJlbmRl
clB0cjxSZW5kZXJPYmplY3Q+IGRldGFjaChSZW5kZXJTVkdUZXh0JiBwYXJlbnQsIFJlbmRlck9i
amVjdCYgY2hpbGQpOwotICAgIFJlbmRlclB0cjxSZW5kZXJPYmplY3Q+IGRldGFjaChSZW5kZXJT
VkdJbmxpbmUmIHBhcmVudCwgUmVuZGVyT2JqZWN0JiBjaGlsZCk7Ci0gICAgUmVuZGVyUHRyPFJl
bmRlck9iamVjdD4gZGV0YWNoKFJlbmRlclNWR0NvbnRhaW5lciYgcGFyZW50LCBSZW5kZXJPYmpl
Y3QmIGNoaWxkKTsKLSAgICBSZW5kZXJQdHI8UmVuZGVyT2JqZWN0PiBkZXRhY2goUmVuZGVyU1ZH
Um9vdCYgcGFyZW50LCBSZW5kZXJPYmplY3QmIGNoaWxkKTsKKyAgICBSZW5kZXJQdHI8UmVuZGVy
T2JqZWN0PiBkZXRhY2goUmVuZGVyU1ZHVGV4dCYgcGFyZW50LCBSZW5kZXJPYmplY3QmIGNoaWxk
KSBXQVJOX1VOVVNFRF9SRVRVUk47CisgICAgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gZGV0YWNo
KFJlbmRlclNWR0lubGluZSYgcGFyZW50LCBSZW5kZXJPYmplY3QmIGNoaWxkKSBXQVJOX1VOVVNF
RF9SRVRVUk47CisgICAgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gZGV0YWNoKFJlbmRlclNWR0Nv
bnRhaW5lciYgcGFyZW50LCBSZW5kZXJPYmplY3QmIGNoaWxkKSBXQVJOX1VOVVNFRF9SRVRVUk47
CisgICAgUmVuZGVyUHRyPFJlbmRlck9iamVjdD4gZGV0YWNoKFJlbmRlclNWR1Jvb3QmIHBhcmVu
dCwgUmVuZGVyT2JqZWN0JiBjaGlsZCkgV0FSTl9VTlVTRURfUkVUVVJOOwogCiBwcml2YXRlOgog
ICAgIFJlbmRlclRyZWVCdWlsZGVyJiBtX2J1aWxkZXI7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>