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

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

Issue 2799963003: Fix errors in display resolution change notifications (Closed)
Patch Set: Update commit message 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 3a82ed535ab161a16d56c6e922848ba50fb36e96..1cf059762693584cf222a2863cf57b702c0e9d15 100644
--- a/chrome/browser/chromeos/display/display_preferences_unittest.cc
+++ b/chrome/browser/chromeos/display/display_preferences_unittest.cc
@@ -573,10 +573,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