Disable Physical Web onboarding tests

Disable Physical Web onboarding tests

These tests succeed locally but fail on the autobuilders. Temporarily
disable them until we can determine why setting the onboarding state
fails in this case.

BUG=604502
Committed: https://crrev.com/9bc94e62a536efc435ea90a4c825e8a227b030b5
Cr-Commit-Position: refs/heads/master@{#391369}

[mattreynolds, 2016-04-29 23:09:17]

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

[mattreynolds, 2016-04-29 23:09:18]

Hi Tommy, PTAL

These tests were failing on the builders and blocking the Discoverer flag CL
from landing. The changes in that CL shouldn't affect the behavior tested by
these tests, and I've verified that the tests succeed locally.

[mattreynolds, 2016-05-03 18:36:48]

Friendly ping (these tests are blocking the Discoverer flag changes)

[nyquist, 2016-05-03 20:06:58]

lgtm, but add a comment above both tests referring to the bug before you submit

https://codereview.chromium.org/1937643002/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivityTest.java
(right):

https://codereview.chromium.org/1937643002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivityTest.java:108:
@DisabledTest
Please add a comment referring to the bug both here and below.

[mattreynolds, 2016-05-03 21:18:28]

Thanks!

https://codereview.chromium.org/1937643002/diff/1/chrome/android/javatests/sr...
File
chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivityTest.java
(right):

https://codereview.chromium.org/1937643002/diff/1/chrome/android/javatests/sr...
chrome/android/javatests/src/org/chromium/chrome/browser/physicalweb/ListUrlsActivityTest.java:108:
@DisabledTest
On 2016/05/03 20:06:58, nyquist wrote:
> Please add a comment referring to the bug both here and below.

Done.

[mattreynolds, 2016-05-03 21:18:32]

The CQ bit was checked by mattreynolds@chromium.org

[mattreynolds, 2016-05-03 21:18:32]

The patchset sent to the CQ was uploaded after l-g-t-m from nyquist@chromium.org
Link to the patchset: https://codereview.chromium.org/1937643002/#ps20001
(title: "reference bug in disabled tests")

[commit-bot: I haz the power, 2016-05-03 21:19:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1937643002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1937643002/20001

[commit-bot: I haz the power, 2016-05-03 21:59:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-03 22:01:36]

Description was changed from

==========
Disable Physical Web onboarding tests

These tests succeed locally but fail on the autobuilders. Temporarily
disable them until we can determine why setting the onboarding state
fails in this case.

BUG=604502
==========

to

==========
Disable Physical Web onboarding tests

These tests succeed locally but fail on the autobuilders. Temporarily
disable them until we can determine why setting the onboarding state
fails in this case.

BUG=604502

Committed: https://crrev.com/9bc94e62a536efc435ea90a4c825e8a227b030b5
Cr-Commit-Position: refs/heads/master@{#391369}
==========

[commit-bot: I haz the power, 2016-05-03 22:01:37]

Patchset 2 (id:??) landed as
https://crrev.com/9bc94e62a536efc435ea90a4c825e8a227b030b5
Cr-Commit-Position: refs/heads/master@{#391369}