<?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>12109</bug_id>
          
          <creation_ts>2007-01-04 04:22:55 -0800</creation_ts>
          <short_desc>layer visibility incorrect in some cases</short_desc>
          <delta_ts>2022-01-13 23:20:03 -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>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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Antti Koivisto">koivisto</reporter>
          <assigned_to name="Antti Koivisto">koivisto</assigned_to>
          <cc>mitz</cc>
    
    <cc>simon.fraser</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>37174</commentid>
    <comment_count>0</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2007-01-04 04:22:55 -0800</bug_when>
    <thetext>Mishandling of sublayers in RenderLayer::updateVisibilityStatus() can cause a layer that has visible content be marked as not having any.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>37172</commentid>
    <comment_count>1</comment_count>
      <attachid>12212</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2007-01-04 04:25:16 -0800</bug_when>
    <thetext>Created attachment 12212
test case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>37170</commentid>
    <comment_count>2</comment_count>
      <attachid>12214</attachid>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2007-01-04 04:36:44 -0800</bug_when>
    <thetext>Created attachment 12214
fixed

My original visibility crawl loop is screwed up, it does not check nodes that are succeeding siblings of layered nodes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36919</commentid>
    <comment_count>3</comment_count>
      <attachid>12214</attachid>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2007-01-05 00:16:21 -0800</bug_when>
    <thetext>Comment on attachment 12214
fixed

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36722</commentid>
    <comment_count>4</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2007-01-06 04:26:09 -0800</bug_when>
    <thetext>Committed in r18616 by antti with test case LayoutTests/fast/layers/layer-visibility-sublayer.html.

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36725</commentid>
    <comment_count>5</comment_count>
    <who name="">mitz</who>
    <bug_when>2007-01-06 05:52:25 -0800</bug_when>
    <thetext>The layout test is invalid, giving the same results with and without the patch. Reopening the bug until a valid layout test is checked in.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>36729</commentid>
    <comment_count>6</comment_count>
      <attachid>12214</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2007-01-06 07:42:36 -0800</bug_when>
    <thetext>Comment on attachment 12214
fixed

Clearing hyatt&apos;s r+ so this doesn&apos;t show up in commit queue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1830822</commentid>
    <comment_count>7</comment_count>
    <who name="Antti Koivisto">koivisto</who>
    <bug_when>2022-01-13 23:18:19 -0800</bug_when>
    <thetext>Doesn&apos;t seem like the test was ever fixed but we likely have coverage elsewhere by now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1830824</commentid>
    <comment_count>8</comment_count>
    <who name="">mitz</who>
    <bug_when>2022-01-13 23:20:03 -0800</bug_when>
    <thetext>OK.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>12212</attachid>
            <date>2007-01-04 04:25:16 -0800</date>
            <delta_ts>2007-01-04 04:25:16 -0800</delta_ts>
            <desc>test case</desc>
            <filename>visibility-sublayer-bug.html</filename>
            <type>text/html</type>
            <size>2017</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">PGh0bWw+CjxzdHlsZT4KLnJlZCB7IGJvcmRlcjogMnB4IHNvbGlkIHJlZCB9Ci5ncmVlbiB7IGJv
