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

Issue 23536051: GTTF: Remove unused reliability tests code. (Closed)

Created:
7 years, 3 months ago by Paweł Hajdan Jr.
Modified:
7 years, 2 months ago
Reviewers:
Nico, sky
CC:
chromium-reviews
Visibility:
Public.

Description

GTTF: Remove unused reliability tests code. BUG=288383 R=sky@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224201

Patch Set 1 #

Patch Set 2 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -2552 lines) Patch
M chrome/chrome_tests.gypi View 1 6 chunks +0 lines, -148 lines 0 comments Download
D chrome/test/reliability/automated_ui_test_base.h View 1 chunk +0 lines, -142 lines 0 comments Download
D chrome/test/reliability/automated_ui_test_base.cc View 1 1 chunk +0 lines, -234 lines 0 comments Download
D chrome/test/reliability/automated_ui_tests.h View 1 1 chunk +0 lines, -362 lines 0 comments Download
D chrome/test/reliability/automated_ui_tests.cc View 1 1 chunk +0 lines, -765 lines 0 comments Download
D chrome/test/reliability/page_load_test.cc View 1 chunk +0 lines, -796 lines 0 comments Download
D chrome/tools/automated_ui_test_tools/ui_action_generator.py View 1 chunk +0 lines, -105 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
Paweł Hajdan Jr.
7 years, 3 months ago (2013-09-13 22:12:31 UTC) #1
sky
LGTM
7 years, 3 months ago (2013-09-14 15:15:03 UTC) #2
Paweł Hajdan Jr.
Committed patchset #2 manually as r224201.
7 years, 3 months ago (2013-09-19 20:58:05 UTC) #3
Nico
The valgrind relability bot builds the reliability_tests target. It's red since this CL: http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20Reliability%20(valgrind%20and%20tsan) Is ...
7 years, 3 months ago (2013-09-19 22:54:39 UTC) #4
Paweł Hajdan Jr.
On 2013/09/19 22:54:39, Nico wrote: > The valgrind relability bot builds the reliability_tests target. It's ...
7 years, 3 months ago (2013-09-19 23:00:13 UTC) #5
Timur Iskhodzhanov
On 2013/09/19 23:00:13, Paweł Hajdan Jr. wrote: > On 2013/09/19 22:54:39, Nico wrote: > > ...
7 years, 3 months ago (2013-09-22 06:35:45 UTC) #6
Paweł Hajdan Jr.
7 years, 2 months ago (2013-09-23 19:23:50 UTC) #7
Message was sent while issue was closed.
On 2013/09/22 06:35:45, Timur Iskhodzhanov wrote:
> Can we bring back the code instead?
> reliability_tests is/was a great way to find Valgrind/TSan bugs in full
Chrome.

sorry, no - you should be able to get something similar with e.g. Telemetry
(http://www.chromium.org/developers/telemetry).

Powered by Google App Engine
This is Rietveld 408576698