<?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>9463</bug_id>
          
          <creation_ts>2006-06-15 23:35:21 -0700</creation_ts>
          <short_desc>REGRESSION (r14879): Assertion failure in CSSParser::sinkFloatingSelector() (m_floatingSelectors.contains(selector)) in some CSS tests</short_desc>
          <delta_ts>2006-06-16 18:28:02 -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>CSS</component>
          <version>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Regression</keywords>
          <priority>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter>mitz</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ddkilzer</cc>
    
    <cc>nickshanks</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>45889</commentid>
    <comment_count>0</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-06-15 23:35:21 -0700</bug_when>
    <thetext>The debug build of TOT fails on css1/conformance/forward_compatible_parsing.html and fast/css/css3-modsel-22.html with this error:

ASSERTION FAILED: m_floatingSelectors.contains(selector) (WebCore/css/cssparser.cpp:2864 WebCore::CSSSelector* WebCore::CSSParser::sinkFloatingSelector(WebCore::CSSSelector*))</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45896</commentid>
    <comment_count>1</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-16 03:30:00 -0700</bug_when>
    <thetext>Probably related to the fix for Bug 3233 (r14879).
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45897</commentid>
    <comment_count>2</comment_count>
    <who name="">mitz</who>
    <bug_when>2006-06-16 04:09:19 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; Probably related to the fix for Bug 3233 (r14879).
&gt; 

Yes, this is a regression from that fix (says so in the bug summary ;-) ).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45902</commentid>
    <comment_count>3</comment_count>
      <attachid>8866</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-16 04:29:57 -0700</bug_when>
    <thetext>Created attachment 8866
Patch v1

This patch fixes the two regressions in this bug.  The full test suite is still running on my PB G4, though.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45903</commentid>
    <comment_count>4</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-16 04:31:45 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (In reply to comment #1)
&gt; &gt; Probably related to the fix for Bug 3233 (r14879).
&gt; Yes, this is a regression from that fix (says so in the bug summary ;-) ).

It was 5:30 AM local time when I wrote that.  I was not fully mentally engaged yet.  :)
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45905</commentid>
    <comment_count>5</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-16 05:39:32 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; Created an attachment (id=8866) [edit]
&gt; Patch v1
&gt; 
&gt; This patch fixes the two regressions in this bug.  The full test suite is still
&gt; running on my PB G4, though.

All of the tests passed.  This looks like the fix.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45929</commentid>
    <comment_count>6</comment_count>
    <who name="Nicholas Shanks">nickshanks</who>
    <bug_when>2006-06-16 09:22:04 -0700</bug_when>
    <thetext>Thanks. I had no clue what the cause was, nor how to go about fixing it. :-/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>45962</commentid>
    <comment_count>7</comment_count>
      <attachid>8866</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2006-06-16 13:23:04 -0700</bug_when>
    <thetext>Comment on attachment 8866
Patch v1

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>46006</commentid>
    <comment_count>8</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-16 18:10:48 -0700</bug_when>
    <thetext>Committed revision 14895.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>46008</commentid>
    <comment_count>9</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2006-06-16 18:28:02 -0700</bug_when>
    <thetext>(In reply to comment #6)
&gt; Thanks. I had no clue what the cause was, nor how to go about fixing it. :-/

This fix was brought to you by Sesame Street:  &quot;One of these things is not like the others.  One of these things just doesn&apos;t belong.&quot;

http://members.tripod.com/tiny_dancer/one.html
</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>8866</attachid>
            <date>2006-06-16 04:29:57 -0700</date>
            <delta_ts>2006-06-16 13:23:04 -0700</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-9463-v1.diff</filename>
            <type>text/plain</type>
            <size>1216</size>
            <attacher name="David Kilzer (:ddkilzer)">ddkilzer</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiAxNDg4NikKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTQgQEAKKzIwMDYtMDYtMTYgIERhdmlkIEtpbHplciAgPGRka2lsemVyQGtpbHpl
ci5uZXQ+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg
aHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4ub3JnL3Nob3dfYnVnLmNnaT9pZD05NDYzCisgICAg
ICAgIFJFR1JFU1NJT04gKHIxNDg3OSk6IEFzc2VydGlvbiBmYWlsdXJlIGluIENTU1BhcnNlcjo6
c2lua0Zsb2F0aW5nU2VsZWN0b3IoKQorICAgICAgICAobV9mbG9hdGluZ1NlbGVjdG9ycy5jb250
YWlucyhzZWxlY3RvcikpIGluIHNvbWUgQ1NTIHRlc3RzCisKKyAgICAgICAgKiBjc3MvQ1NTR3Jh
bW1hci55OiBDcmVhdGUgQ1NTU2VsZWN0b3IgZm9yIEZVTkNUSU9OIHVzaW5nIENTU1BhcnNlcjo6
Y3JlYXRlRmxvYXRpbmdTZWxlY3RvcigpCisgICAgICAgIGluc3RlYWQgb2YgdGhlIGJhcmUgY29u
c3RydWN0b3IuCisKIDIwMDYtMDYtMTUgIEp1c3RpbiBHYXJjaWEgIDxqdXN0aW4uZ2FyY2lhQGFw
cGxlLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBsZXZpCkluZGV4OiBXZWJDb3JlL2Nzcy9D
U1NHcmFtbWFyLnkKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9jc3MvQ1NTR3JhbW1hci55CShyZXZp
c2lvbiAxNDg4NikKKysrIFdlYkNvcmUvY3NzL0NTU0dyYW1tYXIueQkod29ya2luZyBjb3B5KQpA
QCAtODM4LDcgKzgzOCw3IEBAIHBzZXVkbzoKICAgICB9CiAgICAgLy8gdXNlZCBieSA6bGFuZwog
ICAgIHwgJzonIEZVTkNUSU9OIElERU5UICcpJyB7Ci0gICAgICAgICQkID0gbmV3IENTU1NlbGVj
dG9yKCk7CisgICAgICAgICQkID0gc3RhdGljX2Nhc3Q8Q1NTUGFyc2VyKj4ocGFyc2VyKS0+Y3Jl
YXRlRmxvYXRpbmdTZWxlY3RvcigpOwogICAgICAgICAkJC0+bWF0Y2ggPSBDU1NTZWxlY3Rvcjo6
UHNldWRvQ2xhc3M7CiAgICAgICAgICQkLT5hcmd1bWVudCA9IGF0b21pY1N0cmluZygkMyk7CiAg
ICAgICAgICQyLmxvd2VyKCk7Cg==
</data>
<flag name="review"
          id="2584"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>