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

Unified Diff: chrome/browser/chromeos/display/display_preferences_unittest.cc

Issue 2799963003: Fix errors in display resolution change notifications (Closed)
Patch Set: Fix compile errors on non-cros systems 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/display/display_preferences_unittest.cc
diff --git a/chrome/browser/chromeos/display/display_preferences_unittest.cc b/chrome/browser/chromeos/display/display_preferences_unittest.cc
index e2cf68caabaeeb28fa3cca6e72e2413fe8f71706..0f35b28ad4b4e6e3ebfd06c7a81a00f68e5c0c08 100644
--- a/chrome/browser/chromeos/display/display_preferences_unittest.cc
+++ b/chrome/browser/chromeos/display/display_preferences_unittest.cc
@@ -574,10 +574,9 @@ TEST_F(DisplayPreferencesTest, PreventStore) {
new display::ManagedDisplayMode(gfx::Size(400, 300)));
scoped_refptr<display::ManagedDisplayMode> new_mode(
new display::ManagedDisplayMode(gfx::Size(500, 400)));
- if (shell->display_manager()->SetDisplayMode(id, new_mode)) {
- shell->resolution_notification_controller()->PrepareNotification(
- id, old_mode, new_mode, base::Closure());
- }
+ EXPECT_TRUE(shell->resolution_notification_controller()
+ ->PrepareNotificationAndSetDisplayMode(id, old_mode, new_mode,
+ base::Closure()));
UpdateDisplay("500x400#500x400|400x300|300x200");
const base::DictionaryValue* properties =

Powered by Google App Engine
This is Rietveld 408576698