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

Issue 10868053: Remove the protector pyauto tests. This feature is now disabled. When it's rewritten, new tests tha… (Closed)

Created:
8 years, 4 months ago by jam
Modified:
8 years, 4 months ago
CC:
chromium-reviews, Nirnimesh, kkania, anantha, robertshield, dyu1, dennis_jeffrey, Ivan Korotkov
Visibility:
Public.

Description

Remove the protector pyauto tests. This feature is now disabled. When it's rewritten, new tests that run on the chrome waterfall and bots will need to be written (i.e. unit_tests and browser_tests), per the requirements for all features. BUG=143637 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=153070

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5 lines) Patch
M chrome/test/functional/PYAUTO_TESTS View 1 3 chunks +0 lines, -5 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jam
8 years, 4 months ago (2012-08-23 20:30:24 UTC) #1
Nirnimesh
+cc ivankr who wrote this.
8 years, 4 months ago (2012-08-23 20:34:57 UTC) #2
Ivan Korotkov
Please don't delete protector.py or the code supporting it -- yet. The protector guts have ...
8 years, 4 months ago (2012-08-23 20:56:20 UTC) #3
jam
On 2012/08/23 20:56:20, Ivan Korotkov wrote: > Please don't delete protector.py or the code supporting ...
8 years, 4 months ago (2012-08-23 21:03:32 UTC) #4
Ben Goodger (Google)
8 years, 4 months ago (2012-08-23 21:28:44 UTC) #5
lgtm

Powered by Google App Engine
This is Rietveld 408576698