Bug 86350 - [Refactoring] Get rid of ContentDistribution::Item
Summary: [Refactoring] Get rid of ContentDistribution::Item
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Hajime Morrita
URL:
Keywords:
Depends on: 86064
Blocks:
  Show dependency tree / graph
 
Reported: 2012-05-14 03:51 PDT by Hajime Morrita
Modified: 2012-05-14 23:29 PDT (History)
2 users (show)

See Also:

Attachments
Patch (18.42 KB, patch)
2012-05-14 18:50 PDT, Hajime Morrita 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.