Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
R=kjellander@webrtc.org
TBR=stefan

Review-Url: https://codereview.webrtc.org/2814313002 .
Cr-Commit-Position: refs/heads/master@{#17701}
Committed: https://chromium.googlesource.com/external/webrtc/+/fb9e7512cba849ec6c03ffac2b5768a678c0eed9

[lliuu, 2017-04-12 23:24:29]

The CQ bit was checked by lliuu@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-12 23:24:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[lliuu, 2017-04-12 23:25:03]

Description was changed from

==========
Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
==========

to

==========
Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
==========

[lliuu, 2017-04-12 23:25:04]

lliuu@webrtc.org changed reviewers:
+ kjellander@webrtc.org

[commit-bot: I haz the power, 2017-04-12 23:46:05]

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

[commit-bot: I haz the power, 2017-04-12 23:46:06]

Dry run: This issue passed the CQ dry run.

[kjellander_webrtc, 2017-04-13 20:18:54]

lgtm with my comment addressed.

You can TBR any owner for a change like this, no need to wait for approval for
such a simple change when it's just about disabling a test.

https://codereview.webrtc.org/2814313002/diff/1/webrtc/video/end_to_end_tests.cc
File webrtc/video/end_to_end_tests.cc (right):

https://codereview.webrtc.org/2814313002/diff/1/webrtc/video/end_to_end_tests...
webrtc/video/end_to_end_tests.cc:2696: #if defined(WEBRTC_WIN)
Please add a comment like this:
// Disabled due to flakiness on Windows (bugs.webrtc.org/7483).

[lliuu, 2017-04-13 20:57:59]

On 2017/04/13 20:18:54, kjellander_webrtc wrote:
> lgtm with my comment addressed.
> 
> You can TBR any owner for a change like this, no need to wait for approval for
> such a simple change when it's just about disabling a test.
> 
>
https://codereview.webrtc.org/2814313002/diff/1/webrtc/video/end_to_end_tests.cc
> File webrtc/video/end_to_end_tests.cc (right):
> 
>
https://codereview.webrtc.org/2814313002/diff/1/webrtc/video/end_to_end_tests...
> webrtc/video/end_to_end_tests.cc:2696: #if defined(WEBRTC_WIN)
> Please add a comment like this:
> // Disabled due to flakiness on Windows (bugs.webrtc.org/7483).

Thanks! will do that in the future

[lliuu, 2017-04-13 21:07:33]

Description was changed from

==========
Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
==========

to

==========
Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
TBR=stefan
==========

[lliuu, 2017-04-13 21:08:27]

The CQ bit was checked by lliuu@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-13 21:08:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[lliuu, 2017-04-13 21:10:01]

Description was changed from

==========
Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
TBR=stefan
==========

to

==========
Disable EndToEndTest.ContentTypeSwitches test on Windows due to flakiness

BUG=webrtc:7483
R=kjellander@webrtc.org
TBR=stefan

Review-Url: https://codereview.webrtc.org/2814313002 .
Cr-Commit-Position: refs/heads/master@{#17701}
Committed:
https://chromium.googlesource.com/external/webrtc/+/fb9e7512cba849ec6c03ffac2...
==========

[lliuu, 2017-04-13 21:10:02]

Committed patchset #3 (id:40001) manually as
fb9e7512cba849ec6c03ffac2b5768a678c0eed9 (presubmit successful).