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

Unified Diff: chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc

Issue 2486813002: Add DeviceADPolicyManager to provide AD policy. (Closed)
Patch Set: Address Bernhard's comments Created 4 years, 1 month 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/ui/webui/options/chromeos/accounts_options_browsertest.cc
diff --git a/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc b/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
index 956a0a961ee83bb9da9806515fb50866932836c7..d03da55880f4da8e9f8fe1740e25feb5ba10b781 100644
--- a/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
+++ b/chrome/browser/ui/webui/options/chromeos/accounts_options_browsertest.cc
@@ -8,6 +8,7 @@
#include "base/compiler_specific.h"
#include "base/macros.h"
#include "base/memory/ptr_util.h"
+#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/chromeos/login/login_manager_test.h"
#include "chrome/browser/chromeos/login/startup_utils.h"
#include "chrome/browser/chromeos/login/ui/user_adding_screen.h"
@@ -22,6 +23,8 @@
#include "chromeos/settings/cros_settings_names.h"
#include "components/prefs/pref_service.h"
#include "components/user_manager/user_manager.h"
+#include "content/public/browser/notification_service.h"
+#include "content/public/browser/notification_source.h"
#include "content/public/browser/web_contents.h"
#include "content/public/test/browser_test_utils.h"
#include "content/public/test/test_utils.h"
@@ -58,6 +61,12 @@ class AccountsOptionsTest : public LoginManagerTest {
device_settings_provider_ =
settings->RemoveSettingsProvider(device_settings_provider);
settings->AddSettingsProvider(std::move(stub_settings_provider_));
+
+ // Notify ChromeUserManager of ownership change.
+ content::NotificationService::current()->Notify(
Dan Beam 2016/11/17 19:09:10 the notification service is deprecated (as is code
Thiemo Nagel 2016/11/18 09:46:51 As discussed offline: The CL fixes some useless fi
+ chrome::NOTIFICATION_OWNERSHIP_STATUS_CHANGED,
+ content::Source<AccountsOptionsTest>(this),
+ content::NotificationService::NoDetails());
}
void TearDownOnMainThread() override {

Powered by Google App Engine
This is Rietveld 408576698