Bug 114607 - [Dashboard] Clean up the builder-loading code
Summary: [Dashboard] Clean up the builder-loading code
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zan Dobersek
URL:
Keywords:
Depends on: 114795 114796 114797 114799 114801 114802 114803 114804 114805
Blocks: 114971
  Show dependency treegraph
 
Reported: 2013-04-15 00:54 PDT by Zan Dobersek
Modified: 2013-04-22 11:28 PDT (History)
1 user (show)

See Also:


Attachments
Patch (17.83 KB, patch)
2013-04-15 01:05 PDT, Zan Dobersek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2013-04-15 00:54:10 PDT
[Dashboard] Clean up the builder-loading code
Comment 1 Zan Dobersek 2013-04-15 01:05:45 PDT
Created attachment 198031 [details]
Patch
Comment 2 Ryosuke Niwa 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.
Comment 3 Zan Dobersek 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.
Comment 4 Zan Dobersek 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
Comment 5 Zan Dobersek 2013-04-19 01:49:53 PDT
Split patches landed, thanks for the reviews.