<?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>118615</bug_id>
          
          <creation_ts>2013-07-12 14:36:02 -0700</creation_ts>
          <short_desc>Minor cleaning of AtomicString::addSlowCase</short_desc>
          <delta_ts>2013-07-15 01:08:45 -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>528+ (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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Benjamin Poulain">benjamin</reporter>
          <assigned_to name="Benjamin Poulain">benjamin</assigned_to>
          <cc>cmarcelo</cc>
    
    <cc>commit-queue</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>908070</commentid>
    <comment_count>0</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2013-07-12 14:36:02 -0700</bug_when>
    <thetext>Minor cleaning of AtomicString::addSlowCase</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>908071</commentid>
    <comment_count>1</comment_count>
      <attachid>206573</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2013-07-12 14:36:53 -0700</bug_when>
    <thetext>Created attachment 206573
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>908074</commentid>
    <comment_count>2</comment_count>
      <attachid>206573</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2013-07-12 14:46:27 -0700</bug_when>
    <thetext>Comment on attachment 206573
Patch

return statement is wrong.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>908076</commentid>
    <comment_count>3</comment_count>
      <attachid>206574</attachid>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2013-07-12 14:51:17 -0700</bug_when>
    <thetext>Created attachment 206574
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>908078</commentid>
    <comment_count>4</comment_count>
      <attachid>206574</attachid>
    <who name="Geoffrey Garen">ggaren</who>
    <bug_when>2013-07-12 14:55:49 -0700</bug_when>
    <thetext>Comment on attachment 206574
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>908326</commentid>
    <comment_count>5</comment_count>
    <who name="Benjamin Poulain">benjamin</who>
    <bug_when>2013-07-15 01:08:45 -0700</bug_when>
    <thetext>Committed r152625: &lt;http://trac.webkit.org/changeset/152625&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>206573</attachid>
            <date>2013-07-12 14:36:53 -0700</date>
            <delta_ts>2013-07-12 14:51:15 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-118615-20130712143436.patch</filename>
            <type>text/plain</type>
            <size>2015</size>
            <attacher name="Benjamin Poulain">benjamin</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTUyNjAzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV1RGL0NoYW5n
ZUxvZyBiL1NvdXJjZS9XVEYvQ2hhbmdlTG9nCmluZGV4IDc4MTlkNGExZjExODI2ODM5OWJiMmY1
Mjk0NTlkNzk0ZjRkMGVlYzEuLjcyZTdiZDE3NTA3MzY1OGI1ZDc5MGJiM2MzMTQzNWVlNTAzOTQ4
MmMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XVEYvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XVEYvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTMtMDctMTIgIEJlbmphbWluIFBvdWxhaW4gIDxi
ZW5qYW1pbkB3ZWJraXQub3JnPgorCisgICAgICAgIE1pbm9yIGNsZWFuaW5nIG9mIEF0b21pY1N0
cmluZzo6YWRkU2xvd0Nhc2UKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTExODYxNQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIEdpdmUgdGhlIHZhcmlhYmxlIGEgcHJvcGVyIG5hbWUuCisgICAgICAgIEFkZCBh
c3NlcnRpb25zIHRvIGNsYXJpZnkgdGhlIGluIGFuZCBvdXQgc3RhdGVzLgorCisgICAgICAgICog
d3RmL3RleHQvQXRvbWljU3RyaW5nLmNwcDoKKyAgICAgICAgKFdURjo6QXRvbWljU3RyaW5nOjph
ZGRTbG93Q2FzZSk6CisKIDIwMTMtMDctMTIgIE1pa2hhaWwgUG96ZG55YWtvdiAgPG1pa2hhaWwu
cG96ZG55YWtvdkBpbnRlbC5jb20+CiAKICAgICAgICAgSW1wcm92ZSBTdHJpbmdJbXBsOjpjb25z
dHJ1Y3RJbnRlcm5hbCgpIG1ldGhvZApkaWZmIC0tZ2l0IGEvU291cmNlL1dURi93dGYvdGV4dC9B
dG9taWNTdHJpbmcuY3BwIGIvU291cmNlL1dURi93dGYvdGV4dC9BdG9taWNTdHJpbmcuY3BwCmlu
ZGV4IGU5NTIyMGIxMTllOTU5OGI2NGQ3M2VjNDUyY2U5YzE0NDkxMWRjMDIuLjFhOWI4ZjViY2Ew
MzJhMjE5ZWNhNzhhYTlkYjlhNTlhMjQzNjk4OGUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XVEYvd3Rm
L3RleHQvQXRvbWljU3RyaW5nLmNwcAorKysgYi9Tb3VyY2UvV1RGL3d0Zi90ZXh0L0F0b21pY1N0
cmluZy5jcHAKQEAgLTM3NywxNiArMzc3LDIyIEBAIFBhc3NSZWZQdHI8U3RyaW5nSW1wbD4gQXRv
bWljU3RyaW5nOjphZGRGcm9tTGl0ZXJhbERhdGEoY29uc3QgY2hhciogY2hhcmFjdGVycywKICAg
ICByZXR1cm4gYWRkVG9TdHJpbmdUYWJsZTxDaGFyQnVmZmVyLCBDaGFyQnVmZmVyRnJvbUxpdGVy
YWxEYXRhVHJhbnNsYXRvcj4oYnVmZmVyKTsKIH0KIAotUGFzc1JlZlB0cjxTdHJpbmdJbXBsPiBB
dG9taWNTdHJpbmc6OmFkZFNsb3dDYXNlKFN0cmluZ0ltcGwqIHIpCitQYXNzUmVmUHRyPFN0cmlu
Z0ltcGw+IEF0b21pY1N0cmluZzo6YWRkU2xvd0Nhc2UoU3RyaW5nSW1wbCogc3RyaW5nKQogewot
ICAgIGlmICghci0+bGVuZ3RoKCkpCisgICAgaWYgKCFzdHJpbmctPmxlbmd0aCgpKQogICAgICAg
ICByZXR1cm4gU3RyaW5nSW1wbDo6ZW1wdHkoKTsKIAorICAgIEFTU0VSVF9XSVRIX01FU1NBR0Uo
IXN0cmluZy0+aXNBdG9taWMoKSwgIkF0b21pY1N0cmluZyBzaG91bGQgbm90IGhpdCB0aGUgc2xv
dyBjYXNlIGlmIHRoZSBzdHJpbmcgaXMgYWxyZWFkeSBhdG9taWMuIik7CisKICAgICBBdG9taWNT
dHJpbmdUYWJsZUxvY2tlciBsb2NrZXI7Ci0gICAgU3RyaW5nSW1wbCogcmVzdWx0ID0gKnN0cmlu
Z1RhYmxlKCkuYWRkKHIpLml0ZXJhdG9yOwotICAgIGlmIChyZXN1bHQgPT0gcikKLSAgICAgICAg
ci0+c2V0SXNBdG9taWModHJ1ZSk7Ci0gICAgcmV0dXJuIHJlc3VsdDsKKyAgICBIYXNoU2V0PFN0
cmluZ0ltcGwqPjo6QWRkUmVzdWx0IGFkZFJlc3VsdCA9IHN0cmluZ1RhYmxlKCkuYWRkKHN0cmlu
Zyk7CisKKyAgICBpZiAoYWRkUmVzdWx0LmlzTmV3RW50cnkpIHsKKyAgICAgICAgQVNTRVJUKCph
ZGRSZXN1bHQuaXRlcmF0b3IgPT0gc3RyaW5nKTsKKyAgICAgICAgc3RyaW5nLT5zZXRJc0F0b21p
Yyh0cnVlKTsKKyAgICB9CisKKyAgICByZXR1cm4gc3RyaW5nOwogfQogCiB0ZW1wbGF0ZTx0eXBl
bmFtZSBDaGFyYWN0ZXJUeXBlPgo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>206574</attachid>
            <date>2013-07-12 14:51:17 -0700</date>
            <delta_ts>2013-07-12 14:55:49 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-118615-20130712144900.patch</filename>
            <type>text/plain</type>
            <size>2028</size>
            <attacher name="Benjamin Poulain">benjamin</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTUyNjAzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV1RGL0NoYW5n
ZUxvZyBiL1NvdXJjZS9XVEYvQ2hhbmdlTG9nCmluZGV4IDc4MTlkNGExZjExODI2ODM5OWJiMmY1
Mjk0NTlkNzk0ZjRkMGVlYzEuLjcyZTdiZDE3NTA3MzY1OGI1ZDc5MGJiM2MzMTQzNWVlNTAzOTQ4
MmMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XVEYvQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XVEYvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTMtMDctMTIgIEJlbmphbWluIFBvdWxhaW4gIDxi
ZW5qYW1pbkB3ZWJraXQub3JnPgorCisgICAgICAgIE1pbm9yIGNsZWFuaW5nIG9mIEF0b21pY1N0
cmluZzo6YWRkU2xvd0Nhc2UKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTExODYxNQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgIEdpdmUgdGhlIHZhcmlhYmxlIGEgcHJvcGVyIG5hbWUuCisgICAgICAgIEFkZCBh
c3NlcnRpb25zIHRvIGNsYXJpZnkgdGhlIGluIGFuZCBvdXQgc3RhdGVzLgorCisgICAgICAgICog
d3RmL3RleHQvQXRvbWljU3RyaW5nLmNwcDoKKyAgICAgICAgKFdURjo6QXRvbWljU3RyaW5nOjph
ZGRTbG93Q2FzZSk6CisKIDIwMTMtMDctMTIgIE1pa2hhaWwgUG96ZG55YWtvdiAgPG1pa2hhaWwu
cG96ZG55YWtvdkBpbnRlbC5jb20+CiAKICAgICAgICAgSW1wcm92ZSBTdHJpbmdJbXBsOjpjb25z
dHJ1Y3RJbnRlcm5hbCgpIG1ldGhvZApkaWZmIC0tZ2l0IGEvU291cmNlL1dURi93dGYvdGV4dC9B
dG9taWNTdHJpbmcuY3BwIGIvU291cmNlL1dURi93dGYvdGV4dC9BdG9taWNTdHJpbmcuY3BwCmlu
ZGV4IGU5NTIyMGIxMTllOTU5OGI2NGQ3M2VjNDUyY2U5YzE0NDkxMWRjMDIuLjc5NWI0YjA0OWIz
Nzc2YWJlODY4N2U1ZTc0NmU4NGRjNzNkODZmNTMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XVEYvd3Rm
L3RleHQvQXRvbWljU3RyaW5nLmNwcAorKysgYi9Tb3VyY2UvV1RGL3d0Zi90ZXh0L0F0b21pY1N0
cmluZy5jcHAKQEAgLTM3NywxNiArMzc3LDIyIEBAIFBhc3NSZWZQdHI8U3RyaW5nSW1wbD4gQXRv
bWljU3RyaW5nOjphZGRGcm9tTGl0ZXJhbERhdGEoY29uc3QgY2hhciogY2hhcmFjdGVycywKICAg
ICByZXR1cm4gYWRkVG9TdHJpbmdUYWJsZTxDaGFyQnVmZmVyLCBDaGFyQnVmZmVyRnJvbUxpdGVy
YWxEYXRhVHJhbnNsYXRvcj4oYnVmZmVyKTsKIH0KIAotUGFzc1JlZlB0cjxTdHJpbmdJbXBsPiBB
dG9taWNTdHJpbmc6OmFkZFNsb3dDYXNlKFN0cmluZ0ltcGwqIHIpCitQYXNzUmVmUHRyPFN0cmlu
Z0ltcGw+IEF0b21pY1N0cmluZzo6YWRkU2xvd0Nhc2UoU3RyaW5nSW1wbCogc3RyaW5nKQogewot
ICAgIGlmICghci0+bGVuZ3RoKCkpCisgICAgaWYgKCFzdHJpbmctPmxlbmd0aCgpKQogICAgICAg
ICByZXR1cm4gU3RyaW5nSW1wbDo6ZW1wdHkoKTsKIAorICAgIEFTU0VSVF9XSVRIX01FU1NBR0Uo
IXN0cmluZy0+aXNBdG9taWMoKSwgIkF0b21pY1N0cmluZyBzaG91bGQgbm90IGhpdCB0aGUgc2xv
dyBjYXNlIGlmIHRoZSBzdHJpbmcgaXMgYWxyZWFkeSBhdG9taWMuIik7CisKICAgICBBdG9taWNT
dHJpbmdUYWJsZUxvY2tlciBsb2NrZXI7Ci0gICAgU3RyaW5nSW1wbCogcmVzdWx0ID0gKnN0cmlu
Z1RhYmxlKCkuYWRkKHIpLml0ZXJhdG9yOwotICAgIGlmIChyZXN1bHQgPT0gcikKLSAgICAgICAg
ci0+c2V0SXNBdG9taWModHJ1ZSk7Ci0gICAgcmV0dXJuIHJlc3VsdDsKKyAgICBIYXNoU2V0PFN0
cmluZ0ltcGwqPjo6QWRkUmVzdWx0IGFkZFJlc3VsdCA9IHN0cmluZ1RhYmxlKCkuYWRkKHN0cmlu
Zyk7CisKKyAgICBpZiAoYWRkUmVzdWx0LmlzTmV3RW50cnkpIHsKKyAgICAgICAgQVNTRVJUKCph
ZGRSZXN1bHQuaXRlcmF0b3IgPT0gc3RyaW5nKTsKKyAgICAgICAgc3RyaW5nLT5zZXRJc0F0b21p
Yyh0cnVlKTsKKyAgICB9CisKKyAgICByZXR1cm4gKmFkZFJlc3VsdC5pdGVyYXRvcjsKIH0KIAog
dGVtcGxhdGU8dHlwZW5hbWUgQ2hhcmFjdGVyVHlwZT4K
</data>
<flag name="review"
          id="228262"
          type_id="1"
          status="+"
          setter="ggaren"
    />
          </attachment>
      

    </bug>

</bugzilla>