Roll chromium_revision 1b6c421..db567a8 (365999:366304)

Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src: https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv: https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss: https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org

Committed: https://crrev.com/2f042f26a3d0c062c43dc553058a286bd4dd8f19
Cr-Commit-Position: refs/heads/master@{#11095}

[kjellander_webrtc, 2015-12-20 19:02:44]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2015-12-20 19:02:53]

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

[commit-bot: I haz the power, 2015-12-20 19:31:20]

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

[commit-bot: I haz the power, 2015-12-20 19:31:20]

Try jobs failed on following builders:
  linux_msan on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/6831)

[kjellander_webrtc, 2015-12-20 20:05:16]

Description was changed from

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

TBR=
==========

to

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
==========

[kjellander_webrtc, 2015-12-20 20:05:16]

kjellander@webrtc.org changed reviewers:
+ pthatcher@webrtc.org, torbjorng@webrtc.org

[kjellander_webrtc, 2015-12-20 20:06:06]

Description was changed from

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
==========

to

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
==========

[kjellander_webrtc, 2015-12-20 20:21:11]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2015-12-20 20:21:19]

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

[commit-bot: I haz the power, 2015-12-20 20:25:15]

Description was changed from

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
==========

to

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
==========

[commit-bot: I haz the power, 2015-12-20 20:25:16]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-20 20:25:25]

Description was changed from

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org
==========

to

==========
Roll chromium_revision 1b6c421..db567a8 (365999:366304)

I had to disable some Dtls12Both tests failing under MSan (see bug).
Notice those errors started happening in the range of
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..9f897b2
while this CL brings in an even newer BoringSSL (that still has the same
problem).

Change log: https://chromium.googlesource.com/chromium/src/+log/1b6c421..db567a8
Full diff: https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8

Changed dependencies:
* src/third_party/boringssl/src:
https://boringssl.googlesource.com/boringssl.git/+log/afd565f..afe57cb
* src/third_party/libyuv:
https://chromium.googlesource.com/libyuv/libyuv.git/+log/1019e45..1ccbf8f
* src/third_party/nss:
https://chromium.googlesource.com/chromium/deps/nss.git/+log/a676aa0..aee1b12
DEPS diff:
https://chromium.googlesource.com/chromium/src/+/1b6c421..db567a8/DEPS

No update to Clang.

NOTRY=True
BUG=webrtc:5381
TBR=torbjorng@webrtc.org, pthatcher@webrtc.org

Committed: https://crrev.com/2f042f26a3d0c062c43dc553058a286bd4dd8f19
Cr-Commit-Position: refs/heads/master@{#11095}
==========

[commit-bot: I haz the power, 2015-12-20 20:25:26]

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

[torbjorng (webrtc), 2015-12-21 17:44:04]

It seems a bit rash to disable the entire set of tests instantiated by
INSTANTIATE_TEST_CASE_P in sslstreamadapter_unittest.cc. Why not (conditionally)
disable just TestGetSslCipherSuiteDtls12Both?

[kjellander_webrtc, 2015-12-21 18:58:20]

On 2015/12/21 17:44:04, torbjorng (webrtc) wrote:
> It seems a bit rash to disable the entire set of tests instantiated by
> INSTANTIATE_TEST_CASE_P in sslstreamadapter_unittest.cc. Why not
(conditionally)
> disable just TestGetSslCipherSuiteDtls12Both?

You're right, I should have done that. It's not a big deal since this only
affected a single bot and that I thought it was going to be fixed reasonably
soon.
Make another CL if you think it's important.

[torbjorng (webrtc), 2015-12-21 19:24:00]

I'll make the disabling more fine-grained unless I can fix this soon.

I understand the cause for the failure, and I expect to fix it before the
holidays.

[torbjorng (webrtc), 2015-12-28 15:26:21]

We re-enable tests in https://codereview.webrtc.org/1550773002.