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

Unified Diff: gpu/config/gpu_control_list_machine_model_info_unittest.cc

Issue 241793002: Fix machine_model behaviors in gpu blacklist. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: gpu/config/gpu_control_list_machine_model_info_unittest.cc
diff --git a/gpu/config/gpu_control_list_machine_model_info_unittest.cc b/gpu/config/gpu_control_list_machine_model_info_unittest.cc
index 9c681a9264f8609aeb614a323847e9e2595ad404..4fa7b80d360c5024e31caba9cc470b6dfa088d6c 100644
--- a/gpu/config/gpu_control_list_machine_model_info_unittest.cc
+++ b/gpu/config/gpu_control_list_machine_model_info_unittest.cc
@@ -48,10 +48,17 @@ TEST_F(MachineModelInfoTest, ValidModelInfo) {
TEST_F(MachineModelInfoTest, ModelComparison) {
MachineModelInfo info("=", "model_a", ">", "3.4", std::string());
+ EXPECT_TRUE(info.IsValid());
EXPECT_TRUE(info.Contains("model_a", "4"));
EXPECT_FALSE(info.Contains("model_b", "4"));
EXPECT_FALSE(info.Contains("model_a", "3.2"));
}
+TEST_F(MachineModelInfoTest, ModelNameNoVersion) {
+ MachineModelInfo info("=", "model_a", "any", std::string(), std::string());
+ EXPECT_TRUE(info.IsValid());
+ EXPECT_TRUE(info.Contains("model_a", std::string()));
+}
+
} // namespace gpu
« gpu/config/gpu_control_list.cc ('K') | « gpu/config/gpu_control_list_entry_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698