diff options
-rw-r--r-- | chrome/common/extensions/docs/examples/extensions/buildbot/bg.js | 9 | ||||
-rw-r--r-- | chrome/common/extensions/docs/examples/extensions/buildbot/manifest.json | 2 |
2 files changed, 8 insertions, 3 deletions
diff --git a/chrome/common/extensions/docs/examples/extensions/buildbot/bg.js b/chrome/common/extensions/docs/examples/extensions/buildbot/bg.js index cff9f8e..a351678 100644 --- a/chrome/common/extensions/docs/examples/extensions/buildbot/bg.js +++ b/chrome/common/extensions/docs/examples/extensions/buildbot/bg.js @@ -176,7 +176,9 @@ function fetchPatches(issue, updatedCallback) { issue.full_patchsets[patch.patchset] = patch; - // TODO(wittman): Revise to reduce load on the try servers. + // TODO(wittman): Revise to reduce load on the try servers. Repeatedly + // loading old try results increases the size of the working set of try + // jobs on the try servers, causing them to become disk-bound. // patch.try_job_results.forEach(function(results) { // if (results.buildnumber) { // tryJobResultsOutstanding++; @@ -189,8 +191,11 @@ function fetchPatches(issue, updatedCallback) { // } // }); - if (++patchsetsRetrieved == issue.patchsets.length) + if (++patchsetsRetrieved == issue.patchsets.length) { updatedCallback(PATCHES_COMPLETE); + // TODO(wittman): Remove once we revise the try job fetching code. + updatedCallback(TRY_JOBS_COMPLETE); + } }); }); } diff --git a/chrome/common/extensions/docs/examples/extensions/buildbot/manifest.json b/chrome/common/extensions/docs/examples/extensions/buildbot/manifest.json index 3b0819a..1c1f5e0 100644 --- a/chrome/common/extensions/docs/examples/extensions/buildbot/manifest.json +++ b/chrome/common/extensions/docs/examples/extensions/buildbot/manifest.json @@ -1,6 +1,6 @@ { "name": "Chromium Buildbot Monitor", - "version": "0.8.3", + "version": "0.8.4", "description": "Displays the status of the Chromium buildbot in the toolbar. Click to see more detailed status in a popup.", "icons": { "128": "icon.png" }, "background": { |