Bug 83641 - Chromium perf bots should not build
Summary: Chromium perf bots should not build
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: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks: 77037
  Show dependency tree / graph
 
Reported: 2012-04-10 17:35 PDT by Ryosuke Niwa
Modified: 2012-04-15 16:15 PDT (History)
5 users (show)

See Also:

Attachments
Patch (5.20 KB, patch)
2012-04-10 17:37 PDT, Ryosuke Niwa 		eric: review+
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 Ryosuke Niwa 2012-04-10 17:35:57 PDT 
Chromium perf bots should not build
Comment 1 Ryosuke Niwa 2012-04-10 17:37:02 PDT 
Created attachment 136582 [details]
Patch
Comment 2 Ryosuke Niwa 2012-04-10 17:39:42 PDT 
e.g. on this Chromium Mac build, we could reduce the cycle time by as much as 38 minutes (out of 88 minutes):
http://build.webkit.org/builders/Chromium%20Mac%20Release%20%28Perf%29/builds/1465
Comment 3 Ryosuke Niwa 2012-04-10 17:41:43 PDT 
(In reply to comment #2)
> e.g. on this Chromium Mac build, we could reduce the cycle time by as much as 38 minutes (out of 88 minutes):
> http://build.webkit.org/builders/Chromium%20Mac%20Release%20%28Perf%29/builds/1465

Ugh... actually 45 minutes - (whatever takes to download the archive; roughly one minute on http://build.webkit.org/builders/Chromium%20Mac%20Release%20%28Tests%29/builds/15116).
Comment 4 Eric Seidel (no email) 2012-04-12 20:43:10 PDT 
Comment on attachment 136582 [details]
Patch

I can rubber-stamp this, but I'm not useful for a real review.
Comment 5 Ryosuke Niwa 2012-04-15 16:15:22 PDT 
Committed r114214: <http://trac.webkit.org/changeset/114214>