cmRlcjogMnB4IHNvbGlkIGdyZWVuIH0KLnZpc2libGUgeyB2aXNpYmlsaXR5OiB2aXNpYmxlfQou
aW52aXNpYmxlIHsgdmlzaWJpbGl0eTogaGlkZGVufQovLy5pbnZpc2libGU6aG92ZXIgeyB2aXNp
YmlsaXR5OiB2aXNpYmxlfQouYWJzdG9wIHsgcG9zaXRpb246IGFic29sdXRlOyBsZWZ0OjA7IHJp
Z2h0OjA7IGhlaWdodDozMHB4ICAgfQouYWJzIHsgcG9zaXRpb246IGFic29sdXRlOyBsZWZ0OjA7
IHJpZ2h0OjA7IHRvcDowOyBib3R0b206MCB9Cjwvc3R5bGU+Cgo8c2NyaXB0Pgp2YXIgbm9kZTE7
CnZhciBub2RlMjsKZnVuY3Rpb24gZG90ZXN0KCkKewoKLy8yNApkb2N1bWVudC5nZXRFbGVtZW50
QnlJZCgnMjRjJykuc3R5bGUuc2V0UHJvcGVydHkoJ3Zpc2liaWxpdHknLCdoaWRkZW4nLCcnKTsK
ZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJzI0YScpLnJlbW92ZUNoaWxkKGRvY3VtZW50LmdldEVs
ZW1lbnRCeUlkKCcyNGInKSk7CgovLzI1CmRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCcyNWMnKS5z
dHlsZS5zZXRQcm9wZXJ0eSgndmlzaWJpbGl0eScsJ2hpZGRlbicsJycpOwpkb2N1bWVudC5nZXRF
bGVtZW50QnlJZCgnMjVhJykucmVtb3ZlQ2hpbGQoZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJzI1
YicpKTsKCi8vMjYKZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJzI2YycpLnN0eWxlLnNldFByb3Bl
cnR5KCd2aXNpYmlsaXR5JywnaGlkZGVuJywnJyk7CmRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCcy
NmEnKS5yZW1vdmVDaGlsZChkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgnMjZiJykpOwpzZXRUaW1l
b3V0KGRvdGVzdDIsMjApOwp9CgpmdW5jdGlvbiBkb3Rlc3QyKCkKewoKLy94eAovL2RvY3VtZW50
LmdldEVsZW1lbnRCeUlkKCcyMycpLnN0eWxlLnNldFByb3BlcnR5KCd2aXNpYmlsaXR5JywnaGlk
ZGVuJywnJyk7Cn0KCjwvc2NyaXB0Pgo8L2hlYWQ+Cjxib2R5IG9ubG9hZD0ic2V0VGltZW91dChk
b3Rlc3QsMjApIj4KPGJyPjxicj48YnI+PGJyPgoyNCBncmVlbiBib3ggd2l0aCB3b3JkIG9rOgo8
ZGl2IGlkPSIyNGEiIGNsYXNzPSJpbnZpc2libGUgYWJzdG9wIHJlZCI+CjxkaXYgY2xhc3M9ImFi
cyBpbnZpc2libGUgcmVkIj4KPHNwYW4+YmxhaDwvc3Bhbj4KPC9kaXY+CjxkaXYgaWQ9IjI0YiIg
Y2xhc3M9ImFicyBpbnZpc2libGUgcmVkIj4KPC9kaXY+CjxkaXYgaWQ9IjI0YyIgY2xhc3M9InZp
c2libGUgcmVkIj4KPC9kaXY+CjxkaXYgY2xhc3M9InZpc2libGUgZ3JlZW4iPgpvawo8L2Rpdj4K
PC9kaXY+Cjxicj48YnI+PGJyPjxicj4KCjI1IGdyZWVuIGJveCB3aXRoIHdvcmQgb2s6CjxkaXYg
aWQ9IjI1YSIgY2xhc3M9ImludmlzaWJsZSBhYnN0b3AgcmVkIj4KPGRpdiBjbGFzcz0iaW52aXNp
YmxlIHJlZCI+CjwvZGl2Pgo8ZGl2IGNsYXNzPSJhYnMgaW52aXNpYmxlIHJlZCI+CjwvZGl2Pgo8
ZGl2IGlkPSIyNWIiIGNsYXNzPSJhYnMgaW52aXNpYmxlIHJlZCI+CjxzcGFuPmJsYWg8L3NwYW4+
CjwvZGl2Pgo8ZGl2IGlkPSIyNWMiIGNsYXNzPSJ2aXNpYmxlIHJlZCI+CjwvZGl2Pgo8ZGl2IGNs
YXNzPSJ2aXNpYmxlIGdyZWVuIj4Kb2sKPC9kaXY+CjwvZGl2Pgo8YnI+PGJyPjxicj48YnI+Cgoy
NiBncmVlbiBib3ggd2l0aCB3b3JkIG9rOgo8ZGl2IGlkPSIyNmEiIGNsYXNzPSJpbnZpc2libGUg
YWJzdG9wIHJlZCI+CjxkaXYgY2xhc3M9ImludmlzaWJsZSByZWQiPgo8ZGl2PmJsYWg8L2Rpdj4K
PC9kaXY+CjxkaXYgY2xhc3M9ImFicyBpbnZpc2libGUgcmVkIj4KPHNwYW4+YmxhaDwvc3Bhbj4K
PC9kaXY+CjxkaXYgaWQ9IjI2YiIgY2xhc3M9ImFicyBpbnZpc2libGUgcmVkIj4KPHNwYW4+Ymxh
aDwvc3Bhbj4KPC9kaXY+CjxkaXYgaWQ9IjI2YyIgY2xhc3M9InZpc2libGUgcmVkIj4KPC9kaXY+
CjxkaXYgY2xhc3M9InZpc2libGUgZ3JlZW4iPgpvawo8L2Rpdj4KPC9kaXY+Cjxicj48YnI+PGJy
Pjxicj4KPC9ib2R5Pgo8L2h0bWw+Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>12214</attachid>
            <date>2007-01-04 04:36:44 -0800</date>
            <delta_ts>2010-06-10 14:25:49 -0700</delta_ts>
            <desc>fixed</desc>
            <filename>visibility-sublayer.patch</filename>
            <type>text/plain</type>
            <size>1329</size>
            <attacher name="Antti Koivisto">koivisto</attacher>
            
              <data encoding="base64">SW5kZXg6IFJlbmRlckxheWVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBSZW5kZXJMYXllci5jcHAJKHJl
