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

Side by Side Diff: content/browser/media/session/media_session_visibility_browsertest_instances.cc

Issue 1819113002: [MediaSession, Android] Adding visibility tests for MediaSession (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use default timeouts Created 4 years, 8 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
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "base/command_line.h"
6 #include "content/browser/media/session/media_session_visibility_browsertest_bas e.h"
7 #include "media/base/media_switches.h"
8
9 using content::MediaSessionVisibilityBrowserTestBase;
10
11 // UnifiedPipeline + SuspendOnHide
12 class MediaSessionVisibilityBrowserTest_UnifiedPipeline_SuspendOnHide :
13 public MediaSessionVisibilityBrowserTestBase {
14 void SetUpCommandLine(base::CommandLine* command_line) override {
15 MediaSessionVisibilityBrowserTestBase::SetUpCommandLine(command_line);
16 #if defined(OS_ANDROID)
17 command_line->AppendSwitch(switches::kEnableUnifiedMediaPipeline);
18 #else // defined(OS_ANDROID)
19 command_line->AppendSwitch(switches::kEnableMediaSuspend);
20 #endif // defined(OS_ANDROID)
21 }
22 };
23
24 INCLUDE_TEST_FROM_BASE_CLASS(
25 MediaSessionVisibilityBrowserTest_UnifiedPipeline_SuspendOnHide,
26 TestSessionInactiveWhenHiddenAfterContentPause)
27 INCLUDE_TEST_FROM_BASE_CLASS(
28 MediaSessionVisibilityBrowserTest_UnifiedPipeline_SuspendOnHide,
29 TestSessionInactiveWhenHiddenWhilePlaying)
30
31 // UnifiedPipeline + NosuspendOnHide
32 class MediaSessionVisibilityBrowserTest_UnifiedPipeline_NosuspendOnHide :
33 public MediaSessionVisibilityBrowserTestBase {
34 void SetUpCommandLine(base::CommandLine* command_line) override {
35 MediaSessionVisibilityBrowserTestBase::SetUpCommandLine(command_line);
36 #if defined(OS_ANDROID)
37 command_line->AppendSwitch(switches::kEnableUnifiedMediaPipeline);
38 command_line->AppendSwitch(switches::kDisableMediaSuspend);
39 #endif // defined(OS_ANDROID)
40 }
41 };
42
43 INCLUDE_TEST_FROM_BASE_CLASS(
44 MediaSessionVisibilityBrowserTest_UnifiedPipeline_NosuspendOnHide,
45 TestSessionSuspendedWhenHiddenAfterContentPause)
46 INCLUDE_TEST_FROM_BASE_CLASS(
47 MediaSessionVisibilityBrowserTest_UnifiedPipeline_NosuspendOnHide,
48 TestSessionActiveWhenHiddenWhilePlaying)
49
50 #if defined(OS_ANDROID)
51 // AndroidPipeline + SuspendOnHide
52 class MediaSessionVisibilityBrowserTest_AndroidPipeline_SuspendOnHide :
53 public MediaSessionVisibilityBrowserTestBase {
mlamouri (slow - plz ping) 2016/03/31 10:27:53 I think you might need to use a kDisableUnifiedMed
Zhiqiang Zhang (Slow) 2016/03/31 13:00:19 Done.
54 };
55
56 INCLUDE_TEST_FROM_BASE_CLASS(
57 MediaSessionVisibilityBrowserTest_AndroidPipeline_SuspendOnHide,
58 TestSessionInactiveWhenHiddenAfterContentPause)
59 INCLUDE_TEST_FROM_BASE_CLASS(
60 MediaSessionVisibilityBrowserTest_AndroidPipeline_SuspendOnHide,
61 TestSessionInactiveWhenHiddenWhilePlaying)
62
63 // AndroidPipeline + NosuspendOnHide
64 class MediaSessionVisibilityBrowserTest_AndroidPipeline_NosuspendOnHide :
65 public MediaSessionVisibilityBrowserTestBase {
66 void SetUpCommandLine(base::CommandLine* command_line) override {
67 MediaSessionVisibilityBrowserTestBase::SetUpCommandLine(command_line);
68 command_line->AppendSwitch(switches::kDisableMediaSuspend);
69 }
70 };
71
72 INCLUDE_TEST_FROM_BASE_CLASS(
73 MediaSessionVisibilityBrowserTest_AndroidPipeline_NosuspendOnHide,
74 TestSessionSuspendedWhenHiddenAfterContentPause)
75 INCLUDE_TEST_FROM_BASE_CLASS(
76 MediaSessionVisibilityBrowserTest_AndroidPipeline_NosuspendOnHide,
77 TestSessionActiveWhenHiddenWhilePlaying)
78
79 #endif // defined(OS_ANDROID)
OLDNEW
« no previous file with comments | « content/browser/media/session/media_session_visibility_browsertest_base.cc ('k') | content/content_tests.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698