<?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>61907</bug_id>
          
          <creation_ts>2011-06-01 23:13:04 -0700</creation_ts>
          <short_desc>Remove superfluous renderer() test</short_desc>
          <delta_ts>2011-12-08 17:40:37 -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>CSS</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P4</priority>
          <bug_severity>Trivial</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Roland Steiner">rolandsteiner</reporter>
          <assigned_to name="Roland Steiner">rolandsteiner</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>413930</commentid>
    <comment_count>0</comment_count>
    <who name="Roland Steiner">rolandsteiner</who>
    <bug_when>2011-06-01 23:13:04 -0700</bug_when>
    <thetext>Document::recalcStyle() at one point verifies that a renderer is present or bails out (&apos;goto bail_out&apos;).

The code after that still contains a redundant check &quot;if (renderer() &amp;&amp; ...&quot; that can be removed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>413932</commentid>
    <comment_count>1</comment_count>
      <attachid>95727</attachid>
    <who name="Roland Steiner">rolandsteiner</who>
    <bug_when>2011-06-01 23:16:43 -0700</bug_when>
    <thetext>Created attachment 95727
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>413937</commentid>
    <comment_count>2</comment_count>
      <attachid>95727</attachid>
    <who name="Kent Tamura">tkent</who>
    <bug_when>2011-06-01 23:23:40 -0700</bug_when>
    <thetext>Comment on attachment 95727
Patch

ok</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>413940</commentid>
    <comment_count>3</comment_count>
    <who name="Roland Steiner">rolandsteiner</who>
    <bug_when>2011-06-01 23:30:41 -0700</bug_when>
    <thetext>Committed r87879: &lt;http://trac.webkit.org/changeset/87879&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>95727</attachid>
            <date>2011-06-01 23:16:43 -0700</date>
            <delta_ts>2011-12-08 17:40:37 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-61907-20110602151641.patch</filename>
            <type>text/plain</type>
            <size>1295</size>
            <attacher name="Roland Steiner">rolandsteiner</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODc4NzcKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA1MWJhYTI4OTg2ZjI0ZDZl
MGNiMGEzOTgwZWY0OWYxMjljMzYyZGE2Li45NmJiN2RkMzkyZTljNDlkOWZlM2QzMTFiNmI2ODBm
MDYyMDY4NWQyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291
cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTEtMDYtMDEgIFJvbGFu
ZCBTdGVpbmVyICA8cm9sYW5kc3RlaW5lckBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVtb3ZlIHN1cGVyZmx1b3VzIHJlbmRl
cmVyKCkgdGVzdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NjE5MDcKKworICAgICAgICBObyBuZXcgdGVzdHMuICh0cml2aWFsKQorCisgICAgICAgICog
ZG9tL0RvY3VtZW50LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkRvY3VtZW50OjpyZWNhbGNTdHls
ZSk6CisKIDIwMTEtMDYtMDEgIEphZWh1biBMaW0gIDxsamFlaHVuLmxpbUBzYW1zdW5nLmNvbT4K
IAogICAgICAgICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KZGlmZiAtLWdpdCBhL1NvdXJjZS9X
ZWJDb3JlL2RvbS9Eb2N1bWVudC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9kb20vRG9jdW1lbnQuY3Bw
CmluZGV4IGJhMzE3MjIxMTYzMTBmYTAxNDI4YmY0MGFkZmY3ODIzODU2YTRlOTkuLjFjN2Q2ZGNj
MGNjNjNmYjJjYzZjMTcyMmJiM2M3NWE0YzkyZmY3ODcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJD
b3JlL2RvbS9Eb2N1bWVudC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvZG9tL0RvY3VtZW50LmNw
cApAQCAtMTUyNCw3ICsxNTI0LDcgQEAgdm9pZCBEb2N1bWVudDo6cmVjYWxjU3R5bGUoU3R5bGVD
aGFuZ2UgY2hhbmdlKQogICAgICAgICAKICAgICAgICAgUmVmUHRyPFJlbmRlclN0eWxlPiBkb2N1
bWVudFN0eWxlID0gQ1NTU3R5bGVTZWxlY3Rvcjo6c3R5bGVGb3JEb2N1bWVudCh0aGlzKTsKICAg
ICAgICAgU3R5bGVDaGFuZ2UgY2ggPSBkaWZmKGRvY3VtZW50U3R5bGUuZ2V0KCksIHJlbmRlcmVy
KCktPnN0eWxlKCkpOwotICAgICAgICBpZiAocmVuZGVyZXIoKSAmJiBjaCAhPSBOb0NoYW5nZSkK
KyAgICAgICAgaWYgKGNoICE9IE5vQ2hhbmdlKQogICAgICAgICAgICAgcmVuZGVyZXIoKS0+c2V0
U3R5bGUoZG9jdW1lbnRTdHlsZS5yZWxlYXNlKCkpOwogICAgIH0KIAo=
</data>
<flag name="review"
          id="89285"
          type_id="1"
          status="+"
          setter="tkent"
    />
          </attachment>
      

    </bug>

</bugzilla>