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

Side by Side Diff: chrome/renderer/media/chrome_key_systems_provider_unittest.cc

Issue 2712983004: Simplify/Cleanup MediaClient (Closed)
Patch Set: Test fix. Created 3 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 2017 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 "chrome/renderer/media/chrome_key_systems_provider.h"
6
7 #include <string>
8 #include <vector>
9
10 #include "base/test/simple_test_tick_clock.h"
11 #include "testing/gtest/include/gtest/gtest.h"
12
13 #include "widevine_cdm_version.h" // In SHARED_INTERMEDIATE_DIR.
14
15 namespace chrome {
16
17 namespace {
18
19 class TestKeySystemProperties : public media::KeySystemProperties {
20 public:
21 explicit TestKeySystemProperties(const std::string& key_system_name)
22 : key_system_name_(key_system_name) {}
23
24 std::string GetKeySystemName() const override { return key_system_name_; }
25 bool IsSupportedInitDataType(
26 media::EmeInitDataType init_data_type) const override {
27 return false;
28 }
29 media::SupportedCodecs GetSupportedCodecs() const override {
30 return media::EME_CODEC_NONE;
31 }
32 media::EmeConfigRule GetRobustnessConfigRule(
33 media::EmeMediaType media_type,
34 const std::string& requested_robustness) const override {
35 return requested_robustness.empty() ? media::EmeConfigRule::SUPPORTED
36 : media::EmeConfigRule::NOT_SUPPORTED;
37 }
38 media::EmeSessionTypeSupport GetPersistentLicenseSessionSupport()
39 const override {
40 return media::EmeSessionTypeSupport::NOT_SUPPORTED;
41 }
42 media::EmeSessionTypeSupport GetPersistentReleaseMessageSessionSupport()
43 const override {
44 return media::EmeSessionTypeSupport::NOT_SUPPORTED;
45 }
46 media::EmeFeatureSupport GetPersistentStateSupport() const override {
47 return media::EmeFeatureSupport::NOT_SUPPORTED;
48 }
49 media::EmeFeatureSupport GetDistinctiveIdentifierSupport() const override {
50 return media::EmeFeatureSupport::NOT_SUPPORTED;
51 }
52
53 private:
54 const std::string key_system_name_;
55 };
56
57 class TestKeySystemsProviderDelegateImpl
58 : public TestKeySystemsProviderDelegate {
59 public:
60 TestKeySystemsProviderDelegateImpl() : include_widevine_(false) {}
61
62 void AddTestKeySystems(
63 std::vector<std::unique_ptr<media::KeySystemProperties>>* key_systems)
64 override {
65 key_systems->emplace_back(
66 new TestKeySystemProperties("com.example.foobar"));
67
68 if (include_widevine_) {
69 #if defined(WIDEVINE_CDM_AVAILABLE)
70 key_systems->emplace_back(
71 new TestKeySystemProperties(kWidevineKeySystem));
72 #else
73 // Tests should only attempt to include Widevine when it is available.
74 NOTREACHED();
75 #endif
76 }
77 }
78
79 void set_include_widevine(bool include_widevine) {
80 include_widevine_ = include_widevine;
81 }
82
83 private:
84 bool include_widevine_;
85 };
86
87 } // namespace
88
89 TEST(ChromeKeySystemsProviderTest, IsKeySystemsUpdateNeeded) {
90 ChromeKeySystemsProvider key_systems_provider;
91
92 base::SimpleTestTickClock* tick_clock = new base::SimpleTestTickClock();
93 key_systems_provider.SetTickClockForTesting(
94 std::unique_ptr<base::TickClock>(tick_clock));
95
96 TestKeySystemsProviderDelegateImpl* provider_delegate =
97 new TestKeySystemsProviderDelegateImpl();
98 key_systems_provider.SetProviderDelegateForTesting(
99 std::unique_ptr<TestKeySystemsProviderDelegate>(provider_delegate));
100
101 // IsKeySystemsUpdateNeeded() always returns true after construction.
102 EXPECT_TRUE(key_systems_provider.IsKeySystemsUpdateNeeded());
103
104 std::vector<std::unique_ptr<media::KeySystemProperties>> key_systems;
105 key_systems_provider.AddSupportedKeySystems(&key_systems);
106
107 // No update needed immediately after AddSupportedKeySystems() call.
108 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
109
110 // Widevine not initially provided.
111 EXPECT_EQ(key_systems.size(), 1U);
112 EXPECT_EQ(key_systems[0]->GetKeySystemName(), "com.example.foobar");
113
114 // This is timing related. The update interval for Widevine is 1000 ms.
115 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
116 tick_clock->Advance(base::TimeDelta::FromMilliseconds(990));
117 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
118 tick_clock->Advance(base::TimeDelta::FromMilliseconds(10));
119
120 #if defined(WIDEVINE_CDM_AVAILABLE) && defined(WIDEVINE_CDM_IS_COMPONENT)
121 // Require update once enough time has passed for builds that install Widevine
122 // as a component.
123 EXPECT_TRUE(key_systems_provider.IsKeySystemsUpdateNeeded());
124
125 // Now add Widevine.
126 provider_delegate->set_include_widevine(true);
127 key_systems.clear();
128 key_systems_provider.AddSupportedKeySystems(&key_systems);
129
130 // Widevine should now be among the list.
131 bool found_widevine = false;
132 for (const auto& key_system_properties : key_systems) {
133 if (key_system_properties->GetKeySystemName() == kWidevineKeySystem) {
134 found_widevine = true;
135 break;
136 }
137 }
138 EXPECT_TRUE(found_widevine);
139
140 // Update not needed now, nor later because Widevine has been described.
141 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
142 tick_clock->Advance(base::TimeDelta::FromMilliseconds(1000));
143 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
144 tick_clock->Advance(base::TimeDelta::FromMilliseconds(1000));
145 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
146 #else
147 // No update needed for builds that either don't offer Widevine or do so
148 // as part of Chrome rather than component installer.
149 EXPECT_FALSE(key_systems_provider.IsKeySystemsUpdateNeeded());
150 #endif // defined(WIDEVINE_CDM_AVAILABLE) && defined(WIDEVINE_CDM_IS_COMPONENT)
151 }
152
153 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698