Bug 92807

Summary: Remove unused method HTMLElementStack::bottom()
Product: WebKit Reporter: Kwang Yul Seo <skyul>
Component: DOMAssignee: Kwang Yul Seo <skyul>
Status: RESOLVED FIXED    
Severity: Normal CC: haraken, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Kwang Yul Seo 2012-07-31 16:19:14 PDT 
HTMLElementStack::bottom() was replaced with HTMLElementStack::rootNode() in r80231.
Comment 1 Kwang Yul Seo 2012-07-31 16:21:01 PDT 
Created attachment 155667 [details]
Patch
Comment 2 Kwang Yul Seo 2012-07-31 16:33:40 PDT 
BTW, r80231 (Bug 48719) seems to be the major source of complexity and irregularity in the HTML parser because HTMLElementStack needs to handle not only Element but also ContainerNode after r80231. I understand this is a necessary evil for performance reasons, but I hope we could lessen the complexity.
Comment 3 Kentaro Hara 2012-07-31 16:46:03 PDT 
Comment on attachment 155667 [details]
Patch

Looks OK
Comment 4 WebKit Review Bot 2012-07-31 17:52:07 PDT 
Comment on attachment 155667 [details]
Patch

Clearing flags on attachment: 155667

Committed r124277: <http://trac.webkit.org/changeset/124277>
Comment 5 WebKit Review Bot 2012-07-31 17:52:10 PDT 
All reviewed patches have been landed.  Closing bug.