Bug 114607

Summary: [Dashboard] Clean up the builder-loading code
Product: WebKit Reporter: Zan Dobersek <zan>
Component: New BugsAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 114795, 114796, 114797, 114799, 114801, 114802, 114803, 114804, 114805    
Bug Blocks: 114971    
Attachments:
Description Flags
Patch none

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.