Bug 86350

Summary: [Refactoring] Get rid of ContentDistribution::Item
Product: WebKit Reporter: Hajime Morrita <morrita>
Component: DOMAssignee: Hajime Morrita <morrita>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 86064    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Description Hajime Morrita 2012-05-14 03:51:34 PDT
This is followup of Bug 86064. 
A dual-link list ContentDistribution::Item has several flaw. And the class itself should be abandoned.
Comment 1 Hajime Morrita 2012-05-14 18:50:20 PDT
Created attachment 141837 [details]
Patch
Comment 2 Hajime Morrita 2012-05-14 18:51:30 PDT
Hi Dimitri, could you take a look?
This simplification makes content related code much easier to read.
Comment 3 Dimitri Glazkov (Google) 2012-05-14 19:51:58 PDT
Comment on attachment 141837 [details]
Patch

beautiful.
Comment 4 Hajime Morrita 2012-05-14 21:58:58 PDT
Comment on attachment 141837 [details]
Patch

Yay!
Comment 5 WebKit Review Bot 2012-05-14 23:29:43 PDT
Comment on attachment 141837 [details]
Patch

Clearing flags on attachment: 141837

Committed r117033: <http://trac.webkit.org/changeset/117033>
Comment 6 WebKit Review Bot 2012-05-14 23:29:48 PDT
All reviewed patches have been landed.  Closing bug.