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

Unified Diff: chrome/browser/audio_layout_uitest.cc

Issue 9959028: Convert the media and audio UI test to a browser_test. browser_tests are sharded and run quicker, a… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/media_uitest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/audio_layout_uitest.cc
===================================================================
--- chrome/browser/audio_layout_uitest.cc (revision 129871)
+++ chrome/browser/audio_layout_uitest.cc (working copy)
@@ -1,86 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "base/file_path.h"
-#include "chrome/test/ui/ui_layout_test.h"
-
-namespace {
- const char* kResources[] = {
- "content",
- "media-file.js",
- "video-test.js",
- };
-} // anonymous namespace
-
-class AudioUILayoutTest : public UILayoutTest {
- protected:
- virtual ~AudioUILayoutTest() { }
-
- void RunMediaLayoutTest(const std::string& test_case_file_name) {
- FilePath test_dir;
- FilePath media_test_dir;
- media_test_dir = media_test_dir.AppendASCII("media");
- InitializeForLayoutTest(test_dir, media_test_dir, kNoHttpPort);
-
- // Copy resources first.
- for (size_t i = 0; i < arraysize(kResources); ++i) {
- AddResourceForLayoutTest(
- test_dir, media_test_dir.AppendASCII(kResources[i]));
- }
-
- printf("Test: %s\n", test_case_file_name.c_str());
- RunLayoutTest(test_case_file_name, kNoHttpPort);
- }
-};
-
-
-TEST_F(AudioUILayoutTest, AudioConstructorPreload) {
- RunMediaLayoutTest("audio-constructor-preload.html");
-}
-
-TEST_F(AudioUILayoutTest, AudioConstructor) {
- RunMediaLayoutTest("audio-constructor.html");
-}
-
-TEST_F(AudioUILayoutTest, AudioConstructorSrc) {
- RunMediaLayoutTest("audio-constructor-src.html");
-}
-
-TEST_F(AudioUILayoutTest, AudioDataUrl) {
- RunMediaLayoutTest("audio-data-url.html");
-}
-
-// The test fails since there is no real audio device on the build bots to get
-// the ended event fired. Should pass once we run it on bots with audio devices.
-TEST_F(AudioUILayoutTest, DISABLED_AudioGarbageCollect) {
- RunMediaLayoutTest("audio-garbage-collect.html");
-}
-
-TEST_F(AudioUILayoutTest, AudioNoInstalledEngines) {
- RunMediaLayoutTest("audio-no-installed-engines.html");
-}
-
-#if defined(OS_CHROMEOS) && defined(USE_AURA)
-// http://crbug.com/115530
-#define MAYBE_AudioOnlyVideoIntrinsicSize DISABLED_AudioOnlyVideoIntrinsicSize
-#else
-#define MAYBE_AudioOnlyVideoIntrinsicSize AudioOnlyVideoIntrinsicSize
-#endif
-
-TEST_F(AudioUILayoutTest, MAYBE_AudioOnlyVideoIntrinsicSize) {
- RunMediaLayoutTest("audio-only-video-intrinsic-size.html");
-}
-
-TEST_F(AudioUILayoutTest, AudioPlayEvent) {
- RunMediaLayoutTest("audio-play-event.html");
-}
-
-TEST_F(AudioUILayoutTest, MediaCanPlayWavAudio) {
- RunMediaLayoutTest("media-can-play-wav-audio.html");
-}
-
-TEST_F(AudioUILayoutTest, MediaDocumentAudioSize) {
- RunMediaLayoutTest("media-document-audio-size.html");
-}
-
« no previous file with comments | « no previous file | chrome/browser/media_uitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698