Chromium Code Reviews| Index: chrome/browser/policy/policy_browsertest.cc |
| diff --git a/chrome/browser/policy/policy_browsertest.cc b/chrome/browser/policy/policy_browsertest.cc |
| index 3bef5048fbc66328b51c287718f731071d71f9f0..5c93a9f403dd561f777254c1bc2988bdff1e881c 100644 |
| --- a/chrome/browser/policy/policy_browsertest.cc |
| +++ b/chrome/browser/policy/policy_browsertest.cc |
| @@ -1305,9 +1305,10 @@ IN_PROC_BROWSER_TEST_F(PolicyTest, ExtensionInstallForcelist) { |
| content::NotificationService::AllSources()); |
| UpdateProviderPolicy(policies); |
| observer.Wait(); |
| - content::Details<const extensions::Extension> details = observer.details(); |
| - EXPECT_EQ(kGoodCrxId, details->id()); |
| - EXPECT_EQ(details.ptr(), service->GetExtensionById(kGoodCrxId, true)); |
| + // Note: Cannot check that the notification details match the expected |
| + // exception, since the details object has already been freed prior to |
| + // the completion of observer.Wait(). |
| + |
|
Joao da Silva
2013/05/03 06:38:45
Check that the extension was installed here:
EXPEC
Matt Giuca
2013/05/03 07:09:32
Done.
|
| // The user is not allowed to uninstall force-installed extensions. |
| UninstallExtension(kGoodCrxId, false); |
| } |
| @@ -1377,9 +1378,9 @@ IN_PROC_BROWSER_TEST_F(PolicyTest, ExtensionInstallSources) { |
| content::NotificationService::AllSources()); |
| PerformClick(1, 0); |
| observer.Wait(); |
| - |
| - content::Details<const extensions::Extension> details = observer.details(); |
| - EXPECT_EQ(kAdBlockCrxId, details->id()); |
| + // Note: Cannot check that the notification details match the expected |
| + // exception, since the details object has already been freed prior to |
| + // the completion of observer.Wait(). |
| // The first extension shouldn't be present, the second should be there. |
| EXPECT_FALSE(extension_service()->GetExtensionById(kGoodCrxId, true)); |