net/ lint suppressions needed for Android SDK roll.

net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting them up into separate CLs to make OWNERS review easier.

BUG=623989
Committed: https://crrev.com/748a60e398d3b38dfc372d56437d3419b230956d
Cr-Commit-Position: refs/heads/master@{#422000}

[ghost stip (do not use), 2016-09-26 03:54:17]

Description was changed from

==========
net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

to

==========
net/test/android lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

[ghost stip (do not use), 2016-09-26 03:56:35]

Description was changed from

==========
net/test/android lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

to

==========
net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

[ghost stip (do not use), 2016-09-26 03:56:51]

stip@chromium.org changed reviewers:
+ jbudorick@chromium.org, rsleevi@chromium.org

[ghost stip (do not use), 2016-09-26 03:57:02]

jbudorick: ptal
rsleevi: OWNERS

[jbudorick, 2016-09-26 13:02:03]

lgtm w/ nit

https://codereview.chromium.org/2367273002/diff/1/net/android/java/src/org/ch...
File net/android/java/src/org/chromium/net/NetworkChangeNotifierAutoDetect.java
(right):

https://codereview.chromium.org/2367273002/diff/1/net/android/java/src/org/ch...
net/android/java/src/org/chromium/net/NetworkChangeNotifierAutoDetect.java:913:
"NewApi", "ParcelCreator",
nit: can this go on one line?

[Ryan Sleevi, 2016-09-26 15:00:30]

LGTM w/ Q

https://codereview.chromium.org/2367273002/diff/1/net/test/android/javatests/...
File
net/test/android/javatests/src/org/chromium/net/test/util/TestWebServer.java
(right):

https://codereview.chromium.org/2367273002/diff/1/net/test/android/javatests/...
net/test/android/javatests/src/org/chromium/net/test/util/TestWebServer.java:402:
@SuppressLint("TrustAllX509TrustManager")
What's this lint actually about?

[ghost stip (do not use), 2016-09-27 15:47:14]

https://codereview.chromium.org/2367273002/diff/1/net/test/android/javatests/...
File
net/test/android/javatests/src/org/chromium/net/test/util/TestWebServer.java
(right):

https://codereview.chromium.org/2367273002/diff/1/net/test/android/javatests/...
net/test/android/javatests/src/org/chromium/net/test/util/TestWebServer.java:402:
@SuppressLint("TrustAllX509TrustManager")
On 2016/09/26 15:00:30, Ryan Sleevi (slow) wrote:
> What's this lint actually about?

I believe it's to make sure that people are actually checking certs instead of
just blindly accepting. Since this is a test, that doesn't necessarily apply
here.

[ghost stip (do not use), 2016-09-30 00:07:01]

The CQ bit was checked by stip@chromium.org

[ghost stip (do not use), 2016-09-30 00:07:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org, rsleevi@chromium.org
Link to the patchset: https://codereview.chromium.org/2367273002/#ps20001
(title: "Address jbudorick's comments, add TODO.")

[commit-bot: I haz the power, 2016-09-30 00:07:58]

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-09-30 00:54:44]

Description was changed from

==========
net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

to

==========
net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

[commit-bot: I haz the power, 2016-09-30 00:54:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-30 00:56:43]

Description was changed from

==========
net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989
==========

to

==========
net/ lint suppressions needed for Android SDK roll.

This is only part of the suppressions needed for a full compile, I'm splitting
them up into separate CLs to make OWNERS review easier.

BUG=623989

Committed: https://crrev.com/748a60e398d3b38dfc372d56437d3419b230956d
Cr-Commit-Position: refs/heads/master@{#422000}
==========

[commit-bot: I haz the power, 2016-09-30 00:56:44]

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