63046 2011-06-20 23:36:36 -0700 HTMLLIElement::attach() seems incorrect 2012-07-25 19:10:50 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 72352 1 rolandsteiner webkit-unassigned dominicc haraken hayato morrita shinyak shinyak oldest_to_newest 424293 0 rolandsteiner 2011-06-20 23:36:36 -0700 HTMLLIElement::attach() searches its parents for the containing list. However: .) It ignores intervening "terrain", so will happily pass through whole tables, etc. .) It doesn't check the element's display type, so will accept an <ol> or <ul>, even if their display type is not [un]ordered list. And conversely, it will not consider other elements even if their display type is set to list. This seems incorrect. 424294 1 rolandsteiner 2011-06-20 23:37:06 -0700 i should add that the only outcome of this is incorrect styling. 424295 2 rolandsteiner 2011-06-20 23:38:11 -0700 It will also fail if the <li> is inside shadow DOM, and the containing <ol>/<ul> is outside - i.e., parentNode() should be changed to parentOrHostNode(). 424366 3 dominicc 2011-06-21 03:22:53 -0700 (In reply to comment #0) > HTMLLIElement::attach() searches its parents for the containing list. However: > > .) It ignores intervening "terrain", so will happily pass through whole tables, etc. That seems busted. > .) It doesn't check the element's display type, so will accept an <ol> or <ul>, even if their display type is not [un]ordered list. And conversely, it will not consider other elements even if their display type is set to list. This seems OK. What is the problem with this behavior? > This seems incorrect. 424821 4 rolandsteiner 2011-06-21 15:52:58 -0700 (In reply to comment #3) > (In reply to comment #0) > > .) It doesn't check the element's display type, so will accept an <ol> or <ul>, even if their display type is not [un]ordered list. And conversely, it will not consider other elements even if their display type is set to list. > > This seems OK. What is the problem with this behavior? You're right, of course, esp. as there IS NO 'list' display type (just 'list-item'). @_@;; 503449 5 dominicc 2011-11-15 16:53:08 -0800 May be related to bug 72440. 678752 6 morrita 2012-07-25 19:10:50 -0700 http://trac.webkit.org/changeset/123570 fixed this.