RESOLVED FIXED 114607
[Dashboard] Clean up the builder-loading code 
https://bugs.webkit.org/show_bug.cgi?id=114607
Summary [Dashboard] Clean up the builder-loading code
Zan Dobersek
Reported 2013-04-15 00:54:10 PDT
[Dashboard] Clean up the builder-loading code
Attachments
Patch (17.83 KB, patch)
2013-04-15 01:05 PDT, Zan Dobersek 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2013-04-15 01:05:45 PDT
Created attachment 198031 [details] Patch
Ryosuke Niwa
Comment 2 2013-04-15 10:20:07 PDT
All these cleanup patches are too big to review. Since I don't know this codebase at all, I need to see much smaller self-contained patches.
Zan Dobersek
Comment 3 2013-04-15 13:37:52 PDT
(In reply to comment #2) > All these cleanup patches are too big to review. Since I don't know this codebase at all, I need to see much smaller self-contained patches. Understood. I'll split this into smaller pieces.
Zan Dobersek
Comment 4 2013-04-18 04:06:37 PDT
Split this up into the following bugs: - bug #114795 - bug #114796 - bug #114797 - bug #114799 - bug #114801 - bug #114802 - bug #114803 - bug #114804 - bug #114805
Zan Dobersek
Comment 5 2013-04-19 01:49:53 PDT
Split patches landed, thanks for the reviews.
Note You need to log in before you can comment on or make changes to this bug.