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

Issue 23572019: [Android] Add HostDrivenTest java annotation. (Closed)

Created:
7 years, 3 months ago by frankf
Modified:
7 years, 3 months ago
Reviewers:
craigdh, nyquist, bulach
CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, erikwright+watch_chromium.org, frankf+watch_chromium.org, nyquist
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -7 lines) Patch
A + base/test/android/javatests/src/org/chromium/base/test/util/HostDrivenTest.java View 1 1 chunk +5 lines, -4 lines 0 comments Download
M build/android/pylib/instrumentation/test_jar.py View 1 chunk +1 line, -1 line 0 comments Download
M chrome/android/javatests/src/org/chromium/chrome/browser/test/DummyTest.java View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
frankf
7 years, 3 months ago (2013-08-30 21:28:14 UTC) #1
craigdh
lgtm
7 years, 3 months ago (2013-08-30 21:53:41 UTC) #2
nyquist
lgtm
7 years, 3 months ago (2013-08-30 22:30:37 UTC) #3
frankf
+bulach for OWNERS
7 years, 3 months ago (2013-08-31 03:04:47 UTC) #4
nyquist
If this CL lands after https://codereview.chromium.org/22914014/ please also update chrome/android/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java. I will naturally update that ...
7 years, 3 months ago (2013-08-31 04:32:53 UTC) #5
bulach
lgtm, thanks!
7 years, 3 months ago (2013-09-03 15:34:34 UTC) #6
frankf
7 years, 3 months ago (2013-09-03 17:11:33 UTC) #7
Message was sent while issue was closed.
Committed patchset #2 manually as r220962 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698