Re: [dev-servo] Layout reads content data members from elements

2014-02-14 Thread Ms2ger
On 02/15/2014 03:16 AM, Josh Matthews wrote: For the short term I'm interested in making layout code always go for the attributes, because relying on sprinkling wait_until_safe_to_modify_dom all over content code is easy to get wrong. Thoughts? As long as it doesn't mean we have to duplicate th

Re: [dev-servo] Layout reads content data members from elements

2014-02-14 Thread Boris Zbarsky
On 2/14/14 9:16 PM, Josh Matthews wrote: For the short term I'm interested in making layout code always go for the attributes, because relying on sprinkling wait_until_safe_to_modify_dom all over content code is easy to get wrong. Thoughts? There's some DOM state layout needs that's not actuall

[dev-servo] Layout reads content data members from elements

2014-02-14 Thread Josh Matthews
I filed https://github.com/mozilla/servo/issues/1676 a few days ago but there hasn't been any discussion yet; I think it's important to sort out this problem now so we don't have to do a lot of backtracking when we get around to implementing the COW DOM. In brief, we have various methods in la