Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1335)

Issue 9513008: Convert the XXX_sync builders to using directly sync_integration_tests. (Closed)

Created:
8 years, 9 months ago by M-A Ruel
Modified:
8 years, 9 months ago
CC:
chromium-reviews, ncarter (slow), akalin, Raghu Simha, cbentzel+watch_chromium.org, darin-cc_chromium.org, brettw-cc_chromium.org, tim (not reviewing)
Visibility:
Public.

Description

Convert the XXX_sync builders to using directly sync_integration_tests. TBR=nsylvain@chromium.org BUG=109517 TEST=sync_integration_tests is properly run. NOTRY=true Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=124732

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -5 lines) Patch
M base/PRESUBMIT.py View 1 chunk +5 lines, -1 line 0 comments Download
M chrome/browser/sync/PRESUBMIT.py View 1 chunk +5 lines, -1 line 0 comments Download
M chrome/common/net/PRESUBMIT.py View 1 chunk +5 lines, -1 line 0 comments Download
M jingle/PRESUBMIT.py View 1 chunk +5 lines, -1 line 0 comments Download
M net/PRESUBMIT.py View 1 chunk +5 lines, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
M-A Ruel
8 years, 9 months ago (2012-02-28 19:53:17 UTC) #1
nsylvain
will that build only sync_integration_tests ? Or will build the whole solution and then run ...
8 years, 9 months ago (2012-02-28 21:57:47 UTC) #2
M-A Ruel
On 2012/02/28 21:57:47, nsylvain wrote: > will that build only sync_integration_tests ? Or will build ...
8 years, 9 months ago (2012-03-01 16:14:47 UTC) #3
nsylvain
On 2012/03/01 16:14:47, Marc-Antoine Ruel wrote: > On 2012/02/28 21:57:47, nsylvain wrote: > > will ...
8 years, 9 months ago (2012-03-01 20:53:05 UTC) #4
M-A Ruel
On 2012/03/01 20:53:05, nsylvain wrote: > On 2012/03/01 16:14:47, Marc-Antoine Ruel wrote: > > On ...
8 years, 9 months ago (2012-03-01 20:56:11 UTC) #5
nsylvain
lgtm smart try server
8 years, 9 months ago (2012-03-02 20:03:38 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/9513008/1
8 years, 9 months ago (2012-03-02 21:16:45 UTC) #7
commit-bot: I haz the power
8 years, 9 months ago (2012-03-02 21:16:59 UTC) #8
Change committed as 124732

Powered by Google App Engine
This is Rietveld 408576698