Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(308)

Side by Side Diff: content/browser/media/webrtc_browsertest.cc

Issue 137403005: Remove some code inside USE_AURA and OS_WIN ifdefs. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/debug/trace_event_impl.h" 6 #include "base/debug/trace_event_impl.h"
7 #include "base/json/json_reader.h" 7 #include "base/json/json_reader.h"
8 #include "base/strings/stringprintf.h" 8 #include "base/strings/stringprintf.h"
9 #include "base/strings/utf_string_conversions.h" 9 #include "base/strings/utf_string_conversions.h"
10 #include "base/values.h" 10 #include "base/values.h"
(...skipping 467 matching lines...) Expand 10 before | Expand all | Expand 10 after
478 IN_PROC_BROWSER_TEST_F(WebrtcBrowserTest, MAYBE_CanForwardRemoteStream720p) { 478 IN_PROC_BROWSER_TEST_F(WebrtcBrowserTest, MAYBE_CanForwardRemoteStream720p) {
479 const std::string javascript = GenerateGetUserMediaCall( 479 const std::string javascript = GenerateGetUserMediaCall(
480 "callAndForwardRemoteStream", 1280, 1280, 720, 720, 30, 30); 480 "callAndForwardRemoteStream", 1280, 1280, 720, 720, 30, 30);
481 MakeTypicalPeerConnectionCall(javascript); 481 MakeTypicalPeerConnectionCall(javascript);
482 } 482 }
483 483
484 // This test will make a complete PeerConnection-based call but remove the 484 // This test will make a complete PeerConnection-based call but remove the
485 // MSID and bundle attribute from the initial offer to verify that 485 // MSID and bundle attribute from the initial offer to verify that
486 // video is playing for the call even if the initiating client don't support 486 // video is playing for the call even if the initiating client don't support
487 // MSID. http://tools.ietf.org/html/draft-alvestrand-rtcweb-msid-02 487 // MSID. http://tools.ietf.org/html/draft-alvestrand-rtcweb-msid-02
488 #if defined(OS_WIN) && defined(USE_AURA) 488 #if defined(OS_WIN)
489 // Disabled for win7_aura, see http://crbug.com/235089. 489 // Disabled for win, see http://crbug.com/235089.
490 #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle\ 490 #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle\
491 DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle 491 DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle
492 #elif defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) 492 #elif defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY)
493 // Timing out on ARM linux, see http://crbug.com/240373 493 // Timing out on ARM linux, see http://crbug.com/240373
494 #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle\ 494 #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle\
495 DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle 495 DISABLED_CanSetupAudioAndVideoCallWithoutMsidAndBundle
496 #else 496 #else
497 #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle\ 497 #define MAYBE_CanSetupAudioAndVideoCallWithoutMsidAndBundle\
498 CanSetupAudioAndVideoCallWithoutMsidAndBundle 498 CanSetupAudioAndVideoCallWithoutMsidAndBundle
499 #endif 499 #endif
(...skipping 15 matching lines...) Expand all
515 } 515 }
516 516
517 // This test can negotiate an SDP offer that includes a b=AS:xx to control 517 // This test can negotiate an SDP offer that includes a b=AS:xx to control
518 // the bandwidth for audio and video 518 // the bandwidth for audio and video
519 IN_PROC_BROWSER_TEST_F(WebrtcBrowserTest, NegotiateOfferWithBLine) { 519 IN_PROC_BROWSER_TEST_F(WebrtcBrowserTest, NegotiateOfferWithBLine) {
520 MakeTypicalPeerConnectionCall("negotiateOfferWithBLine();"); 520 MakeTypicalPeerConnectionCall("negotiateOfferWithBLine();");
521 } 521 }
522 522
523 // This test will make a complete PeerConnection-based call using legacy SDP 523 // This test will make a complete PeerConnection-based call using legacy SDP
524 // settings: GIce, external SDES, and no BUNDLE. 524 // settings: GIce, external SDES, and no BUNDLE.
525 #if defined(OS_WIN) && defined(USE_AURA) 525 #if defined(OS_WIN)
526 // Disabled for win7_aura, see http://crbug.com/235089. 526 // Disabled for win7, see http://crbug.com/235089.
527 #define MAYBE_CanSetupLegacyCall DISABLED_CanSetupLegacyCall 527 #define MAYBE_CanSetupLegacyCall DISABLED_CanSetupLegacyCall
528 #elif defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) 528 #elif defined(OS_LINUX) && !defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY)
529 // Timing out on ARM linux, see http://crbug.com/240373 529 // Timing out on ARM linux, see http://crbug.com/240373
530 #define MAYBE_CanSetupLegacyCall DISABLED_CanSetupLegacyCall 530 #define MAYBE_CanSetupLegacyCall DISABLED_CanSetupLegacyCall
531 #else 531 #else
532 #define MAYBE_CanSetupLegacyCall CanSetupLegacyCall 532 #define MAYBE_CanSetupLegacyCall CanSetupLegacyCall
533 #endif 533 #endif
534 534
535 IN_PROC_BROWSER_TEST_F(WebrtcBrowserTest, MAYBE_CanSetupLegacyCall) { 535 IN_PROC_BROWSER_TEST_F(WebrtcBrowserTest, MAYBE_CanSetupLegacyCall) {
536 MakeTypicalPeerConnectionCall("callWithLegacySdp();"); 536 MakeTypicalPeerConnectionCall("callWithLegacySdp();");
(...skipping 248 matching lines...) Expand 10 before | Expand all | Expand 10 after
785 EXPECT_TRUE(base::PathExists(dump_file)); 785 EXPECT_TRUE(base::PathExists(dump_file));
786 int64 file_size = 0; 786 int64 file_size = 0;
787 EXPECT_TRUE(base::GetFileSize(dump_file, &file_size)); 787 EXPECT_TRUE(base::GetFileSize(dump_file, &file_size));
788 EXPECT_EQ(0, file_size); 788 EXPECT_EQ(0, file_size);
789 789
790 base::DeleteFile(dump_file, false); 790 base::DeleteFile(dump_file, false);
791 } 791 }
792 792
793 793
794 } // namespace content 794 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698