Disable CanSetupAudioAndVideoCallWithoutMsidAndBundle.

Disable CanSetupAudioAndVideoCallWithoutMsidAndBundle.

TBR=perkj@chromium.org
BUG=235089
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196186

[gab, 2013-04-24 17:55:04]

https://codereview.chromium.org/14336011/diff/1/content/browser/media/webrtc_...
File content/browser/media/webrtc_browsertest.cc (right):

https://codereview.chromium.org/14336011/diff/1/content/browser/media/webrtc_...
content/browser/media/webrtc_browsertest.cc:120:
DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle) {
Only needs to be disable on Win7Aura AFAIK.
i.e. Put

#if defined(OS_WIN) && defined(USE_AURA)
#define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle
CanSetupAudioAndVideoCallWithoutMsidAndBundle
#else
#define DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle
#endif

above, and use MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle here.

[gab, 2013-04-24 17:57:04]

Oh and I think disabled tests should be P1 (I think) and have the appropriate
labels to see related people.

[not at google - send to devlin, 2013-04-24 17:58:11]

Sure, I just don't know what label it should be. But i'll mark it p1.

https://codereview.chromium.org/14336011/diff/1/content/browser/media/webrtc_...
File content/browser/media/webrtc_browsertest.cc (right):

https://codereview.chromium.org/14336011/diff/1/content/browser/media/webrtc_...
content/browser/media/webrtc_browsertest.cc:120:
DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle) {
On 2013/04/24 17:55:04, gab wrote:
> Only needs to be disable on Win7Aura AFAIK.
> i.e. Put
> 
> #if defined(OS_WIN) && defined(USE_AURA)
> #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle
> CanSetupAudioAndVideoCallWithoutMsidAndBundle
> #else
> #define DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle
> #endif
> 
> above, and use MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle here.

Done.

[commit-bot: I haz the power, 2013-04-24 17:59:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kalman@chromium.org/14336011/5001

[gab, 2013-04-24 18:03:57]

LGTM, for the labels I just searched for "Media" and "WebRTC" labels and used
those, better to get closer to the people who may know the right people than
nothing :).

[not at google - send to devlin, 2013-04-24 18:24:21]

Committed patchset #2 manually as r196186.