Move PwsResultTest to junit

Move PwsResultTest to junit

PwsResultTest does not need to be an instrumentation test, so this change
moves it from javatests/ to junit/.

BUG=696804
Review-Url: https://codereview.chromium.org/2722623003
Cr-Commit-Position: refs/heads/master@{#454363}
Committed: https://chromium.googlesource.com/chromium/src/+/104f81845c1906da368d74c20e4b161a348c0567

[cco3, 2017-02-28 01:53:00]

cco3@chromium.org changed reviewers:
+ mattreynolds@chromium.org

[cco3, 2017-02-28 01:53:33]

Description was changed from

==========
Move PwsResultTest to junit

UrlInfoTest does not need to be an instrumentation test, so this change
moves it from javatests/ to junit/.

BUG=696804
==========

to

==========
Move PwsResultTest to junit

PwsResultTest does not need to be an instrumentation test, so this change
moves it from javatests/ to junit/.

BUG=696804
==========

[mattreynolds, 2017-02-28 02:12:48]

lgtm

[cco3, 2017-02-28 17:12:42]

cco3@chromium.org changed reviewers:
+ nyquist@chromium.org

[cco3, 2017-02-28 17:12:43]

Hi Tommy, here's another one.

[nyquist, 2017-02-28 18:27:22]

https://codereview.chromium.org/2722623003/diff/20001/chrome/android/java_sou...
File chrome/android/java_sources.gni (right):

https://codereview.chromium.org/2722623003/diff/20001/chrome/android/java_sou...
chrome/android/java_sources.gni:1593:
"junit/src/org/chromium/chrome/browser/physicalweb/PwsResultTest.java",
Shouldn't it also be removed from the other place?

[cco3, 2017-02-28 18:28:39]

https://codereview.chromium.org/2722623003/diff/20001/chrome/android/java_sou...
File chrome/android/java_sources.gni (right):

https://codereview.chromium.org/2722623003/diff/20001/chrome/android/java_sou...
chrome/android/java_sources.gni:1593:
"junit/src/org/chromium/chrome/browser/physicalweb/PwsResultTest.java",
On 2017/02/28 18:27:22, nyquist wrote:
> Shouldn't it also be removed from the other place?

It wasn't in the other place =(

Hence the need for the few actual logic changes in the test (since it's actually
running now).

[nyquist, 2017-02-28 22:43:20]

Ah. It all makes sense now! :-)
lgtm

[cco3, 2017-02-28 22:59:44]

The CQ bit was checked by cco3@chromium.org

[commit-bot: I haz the power, 2017-02-28 23:00:39]

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, 2017-02-28 23:04:02]

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

[commit-bot: I haz the power, 2017-02-28 23:04:03]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[cco3, 2017-03-01 00:33:42]

The CQ bit was checked by cco3@chromium.org

[cco3, 2017-03-01 00:33:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
nyquist@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2722623003/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2017-03-01 00:34:18]

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, 2017-03-01 02:20:30]

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

[commit-bot: I haz the power, 2017-03-01 02:20:30]

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_...)

[cco3, 2017-03-02 18:45:13]

The CQ bit was checked by cco3@chromium.org

[cco3, 2017-03-02 18:45:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
nyquist@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2722623003/#ps60001
(title: "Rebase")

[commit-bot: I haz the power, 2017-03-02 18:45:42]

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, 2017-03-02 20:29:53]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1488480313385300,
"parent_rev": "612c5c49898c0766cd07f65f95bc930e98210ef6", "commit_rev":
"104f81845c1906da368d74c20e4b161a348c0567"}

[commit-bot: I haz the power, 2017-03-02 20:30:38]

Description was changed from

==========
Move PwsResultTest to junit

PwsResultTest does not need to be an instrumentation test, so this change
moves it from javatests/ to junit/.

BUG=696804
==========

to

==========
Move PwsResultTest to junit

PwsResultTest does not need to be an instrumentation test, so this change
moves it from javatests/ to junit/.

BUG=696804

Review-Url: https://codereview.chromium.org/2722623003
Cr-Commit-Position: refs/heads/master@{#454363}
Committed:
https://chromium.googlesource.com/chromium/src/+/104f81845c1906da368d74c20e4b...
==========

[commit-bot: I haz the power, 2017-03-02 20:30:39]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/104f81845c1906da368d74c20e4b...