Chromium Code Reviews| Index: chrome/browser/chromeos/policy/device_cloud_policy_browsertest.cc |
| diff --git a/chrome/browser/chromeos/policy/device_cloud_policy_browsertest.cc b/chrome/browser/chromeos/policy/device_cloud_policy_browsertest.cc |
| index e73c35647e519c586da8e348d7a62fe1c5535369..6ed97329c7bffaa41f9ca071b02c232ea9e01694 100644 |
| --- a/chrome/browser/chromeos/policy/device_cloud_policy_browsertest.cc |
| +++ b/chrome/browser/chromeos/policy/device_cloud_policy_browsertest.cc |
| @@ -27,8 +27,8 @@ |
| #include "chrome/browser/chromeos/policy/proto/chrome_device_policy.pb.h" |
| #include "chrome/browser/chromeos/profiles/profile_helper.h" |
| #include "chrome/browser/chromeos/settings/device_settings_service.h" |
| +#include "chrome/browser/extensions/chrome_test_extension_loader.cc" |
|
emaxx
2017/04/07 10:05:25
.h should be included instead of .cc
limasdf
2017/04/07 12:04:40
Oops. fixed. Thanks!
|
| #include "chrome/browser/extensions/extension_service.h" |
| -#include "chrome/browser/extensions/unpacked_installer.h" |
| #include "chrome/browser/policy/test/local_policy_test_server.h" |
| #include "chrome/browser/profiles/profile.h" |
| #include "chrome/browser/ui/browser.h" |
| @@ -58,7 +58,6 @@ |
| #include "extensions/browser/extension_system.h" |
| #include "extensions/common/constants.h" |
| #include "extensions/common/extension.h" |
| -#include "extensions/test/extension_test_notification_observer.h" |
| #include "extensions/test/result_catcher.h" |
| #include "net/http/http_status_code.h" |
| #include "net/url_request/test_url_fetcher_factory.h" |
| @@ -385,16 +384,10 @@ class SigninExtensionsDeviceCloudPolicyBrowserTest |
| SigninExtensionsDeviceCloudPolicyBrowserTest() |
| : fetcher_factory_(&fetcher_impl_factory_) {} |
| - const extensions::Extension* InstallAndLoadTestExtension() const { |
| - Profile* signin_profile = GetSigninProfile(); |
| - ExtensionService* service = |
| - extensions::ExtensionSystem::Get(signin_profile)->extension_service(); |
| - scoped_refptr<extensions::UnpackedInstaller> installer( |
| - extensions::UnpackedInstaller::Create(service)); |
| - extensions::ExtensionTestNotificationObserver observer(signin_profile); |
| - installer->Load(GetTestExtensionSourcePath()); |
| - observer.WaitForExtensionLoad(); |
| - return GetTestExtension(); |
| + scoped_refptr<const extensions::Extension> InstallAndLoadTestExtension() |
| + const { |
| + extensions::ChromeTestExtensionLoader loader(GetSigninProfile()); |
| + return loader.LoadExtension(GetTestExtensionSourcePath()); |
| } |
| private: |
| @@ -462,7 +455,8 @@ class SigninExtensionsDeviceCloudPolicyBrowserTest |
| IN_PROC_BROWSER_TEST_F(SigninExtensionsDeviceCloudPolicyBrowserTest, |
| InstallAndRunInWindow) { |
| - const extensions::Extension* extension = InstallAndLoadTestExtension(); |
| + scoped_refptr<const extensions::Extension> extension = |
| + InstallAndLoadTestExtension(); |
| ASSERT_TRUE(extension); |
| Browser* browser = CreateBrowser(GetSigninProfile()); |
| extensions::ResultCatcher result_catcher; |