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

Unified Diff: ash/display/display_color_manager_chromeos_unittest.cc

Issue 2431853002: Remove built-in path check from Quirks (Closed)
Patch Set: Created 4 years, 2 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: ash/display/display_color_manager_chromeos_unittest.cc
diff --git a/ash/display/display_color_manager_chromeos_unittest.cc b/ash/display/display_color_manager_chromeos_unittest.cc
index 86a84971f5dbcc398fe0a6f51d97e4c9a5ff0627..83de620f760e298ca3402437c26eb23e4c1c7f9c 100644
--- a/ash/display/display_color_manager_chromeos_unittest.cc
+++ b/ash/display/display_color_manager_chromeos_unittest.cc
@@ -83,17 +83,11 @@ class QuirksManagerDelegateTestImpl : public quirks::QuirksManager::Delegate {
// Unused by these tests.
std::string GetApiKey() const override { return std::string(); }
- base::FilePath GetBuiltInDisplayProfileDirectory() const override {
+ base::FilePath GetDisplayProfileDirectory() const override {
return color_path_;
}
- // Unused by these tests.
- base::FilePath GetDownloadDisplayProfileDirectory() const override {
- return base::FilePath();
- }
-
- // |false| prevents attempts at Quirks Server contact.
- bool DevicePolicyEnabled() const override { return false; }
+ bool DevicePolicyEnabled() const override { return true; }
private:
~QuirksManagerDelegateTestImpl() override = default;
@@ -126,7 +120,7 @@ class DisplayColorManagerTest : public testing::Test {
.Append(FILE_PATH_LITERAL("display"))
.Append(FILE_PATH_LITERAL("test_data"));
path_override_.reset(new base::ScopedPathOverride(
- chromeos::DIR_DEVICE_COLOR_CALIBRATION_PROFILES, color_path_));
+ chromeos::DIR_DEVICE_DISPLAY_PROFILES, color_path_));
quirks::QuirksManager::Initialize(
std::unique_ptr<quirks::QuirksManager::Delegate>(
« no previous file with comments | « no previous file | chrome/browser/chromeos/display/quirks_browsertest.cc » ('j') | components/quirks/quirks_manager.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698