Replace deprecated Android java test annotations

Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org, mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org, msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium.org, xunjieli@chromium.org,yzshen@chromium.org,phajdan.jr@chromium.org,timvolodine@chromium.org

Committed: https://crrev.com/4500147d14161f91d3f39f1ad4ae878ea729e5c1
Cr-Commit-Position: refs/heads/master@{#440211}

[the real yoland, 2016-12-17 01:05:21]

Description was changed from

==========
Replace deprecated test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
==========

to

==========
Replace deprecated test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[the real yoland, 2016-12-17 01:06:27]

yolandyan@chromium.org changed reviewers:
+ jbudorick@chromium.org, mikecase@chromium.org

[the real yoland, 2016-12-17 01:06:31]

here comes the migrations

[jbudorick, 2016-12-17 02:15:53]

Generally fine. Main question is about Smoke.

https://codereview.chromium.org/2583933003/diff/1/android_webview/javatests/s...
File
android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/android_webview/javatests/s...
android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java:12:
import static
org.chromium.android_webview.test.util.CookieUtils.TestValueCallback;
?

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/omaha/OmahaClientTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/omaha/OmahaClientTest.java:7:
import static org.chromium.chrome.test.omaha.MockRequestGenerator.DeviceType;
?

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:9:
import android.test.suitebuilder.annotation.Smoke;
What about Smoke?

https://codereview.chromium.org/2583933003/diff/1/net/android/javatests/src/o...
File net/android/javatests/src/org/chromium/net/AndroidProxySelectorTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/net/android/javatests/src/o...
net/android/javatests/src/org/chromium/net/AndroidProxySelectorTest.java:43: if
(proxy == Proxy.NO_PROXY)
?

[the real yoland, 2016-12-17 03:31:49]

yolandyan@chromium.org changed reviewers:
- mikecase@chromium.org

[the real yoland, 2016-12-17 03:31:53]

https://codereview.chromium.org/2583933003/diff/1/android_webview/javatests/s...
File
android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/android_webview/javatests/s...
android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java:12:
import static
org.chromium.android_webview.test.util.CookieUtils.TestValueCallback;
On 2016/12/17 at 02:15:52, jbudorick wrote:
> ?

Done by the import organize, prob better this way

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:9:
import android.test.suitebuilder.annotation.Smoke;
On 2016/12/17 at 02:15:52, jbudorick wrote:
> What about Smoke?

We can leave them there for now if there are people out there doing
`out/Debug/run... -A Smoke`, lol
or we just get rid of them once for all (not supported by j4 anyway)

https://codereview.chromium.org/2583933003/diff/1/net/android/javatests/src/o...
File net/android/javatests/src/org/chromium/net/AndroidProxySelectorTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/net/android/javatests/src/o...
net/android/javatests/src/org/chromium/net/AndroidProxySelectorTest.java:43: if
(proxy == Proxy.NO_PROXY)
On 2016/12/17 at 02:15:52, jbudorick wrote:
> ?

This is presubmit error

[the real yoland, 2016-12-17 03:32:04]

yolandyan@chromium.org changed reviewers:
+ mikecase@chromium.org

[jbudorick, 2016-12-19 13:47:43]

https://codereview.chromium.org/2583933003/diff/1/android_webview/javatests/s...
File
android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/android_webview/javatests/s...
android_webview/javatests/src/org/chromium/android_webview/test/CookieManagerTest.java:12:
import static
org.chromium.android_webview.test.util.CookieUtils.TestValueCallback;
On 2016/12/17 03:31:53, the real yoland wrote:
> On 2016/12/17 at 02:15:52, jbudorick wrote:
> > ?
> 
> Done by the import organize, prob better this way

Acknowledged.

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:9:
import android.test.suitebuilder.annotation.Smoke;
On 2016/12/17 03:31:53, the real yoland wrote:
> On 2016/12/17 at 02:15:52, jbudorick wrote:
> > What about Smoke?
> 
> We can leave them there for now if there are people out there doing
> `out/Debug/run... -A Smoke`, lol
> or we just get rid of them once for all (not supported by j4 anyway)

Let's get rid of them.

https://codereview.chromium.org/2583933003/diff/1/net/android/javatests/src/o...
File net/android/javatests/src/org/chromium/net/AndroidProxySelectorTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/net/android/javatests/src/o...
net/android/javatests/src/org/chromium/net/AndroidProxySelectorTest.java:43: if
(proxy == Proxy.NO_PROXY)
On 2016/12/17 03:31:53, the real yoland wrote:
> On 2016/12/17 at 02:15:52, jbudorick wrote:
> > ?
> 
> This is presubmit error

Acknowledged.

[the real yoland, 2016-12-19 20:15:04]

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
(left):

https://codereview.chromium.org/2583933003/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:9:
import android.test.suitebuilder.annotation.Smoke;
On 2016/12/19 at 13:47:43, jbudorick wrote:
> On 2016/12/17 03:31:53, the real yoland wrote:
> > On 2016/12/17 at 02:15:52, jbudorick wrote:
> > > What about Smoke?
> > 
> > We can leave them there for now if there are people out there doing
> > `out/Debug/run... -A Smoke`, lol
> > or we just get rid of them once for all (not supported by j4 anyway)
> 
> Let's get rid of them.

Done

[jbudorick, 2016-12-19 22:59:59]

lgtm

[jbudorick, 2016-12-19 23:00:34]

On 2016/12/19 22:59:59, jbudorick wrote:
> lgtm

but please indicate that this is android-specific somewhere in the CL title.

[the real yoland, 2016-12-19 23:38:47]

On 2016/12/19 at 23:00:34, jbudorick wrote:
> On 2016/12/19 22:59:59, jbudorick wrote:
> > lgtm
> 
> but please indicate that this is android-specific somewhere in the CL title.

Done

[the real yoland, 2016-12-20 00:19:12]

yolandyan@chromium.org changed reviewers:
+ sgurun@chromium.org, tedchoc@chromium.org

[the real yoland, 2016-12-20 00:19:12]

yolandyan@chromium.org changed reviewers:
+ sgurun@chromium.org, tedchoc@chromium.org

[the real yoland, 2016-12-20 00:19:14]

Add reviewers for OWNER lgtm:
+sgurun: android_webview/*
+tedchoc: chrome/android/*
          ui/android/*
+nyquist: base/android/*
blimp/*

[the real yoland, 2016-12-20 00:19:14]

Add reviewers for OWNER lgtm:
+sgurun: android_webview/*
+tedchoc: chrome/android/*
          ui/android/*
+nyquist: base/android/*
blimp/*

[jbudorick, 2016-12-20 00:22:19]

https://codereview.chromium.org/2583933003/diff/40001/PRESUBMIT.py
File PRESUBMIT.py (right):

https://codereview.chromium.org/2583933003/diff/40001/PRESUBMIT.py#newcode1725
PRESUBMIT.py:1725: """Check that the deprecated test annotations like, "Smoke"
nit:

"""Checks that android.test.suitebuilder.annotation.* is no longer used."""

https://codereview.chromium.org/2583933003/diff/40001/PRESUBMIT.py#newcode1731
PRESUBMIT.py:1731: sources = lambda x: input_api.FilterSourceFile(x,
white_list=(r'.*\.java$',),
nit: drop all params onto the following line.

https://codereview.chromium.org/2583933003/diff/40001/PRESUBMIT.py#newcode1742
PRESUBMIT.py:1742: 'Annotations in android.test.suitebuilder.annotation is
deprecated since'
nit:

Annotations in android.test.suitebuilder.annotation have been deprecated since
API level 24. Please use android.support.test.filters  from
//third_party/android_support_test_runner:runner_java instead. Contact
yolandyan@chromium.org if you have any questions.

[the real yoland, 2016-12-20 00:36:50]

yolandyan@chromium.org changed reviewers:
+ bengr@chromium.org, boliu@chromium.org, joedow@chromium.org,
mariakhomenko@chromium.org, maxbogue@chromium.org, msarda@chromium.org,
nyquist@chromium.org, xunjieli@chromium.org, yzshen@chromium.org

[the real yoland, 2016-12-20 00:36:52]

Add reviewers for OWNER lgtm:
+sgurun: android_webview/*
+tedchoc: chrome/android/*
          ui/android/*
+nyquist: base/android/*
          blimp/*
          components/invalidation/*
+xunjieli: components/cronet/*
           net/*
+mariakhomenko: components/minidump_uploader/*
+bengr: components/precache/*
+msarda: components/signin/*
+maxbogue: components/sync/*
+boliu: content/*
+timvolodine: device/bettery/*
+yzshen: mojo/*
+joedow: remoting/android/*

[boliu, 2016-12-20 00:38:58]

you can use TBR here

https://www.chromium.org/developers/owners-files#TOC-When-to-use-To-Be-Review...

"""
For widespread or refactoring changes that are mechanical or global in nature,
it is acceptable to TBR the OWNERS required for submission, as long as your
changes have had the appropriate level of review or general approval.  (For
example, if touching hundreds of files across the codebase, a thread on the
chromium-dev mailing list may be better than getting individual approval from
dozens of different OWNERS.)
"""

[the real yoland, 2016-12-20 00:39:25]

Description was changed from

==========
Replace deprecated test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[the real yoland, 2016-12-20 00:41:04]

Description was changed from

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org
==========

[the real yoland, 2016-12-20 01:09:45]

Description was changed from

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org
==========

to

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org,phajdan.jr@chromium.org,timvolodine...
==========

[the real yoland, 2016-12-20 01:40:11]

The CQ bit was checked by yolandyan@chromium.org

[the real yoland, 2016-12-20 01:40:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/2583933003/#ps80001
(title: "Rebase again")

[the real yoland, 2016-12-20 01:40:30]

The CQ bit was unchecked by yolandyan@chromium.org

[commit-bot: I haz the power, 2016-12-20 01:40:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xunjieli, 2016-12-20 15:36:43]

components/cronet/ & net/
LGTM

[yzshen1, 2016-12-20 17:13:45]

mojo/* LGTM

[Maria, 2016-12-20 17:38:17]

lgtm % comments for components/minidump_uploader and chrome/android (since Ted
is OOO)

https://codereview.chromium.org/2583933003/diff/80001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/test/ParametersOnMultiTest.java
(right):

https://codereview.chromium.org/2583933003/diff/80001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/test/ParametersOnMultiTest.java:16:
import org.chromium.base.test.util.parameter.ParameterizedTestAnnotationTest;
why add this new import?

[the real yoland, 2016-12-20 18:45:30]

https://codereview.chromium.org/2583933003/diff/80001/chrome/android/javatest...
File
chrome/android/javatests/src/org/chromium/chrome/test/ParametersOnMultiTest.java
(right):

https://codereview.chromium.org/2583933003/diff/80001/chrome/android/javatest...
chrome/android/javatests/src/org/chromium/chrome/test/ParametersOnMultiTest.java:16:
import org.chromium.base.test.util.parameter.ParameterizedTestAnnotationTest;
On 2016/12/20 at 17:38:16, Maria wrote:
> why add this new import?

hmm, must be a bug on in eclipse (import organize), I didn't add anything, just
macro-ed eclipse to organize import for all the changed java files
deleted

[the real yoland, 2016-12-20 18:45:34]

The CQ bit was checked by yolandyan@chromium.org

[the real yoland, 2016-12-20 18:45:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
mariakhomenko@chromium.org, jbudorick@chromium.org, xunjieli@chromium.org,
yzshen@chromium.org
Link to the patchset: https://codereview.chromium.org/2583933003/#ps100001
(title: "minor fix")

[commit-bot: I haz the power, 2016-12-20 18:46:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-20 22:06:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-20 22:06:34]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[the real yoland, 2016-12-21 09:03:03]

The CQ bit was checked by yolandyan@chromium.org

[commit-bot: I haz the power, 2016-12-21 09:03:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[nyquist, 2016-12-21 09:13:37]

base/android, blimp, components/invalidations lgtm

[commit-bot: I haz the power, 2016-12-21 09:17:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 09:17:41]

Failed to apply patch for
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:
While running git apply --index -p1;
  error: patch failed:
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:54
  error:
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java:
patch does not apply

Patch:      
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
Index:
chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
diff --git
a/chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
b/chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
index
2ac8ac9c331a0bc64788528f6a038328addd72c5..4bc5a3d530f572ca09baeced4e47e14cf3ae0df9
100644
---
a/chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
+++
b/chrome/android/javatests/src/org/chromium/chrome/browser/permissions/GeolocationTest.java
@@ -4,9 +4,8 @@
 
 package org.chromium.chrome.browser.permissions;
 
-import android.test.suitebuilder.annotation.LargeTest;
-import android.test.suitebuilder.annotation.MediumTest;
-import android.test.suitebuilder.annotation.Smoke;
+import android.support.test.filters.LargeTest;
+import android.support.test.filters.MediumTest;
 
 import org.chromium.base.test.util.CommandLineFlags;
 import org.chromium.base.test.util.Feature;
@@ -54,7 +53,6 @@ public class GeolocationTest extends PermissionTestCaseBase {
      * Verify Geolocation creates an InfoBar and receives a mock location.
      * @throws Exception
      */
-    @Smoke
     @MediumTest
     @Feature({"Location", "Main"})
     public void testGeolocationPlumbingAllowedInfoBar() throws Exception {
@@ -65,7 +63,6 @@ public class GeolocationTest extends PermissionTestCaseBase {
      * Verify Geolocation creates a dialog and receives a mock location.
      * @throws Exception
      */
-    @Smoke
     @MediumTest
     @CommandLineFlags.Add("enable-features=" + MODAL_FLAG)
     @Feature({"Location", "Main"})
@@ -78,7 +75,6 @@ public class GeolocationTest extends PermissionTestCaseBase {
      * enabled and permitted to trigger without a gesture.
      * @throws Exception
      */
-    @Smoke
     @MediumTest
     @CommandLineFlags.Add({NO_GESTURE_FEATURE, FORCE_FIELDTRIAL,
FORCE_FIELDTRIAL_PARAMS})
     @Feature({"Location", "Main"})
@@ -91,7 +87,6 @@ public class GeolocationTest extends PermissionTestCaseBase {
      * no user gesture is specified.
      * @throws Exception
      */
-    @Smoke
     @MediumTest
     @CommandLineFlags.Add("enable-features=" + MODAL_FLAG)
     @Feature({"Location", "Main"})

[joedow, 2016-12-21 16:43:11]

lgtm /remoting

[the real yoland, 2016-12-21 19:49:34]

The CQ bit was checked by yolandyan@chromium.org

[the real yoland, 2016-12-21 19:49:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
mariakhomenko@chromium.org, jbudorick@chromium.org, xunjieli@chromium.org,
nyquist@chromium.org, yzshen@chromium.org, joedow@chromium.org
Link to the patchset: https://codereview.chromium.org/2583933003/#ps120001
(title: "Rebase again :(")

[commit-bot: I haz the power, 2016-12-21 19:50:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-21 21:12:48]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1482349774294400,
"parent_rev": "f9ce870e1d5f183697e29e027a3fec5ccbe2ef8f", "commit_rev":
"2b873cc59e6d28e19109fff56c248f99b1782ce2"}

[commit-bot: I haz the power, 2016-12-21 21:13:22]

Description was changed from

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org,phajdan.jr@chromium.org,timvolodine...
==========

to

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org,phajdan.jr@chromium.org,timvolodine...

Review-Url: https://codereview.chromium.org/2583933003
==========

[commit-bot: I haz the power, 2016-12-21 21:13:27]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-12-21 21:16:39]

Description was changed from

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org,phajdan.jr@chromium.org,timvolodine...

Review-Url: https://codereview.chromium.org/2583933003
==========

to

==========
Replace deprecated Android java test annotations

Previous SmallTest, MediumTest, and LargeTest annotations have been
deprecated since level 24, since our test runner only uses these
annotations' names, instead of package names, to identify tests,
We can first replace these annotations

BUG=673824, 640116
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=bengr@chromium.org,boliu@chromium.org,joedow@chromium.org,
mariakhomenko@chromium.org,maxbogue@chromium.org, mikecase@chromium.org,
msarda@chromium.org,nyquist@chromium.org,sgurun@chromium.org,tedchoc@chromium...,
xunjieli@chromium.org,yzshen@chromium.org,phajdan.jr@chromium.org,timvolodine...

Committed: https://crrev.com/4500147d14161f91d3f39f1ad4ae878ea729e5c1
Cr-Commit-Position: refs/heads/master@{#440211}
==========

[commit-bot: I haz the power, 2016-12-21 21:16:44]

Patchset 7 (id:??) landed as
https://crrev.com/4500147d14161f91d3f39f1ad4ae878ea729e5c1
Cr-Commit-Position: refs/heads/master@{#440211}