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

Issue 10382189: Remove the last UITest in interactive_ui_tests which was disabled and is not necessary now since th… (Closed)

Created:
8 years, 7 months ago by jam
Modified:
8 years, 7 months ago
Reviewers:
David Tseng
CC:
chromium-reviews, hashimoto+watch_chromium.org, aboxhall+watch_chromium.org, yoshiki+watch_chromium.org, yuzo+watch_chromium.org, davidbarr+watch_chromium.org, dmazzoni+watch_chromium.org, dtseng+watch_chromium.org, ctguil+watch_chromium.org, zork+watch_chromium.org
Visibility:
Public.

Description

Remove the last UITest in interactive_ui_tests which was disabled and is not necessary now since there are cross-platform tests. Remove the dependency on UITest code to prevent new UITests creeping in. Also move ConstrainedWindowViewTest and InspectUITest tests to browser_tests since they don't need to be interactive. BUG=121574, 89583 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=137435

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : InspectUITest is still flaky #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -204 lines) Patch
D chrome/browser/accessibility/accessibility_mac_uitest.mm View 1 chunk +0 lines, -198 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 2 9 chunks +6 lines, -6 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
jam
8 years, 7 months ago (2012-05-16 05:48:32 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698