(Reland) Update all Robolectric tests to Robolectric 3.0

(Reland) Update all Robolectric tests to Robolectric 3.0

BUG=522043
Committed: https://crrev.com/cf5bc93dae70d15f712f45daf2fb2d696d8241ad
Cr-Commit-Position: refs/heads/master@{#412564}

[mikecase (-- gone --), 2016-08-15 18:05:50]

mikecase@chromium.org changed reviewers:
+ caitkp@chromium.org, cbentzel@chromium.org, jbudorick@chromium.org,
tedchoc@chromium.org, thestig@chromium.org

[mikecase (-- gone --), 2016-08-15 18:05:52]

Had to revert CL because in between me doing a CQ-dry-run and committing the CL,
a new junit test was added which caused compile errors. Won't make this mistake
again.

Link to previous CL: https://codereview.chromium.org/2195143002
Link to revert: https://codereview.chromium.org/2239003002

The only changes between this and previous CL are...

- Fixes to the compile errors for the new junit test
chrome/android/junit/src/org/chromium/chrome/browser/download/DownloadSharedPreferenceEntryTest.java

- Fixes to some deprecation warnings for using deprecated ShadowHandler class.
base/android/junit/src/org/chromium/base/PromiseTest.java

Would appreciate if everyone could restamp CL. ty

[jbudorick, 2016-08-15 18:10:10]

On 2016/08/15 18:05:52, mikecase wrote:
> Had to revert CL because in between me doing a CQ-dry-run and committing the
CL,
> a new junit test was added which caused compile errors. Won't make this
mistake
> again.
> 
> Link to previous CL: https://codereview.chromium.org/2195143002
> Link to revert: https://codereview.chromium.org/2239003002
> 
> The only changes between this and previous CL are...
> 
> - Fixes to the compile errors for the new junit test
>
chrome/android/junit/src/org/chromium/chrome/browser/download/DownloadSharedPreferenceEntryTest.java
> 
> - Fixes to some deprecation warnings for using deprecated ShadowHandler class.
> base/android/junit/src/org/chromium/base/PromiseTest.java

Next time you do a reland, can you upload the original as patchset 1 and the
fixed version as patchset 2? It makes the changes much more obvious.

> 
> Would appreciate if everyone could restamp CL. ty

lgtm

[Lei Zhang, 2016-08-15 18:33:21]

rs lgtm

[Cait (Slow), 2016-08-15 18:47:59]

rs lgtm for components/

[Ted C, 2016-08-15 20:38:13]

On 2016/08/15 18:47:59, Cait (slow) wrote:
> rs lgtm for components/

rs lgtm

[cbentzel, 2016-08-15 22:10:26]

On 2016/08/15 20:38:13, Ted C wrote:
> On 2016/08/15 18:47:59, Cait (slow) wrote:
> > rs lgtm for components/
> 
> rs lgtm

rs lgtm for net/

[mikecase (-- gone --), 2016-08-17 16:01:23]

The CQ bit was checked by mikecase@chromium.org

[commit-bot: I haz the power, 2016-08-17 16:01:59]

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-08-17 16:53:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-17 16:55:35]

Description was changed from

==========
(Reland) Update all Robolectric tests to Robolectric 3.0

BUG=522043
==========

to

==========
(Reland) Update all Robolectric tests to Robolectric 3.0

BUG=522043

Committed: https://crrev.com/cf5bc93dae70d15f712f45daf2fb2d696d8241ad
Cr-Commit-Position: refs/heads/master@{#412564}
==========

[commit-bot: I haz the power, 2016-08-17 16:55:37]

Patchset 1 (id:??) landed as
https://crrev.com/cf5bc93dae70d15f712f45daf2fb2d696d8241ad
Cr-Commit-Position: refs/heads/master@{#412564}