RESOLVED FIXED 89267
nrwt: restructure manager code 
https://bugs.webkit.org/show_bug.cgi?id=89267
Summary nrwt: restructure manager code
Dirk Pranke
Reported 2012-06-15 19:34:27 PDT
nrwt: restructure manager code
Attachments
Patch (126.17 KB, patch)
2012-06-15 19:34 PDT, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
roll-up patch based of of r121304 + mwb_cleanup + tw_cleanup (127.82 KB, patch)
2012-06-26 19:49 PDT, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2012-06-15 19:34:55 PDT
Created attachment 147940 [details] Patch
Dirk Pranke
Comment 2 2012-06-26 19:49:01 PDT
Created attachment 149668 [details] roll-up patch based of of r121304 + mwb_cleanup + tw_cleanup Definitely not for review ... this patch needs a little more work plus all the tests revamped, then broken into more reviewable chunks, of course.
Dirk Pranke
Comment 3 2012-06-26 19:49:42 PDT
I don't think this technically relies on 76022, but it makes my git branching easier to think that it does ...
Dirk Pranke
Comment 4 2012-07-26 19:44:47 PDT
Okay, I've done most of the remaining work for this, so more patches will be showing up shortly ...
Dirk Pranke
Comment 5 2012-07-31 18:56:24 PDT
All patches have been landed (hopefully they'll stick). manager.py is almost a thousand lines smaller compared to the version in r122384 (the last version before I started moving stuff): 1582 lines -> 619 :).
Note You need to log in before you can comment on or make changes to this bug.