dmlzaW9uIDE4NTUzKQorKysgUmVuZGVyTGF5ZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0yOTIs
MjAgKzI5MiwyMCBAQCB2b2lkIFJlbmRlckxheWVyOjp1cGRhdGVWaXNpYmlsaXR5U3RhdHVzCiAg
ICAgICAgICAgICBtX2hhc1Zpc2libGVDb250ZW50ID0gZmFsc2U7CiAgICAgICAgICAgICBSZW5k
ZXJPYmplY3QqIHIgPSBtX29iamVjdC0+Zmlyc3RDaGlsZCgpOwogICAgICAgICAgICAgd2hpbGUg
KHIpIHsKLSAgICAgICAgICAgICAgICBpZiAoci0+c3R5bGUoKS0+dmlzaWJpbGl0eSgpID09IFZJ
U0lCTEUpIHsKKyAgICAgICAgICAgICAgICBpZiAoci0+c3R5bGUoKS0+dmlzaWJpbGl0eSgpID09
IFZJU0lCTEUgJiYgIXItPmxheWVyKCkpIHsKICAgICAgICAgICAgICAgICAgICAgbV9oYXNWaXNp
YmxlQ29udGVudCA9IHRydWU7CiAgICAgICAgICAgICAgICAgICAgIGJyZWFrOwogICAgICAgICAg
ICAgICAgIH0KLSAgICAgICAgICAgICAgICBpZiAoci0+Zmlyc3RDaGlsZCgpICYmICFyLT5maXJz
dENoaWxkKCktPmxheWVyKCkpCisgICAgICAgICAgICAgICAgaWYgKHItPmZpcnN0Q2hpbGQoKSAm
JiAhci0+bGF5ZXIoKSkKICAgICAgICAgICAgICAgICAgICAgciA9IHItPmZpcnN0Q2hpbGQoKTsK
LSAgICAgICAgICAgICAgICBlbHNlIGlmIChyLT5uZXh0U2libGluZygpICYmICFyLT5uZXh0U2li
bGluZygpLT5sYXllcigpKQorICAgICAgICAgICAgICAgIGVsc2UgaWYgKHItPm5leHRTaWJsaW5n
KCkpCiAgICAgICAgICAgICAgICAgICAgIHIgPSByLT5uZXh0U2libGluZygpOwogICAgICAgICAg
ICAgICAgIGVsc2UgewotICAgICAgICAgICAgICAgICAgICB3aGlsZSAociAmJiAoIXItPm5leHRT
aWJsaW5nKCkgfHwgci0+bmV4dFNpYmxpbmcoKS0+bGF5ZXIoKSkpIHsKKyAgICAgICAgICAgICAg
ICAgICAgZG8gewogICAgICAgICAgICAgICAgICAgICAgICAgciA9IHItPnBhcmVudCgpOwogICAg
ICAgICAgICAgICAgICAgICAgICAgaWYgKHI9PW1fb2JqZWN0KQogICAgICAgICAgICAgICAgICAg
ICAgICAgICAgIHIgPSAwOwotICAgICAgICAgICAgICAgICAgICB9CisgICAgICAgICAgICAgICAg
ICAgIH0gd2hpbGUgKHIgJiYgIXItPm5leHRTaWJsaW5nKCkpOwogICAgICAgICAgICAgICAgICAg
ICBpZiAocikKICAgICAgICAgICAgICAgICAgICAgICAgIHIgPSByLT5uZXh0U2libGluZygpOwog
ICAgICAgICAgICAgICAgIH0K
</data>

          </attachment>
      

    </bug>

</bugzilla>