<?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>38272</bug_id>
          
          <creation_ts>2010-04-28 10:52:24 -0700</creation_ts>
          <short_desc>SHOULD NEVER BE REACHED assertion loading forbes.com</short_desc>
          <delta_ts>2010-05-05 18:23:32 -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>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>http://www.forbes.com</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P1</priority>
          <bug_severity>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Simon Fraser (smfr)">simon.fraser</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>darin</cc>
    
    <cc>hamaji</cc>
    
    <cc>yuzo</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>218474</commentid>
    <comment_count>0</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2010-04-28 10:52:24 -0700</bug_when>
    <thetext>I hit the SHOULD NEVER BE REACHED assertion here:

(WebCore/css/CSSStyleSelector.cpp:2562 bool WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector(WebCore::CSSSelector*, WebCore::Element*, WTF::HashSet&lt;WebCore::AtomicStringImpl*, WTF::PtrHash&lt;WebCore::AtomicStringImpl*&gt;, WTF::HashTraits&lt;WebCore::AtomicStringImpl*&gt; &gt;*, WebCore::PseudoId&amp;, bool, WebCore::RenderStyle*, WebCore::RenderStyle*) const)

loading forbes.com</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218489</commentid>
    <comment_count>1</comment_count>
      <attachid>54587</attachid>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2010-04-28 11:00:13 -0700</bug_when>
    <thetext>Created attachment 54587
Testcase</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218490</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2010-04-28 11:01:27 -0700</bug_when>
    <thetext>Why are we picking up :first outside of @page?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>218500</commentid>
    <comment_count>3</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2010-04-28 11:10:53 -0700</bug_when>
    <thetext>Regression from bug 35329, http://trac.webkit.org/changeset/58374</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>219219</commentid>
    <comment_count>4</comment_count>
    <who name="Simon Fraser (smfr)">simon.fraser</who>
    <bug_when>2010-04-29 15:41:15 -0700</bug_when>
    <thetext>Ping? I keep hitting this.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>219234</commentid>
    <comment_count>5</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2010-04-29 16:02:47 -0700</bug_when>
    <thetext>If we don’t get this resolved soon, I suggest we roll out r58374 and add a test for this to the layout tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>219235</commentid>
    <comment_count>6</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2010-04-29 16:03:09 -0700</bug_when>
    <thetext>&lt;rdar://problem/7925163&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>219391</commentid>
    <comment_count>7</comment_count>
      <attachid>54787</attachid>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-04-30 00:35:23 -0700</bug_when>
    <thetext>Created attachment 54787
Patch v1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>219392</commentid>
    <comment_count>8</comment_count>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-04-30 00:43:56 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; Created an attachment (id=54787) [details]
&gt; Patch v1

Hi, sorry for the latency. Yuzo is taking his vacation so I&apos;ll take care this bug.

I guess rolling out r58374 can be difficult as we&apos;ve already landed other patches which depend on r58374. Instead of reverting the change, I&apos;d like to just ignore paged media related pseudo classes for now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>219497</commentid>
    <comment_count>9</comment_count>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-04-30 08:46:18 -0700</bug_when>
    <thetext>Committed r58578: &lt;http://trac.webkit.org/changeset/58578&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221419</commentid>
    <comment_count>10</comment_count>
    <who name="Yuzo Fujishima">yuzo</who>
    <bug_when>2010-05-05 18:23:32 -0700</bug_when>
    <thetext>Sorry for the regression and thank you for the fix.

Yuzo</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>54587</attachid>
            <date>2010-04-28 11:00:13 -0700</date>
            <delta_ts>2010-04-28 11:00:13 -0700</delta_ts>
            <desc>Testcase</desc>
            <filename>pseudo-first.html</filename>
            <type>text/html</type>
            <size>214</size>
            <attacher name="Simon Fraser (smfr)">simon.fraser</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+Cgo8aHRtbD4KPGhlYWQ+CiAgPHN0eWxlIHR5cGU9InRleHQvY3NzIiBt
