widevine-partner/android
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove API Version TODO Comments

Browse Source 
(This is a merge of http://go/wvgerrit/26221)

We have agreed on API 26 for O, so these comments can be removed without
making any further changes.

Bug: 34548395
Test: This is a comment-only change. But I verified it builds.
Change-Id: I490befba4f47902a84b60e562a7a5eb162a9f3be
This commit is contained in:
John W. Bruce
2017-04-19 16:48:14 -07:00
parent ce6bd41627
commit 68d26ef98b
2 changed files with 0 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
libwvdrmengine/src_hidl/WVDrmFactory.cpp
View File

@@ -63,7 +63,6 @@ bool WVDrmFactory::areSpoidsEnabled() {
// version is the first if it's a factory ROM. // version is the first if it's a factory ROM.
firstApiLevel = property_get_int32("ro.build.version.sdk", 0); firstApiLevel = property_get_int32("ro.build.version.sdk", 0);
} }
// TODO(juce): b/34548395 Make sure this API version is correct.
return firstApiLevel >= 26; // Android O return firstApiLevel >= 26; // Android O
} }

1
libwvdrmengine/test/unit/WVDrmFactory_test.cpp
View File

@@ -104,7 +104,6 @@ TEST(WVDrmFactoryTest, CalculatesSpoidUseCorrectly) {
// version is the first if it's a factory ROM. // version is the first if it's a factory ROM.
firstApiLevel = property_get_int32("ro.build.version.sdk", 0); firstApiLevel = property_get_int32("ro.build.version.sdk", 0);
} }
// TODO(juce): b/34548395 Make sure this API version is correct.
bool shouldUseSpoids = (firstApiLevel >= 26); // Android O bool shouldUseSpoids = (firstApiLevel >= 26); // Android O
EXPECT_EQ(shouldUseSpoids, factory.areSpoidsEnabled()) << EXPECT_EQ(shouldUseSpoids, factory.areSpoidsEnabled()) <<