<?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>80638</bug_id>
          
          <creation_ts>2012-03-08 14:04:36 -0800</creation_ts>
          <short_desc>RegionNodeList should be invalidated upon style recalc and DOM attribute changes</short_desc>
          <delta_ts>2017-07-18 08:26:01 -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>DOM</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P1</priority>
          <bug_severity>Critical</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryosuke Niwa">rniwa</reporter>
          <assigned_to name="Mihnea Ovidenie">mihnea</assigned_to>
          <cc>arv</cc>
    
    <cc>eoconnor</cc>
    
    <cc>hyatt</cc>
    
    <cc>mihnea</cc>
    
    <cc>stearns</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>574175</commentid>
    <comment_count>0</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2012-03-08 14:04:36 -0800</bug_when>
    <thetext>Right now, RegionNodeList is only invalidated when the node is added or removed. But it should also be invalidated whenever attribute changes, or for that matter, whenever style recalc needs to happen.

Also, I really don&apos;t think we should implement this as a dynamic node list. static node list would work much better because there&apos;s virtually no use case for using the same region node list while modifying DOM or styles.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>574184</commentid>
    <comment_count>1</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2012-03-08 14:10:39 -0800</bug_when>
    <thetext>We should stop propagating NodeList to new APIs. I don&apos;t think we even need static NodeLists. These should just be regular Node arrays. I suppose DOM4 should officially weigh on on that. We certainly should not add any new dynamic NodeLists to the web platform.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577515</commentid>
    <comment_count>2</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2012-03-13 10:36:30 -0700</bug_when>
    <thetext>I strongly recommend we rollout the patch that implemented the regions node list. The current implementation is making the refactoring work in https://bugs.webkit.org/show_bug.cgi?id=80900 impossible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577531</commentid>
    <comment_count>3</comment_count>
    <who name="Mihnea Ovidenie">mihnea</who>
    <bug_when>2012-03-13 10:47:46 -0700</bug_when>
    <thetext>OK, i will do it asap.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>