ZWRpYT0ic2NyZWVuIj4KICAgIGRpdjpmaXJzdCB7CiAgICAgIGNvbG9yOiByZWQ7CiAgICB9CiAg
PC9zdHlsZT4KPC9oZWFkPgo8Ym9keT4KICA8ZGl2IGNsYXNzPSJ0ZXN0Ij4KICAgIDxwPm9uZTwv
cD4KICAgIDxwPnR3bzwvcD4KICA8L2Rpdj4KPC9ib2R5Pgo8L2h0bWw+Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>54787</attachid>
            <date>2010-04-30 00:35:23 -0700</date>
            <delta_ts>2010-04-30 08:32:23 -0700</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-38272-20100430003521.patch</filename>
            <type>text/plain</type>
            <size>4411</size>
            <attacher name="Shinichiro Hamaji">hamaji</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBjZjNlZGYyZGEyZWNkN2FjNTVlMjU3NTFkYTMxZjFkYjlhNTMzOTAxLi4zYzFiZTRi
ZGJiYjc2MzczNmNhZmEwNmNkZmRiZTMwYjYwNjg2MTExIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAK
KzIwMTAtMDQtMzAgIFNoaW5pY2hpcm8gSGFtYWppICA8aGFtYWppQGNocm9taXVtLm9yZz4KKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBTSE9VTEQgTkVW
RVIgQkUgUkVBQ0hFRCBhc3NlcnRpb24gbG9hZGluZyBmb3JiZXMuY29tCisgICAgICAgIGh0dHBz
Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zODI3MgorCisgICAgICAgICogcHJp
bnRpbmcvcHNldWRvLWNsYXNzLW91dHNpZGUtcGFnZS1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAg
ICAgICAqIHByaW50aW5nL3BzZXVkby1jbGFzcy1vdXRzaWRlLXBhZ2UuaHRtbDogQWRkZWQuCisK
IDIwMTAtMDQtMjkgIEtlbnQgVGFtdXJhICA8dGtlbnRAY2hyb21pdW0ub3JnPgogCiAgICAgICAg
IFVucmV2aWV3ZWQuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9wcmludGluZy9wc2V1ZG8tY2xh
c3Mtb3V0c2lkZS1wYWdlLWV4cGVjdGVkLnR4dCBiL0xheW91dFRlc3RzL3ByaW50aW5nL3BzZXVk
by1jbGFzcy1vdXRzaWRlLXBhZ2UtZXhwZWN0ZWQudHh0Cm5ldyBmaWxlIG1vZGUgMTAwNjQ0Cmlu
ZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLjFkNjQ3ZTA3NTIx
YmU3NTUyMjhhY2ViM2FkZmVhYzM5ODNmNmJmYWQKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRU
ZXN0cy9wcmludGluZy9wc2V1ZG8tY2xhc3Mtb3V0c2lkZS1wYWdlLWV4cGVjdGVkLnR4dApAQCAt
MCwwICsxLDUgQEAKK1Rlc3QgY2FzZSBmb3IgU0hPVUxEIE5FVkVSIEJFIFJFQUNIRUQgYXNzZXJ0
aW9uIGxvYWRpbmcgZm9yYmVzLmNvbS4gUGFnZWQgbWVkaWEgcmVsYXRlZCBwc2V1ZG8tY2xhc3Nl
cyBzaG91bGQgYmUgaWdub3JlZCBvdXRzaWRlIG9mIEBwYWdlLgorCitZb3Ugc2hvdWxkIHNlZSBh
IGdyZWVuIGJveC4KKworUEFTUwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcHJpbnRpbmcvcHNl
dWRvLWNsYXNzLW91dHNpZGUtcGFnZS5odG1sIGIvTGF5b3V0VGVzdHMvcHJpbnRpbmcvcHNldWRv
LWNsYXNzLW91dHNpZGUtcGFnZS5odG1sCm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAw
MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLjlkODhhMDMzMzFkMWEzNGYwYWRj
YTVmNWRkNzc2ODgxY2JjZDliNjIKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRUZXN0cy9wcmlu
dGluZy9wc2V1ZG8tY2xhc3Mtb3V0c2lkZS1wYWdlLmh0bWwKQEAgLTAsMCArMSw1NCBAQAorPCFE
T0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorCis8c3R5bGUgdHlwZT0idGV4dC9jc3MiIG1l
ZGlhPSJzY3JlZW4iPgorICAgICNncmVlbiB7CisgICAgICAgIGJhY2tncm91bmQtY29sb3I6IGdy
ZWVuOworICAgIH0KKyAgICAjdGVzdCB7CisgICAgICAgIHdpZHRoOiAxMDBweDsKKyAgICAgICAg
aGVpZ2h0OiAxMDBweDsKKyAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogZ3JlZW47CisgICAgfQor
ICAgIC8qIFRoZXNlIHNlbGVjdG9ycyBzaG91bGQgYmUgaWdub3JlZCAqLworICAgICN0ZXN0OmZp
cnN0IHsKKyAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogcmVkOworICAgIH0KKyAgICAjdGVzdDps
ZWZ0IHsKKyAgICAgICAgYmFja2dyb3VuZC1jb2xvcjogcmVkOworICAgIH0KKyAgICAjdGVzdDpy
aWdodCB7CisgICAgICAgIGJhY2tncm91bmQtY29sb3I6IHJlZDsKKyAgICB9Cis8L3N0eWxlPgor
PHNjcmlwdD4KKyAgICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgICAgICBs
YXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisKKyAgICBmdW5jdGlvbiB0ZXN0KCkK
KyAgICB7CisgICAgICAgIHZhciBleHBlY3RlZCA9IGdldENvbXB1dGVkU3R5bGUoZG9jdW1lbnQu
Z2V0RWxlbWVudEJ5SWQoImdyZWVuIiksIG51bGwpLmJhY2tncm91bmRDb2xvcjsKKyAgICAgICAg
dmFyIGFjdHVhbCA9IGdldENvbXB1dGVkU3R5bGUoZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRl
c3QiKSwgbnVsbCkuYmFja2dyb3VuZENvbG9yOworICAgICAgICB2YXIgcmVzdWx0ID0gIlBBU1Mi
OworICAgICAgICBpZiAoYWN0dWFsICE9IGV4cGVjdGVkKQorICAgICAgICAgICAgcmVzdWx0ID0g
IkV4cGVjdGVkICIgKyBleHBlY3RlZCArICIsIGJ1dCBjb21lcyAiICsgYWN0dWFsOworICAgICAg
ICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgicmVzdWx0IikuaW5uZXJIVE1MID0gcmVzdWx0Owor
ICAgIH0KKzwvc2NyaXB0PgorPC9oZWFkPgorPGJvZHkgb25sb2FkPSJ0ZXN0KCkiPgorCis8cD4K
K1Rlc3QgY2FzZSBmb3IgPGEgaHJlZj0iaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTM4MjcyIj5TSE9VTEQgTkVWRVIgQkUgUkVBQ0hFRCBhc3NlcnRpb24gbG9hZGluZyBm
b3JiZXMuY29tPC9hPi4KK1BhZ2VkIG1lZGlhIHJlbGF0ZWQgcHNldWRvLWNsYXNzZXMgc2hvdWxk
IGJlIGlnbm9yZWQgb3V0c2lkZSBvZiBAcGFnZS4KKworPHA+CitZb3Ugc2hvdWxkIHNlZSBhIGdy
ZWVuIGJveC4KKworPGRpdiBpZD0iZ3JlZW4iPjwvZGl2PgorPGRpdiBpZD0idGVzdCI+PC9kaXY+
Cis8ZGl2IGlkPSJyZXN1bHQiPjwvZGl2PgorCis8L2JvZHk+Cis8L2h0bWw+CmRpZmYgLS1naXQg
YS9XZWJDb3JlL0NoYW5nZUxvZyBiL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4IDI5YzY2YzhiZGZm
YzQ5ZTU3NTM1ZmMwZGJiNDg0YjZjNDIyOWUwZjEuLjgyMjllZjlmMmY3Njk4NjRmYjhhNzBhYTE5
Yjk3NzM3YTY2Y2E5NDUgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1dlYkNv
cmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTAtMDQtMzAgIFNoaW5pY2hpcm8gSGFt
YWppICA8aGFtYWppQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg
KE9PUFMhKS4KKworICAgICAgICBTSE9VTEQgTkVWRVIgQkUgUkVBQ0hFRCBhc3NlcnRpb24gbG9h
ZGluZyBmb3JiZXMuY29tCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD0zODI3MgorCisgICAgICAgIElnbm9yZSBwYWdlIG1lZGlhIHJlbGF0ZWQgcHNldWRv
IGNsYXNzZXMuCisKKyAgICAgICAgVGVzdDogcHJpbnRpbmcvcHNldWRvLWNsYXNzLW91dHNpZGUt
cGFnZS5odG1sCisKKyAgICAgICAgKiBjc3MvQ1NTU3R5bGVTZWxlY3Rvci5jcHA6CisgICAgICAg
IChXZWJDb3JlOjpDU1NTdHlsZVNlbGVjdG9yOjpTZWxlY3RvckNoZWNrZXI6OmNoZWNrT25lU2Vs
ZWN0b3IpOgorCiAyMDEwLTA0LTI5ICBBZGFtIEJhcnRoICA8YWJhcnRoQHdlYmtpdC5vcmc+CiAK
ICAgICAgICAgVW5yZXZpZXdlZC4gIFVwZGF0ZSBKU0MgQ29kZUdlbmVyYXRvciBiYXNlbGluZS4g
IE5vdCBzdXJlIGhvdyBJIG1pc3NlZApkaWZmIC0tZ2l0IGEvV2ViQ29yZS9jc3MvQ1NTU3R5bGVT
ZWxlY3Rvci5jcHAgYi9XZWJDb3JlL2Nzcy9DU1NTdHlsZVNlbGVjdG9yLmNwcAppbmRleCA1ZmIw
MTA5ZTVjN2M1ZTZiZmY1NzkyYjgzNjk4YjdmYTc2ZDY5OTc5Li45MjdlZjkxN2YzYjA0NDhiNDNi
NzU0NGQ4NmNlZDYwZWFkNTA2MzllIDEwMDY0NAotLS0gYS9XZWJDb3JlL2Nzcy9DU1NTdHlsZVNl
bGVjdG9yLmNwcAorKysgYi9XZWJDb3JlL2Nzcy9DU1NTdHlsZVNlbGVjdG9yLmNwcApAQCAtMjU1
Niw2ICsyNTU2LDExIEBAIGJvb2wgQ1NTU3R5bGVTZWxlY3Rvcjo6U2VsZWN0b3JDaGVja2VyOjpj
aGVja09uZVNlbGVjdG9yKENTU1NlbGVjdG9yKiBzZWwsIEVsZW1lCiAgICAgICAgICAgICAgICAg
ICAgIGJyZWFrOwogICAgICAgICAgICAgICAgIHJldHVybiB0cnVlOwogICAgICAgICAgICAgfQor
ICAgICAgICAgICAgY2FzZSBDU1NTZWxlY3Rvcjo6UHNldWRvTGVmdFBhZ2U6CisgICAgICAgICAg
ICBjYXNlIENTU1NlbGVjdG9yOjpQc2V1ZG9SaWdodFBhZ2U6CisgICAgICAgICAgICBjYXNlIENT
U1NlbGVjdG9yOjpQc2V1ZG9GaXJzdFBhZ2U6CisgICAgICAgICAgICAgICAgLy8gUGFnZSBtZWRp
YSByZWxhdGVkIHBzZXVkby1jbGFzc2VzIGFyZSBub3QgaGFuZGxlZCB5ZXQuCisgICAgICAgICAg
ICAgICAgcmV0dXJuIGZhbHNlOwogICAgICAgICAgICAgY2FzZSBDU1NTZWxlY3Rvcjo6UHNldWRv
VW5rbm93bjoKICAgICAgICAgICAgIGNhc2UgQ1NTU2VsZWN0b3I6OlBzZXVkb05vdFBhcnNlZDoK
ICAgICAgICAgICAgIGRlZmF1bHQ6Cg==
</data>
<flag name="review"
          id="38828"
          type_id="1"
          status="+"
          setter="simon.fraser"
    />
          </attachment>
      

    </bug>

</bugzilla>