| OLD | NEW |
| 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "base/process_util.h" | 5 #include "base/process_util.h" |
| 6 #include "chrome/browser/extensions/crashed_extension_infobar.h" | 6 #include "chrome/browser/extensions/crashed_extension_infobar.h" |
| 7 #include "chrome/browser/extensions/extension_browsertest.h" | 7 #include "chrome/browser/extensions/extension_browsertest.h" |
| 8 #include "chrome/browser/extensions/extension_host.h" | 8 #include "chrome/browser/extensions/extension_host.h" |
| 9 #include "chrome/browser/extensions/extension_process_manager.h" | 9 #include "chrome/browser/extensions/extension_process_manager.h" |
| 10 #include "chrome/browser/extensions/extensions_service.h" | 10 #include "chrome/browser/extensions/extension_service.h" |
| 11 #include "chrome/browser/profiles/profile.h" | 11 #include "chrome/browser/profiles/profile.h" |
| 12 #include "chrome/browser/renderer_host/render_process_host.h" | 12 #include "chrome/browser/renderer_host/render_process_host.h" |
| 13 #include "chrome/browser/renderer_host/render_view_host.h" | 13 #include "chrome/browser/renderer_host/render_view_host.h" |
| 14 #include "chrome/browser/tab_contents/infobar_delegate.h" | 14 #include "chrome/browser/tab_contents/infobar_delegate.h" |
| 15 #include "chrome/browser/tab_contents/tab_contents.h" | 15 #include "chrome/browser/tab_contents/tab_contents.h" |
| 16 #include "chrome/browser/ui/browser.h" | 16 #include "chrome/browser/ui/browser.h" |
| 17 #include "chrome/common/result_codes.h" | 17 #include "chrome/common/result_codes.h" |
| 18 #include "chrome/test/ui_test_utils.h" | 18 #include "chrome/test/ui_test_utils.h" |
| 19 | 19 |
| 20 class ExtensionCrashRecoveryTest : public ExtensionBrowserTest { | 20 class ExtensionCrashRecoveryTest : public ExtensionBrowserTest { |
| 21 protected: | 21 protected: |
| 22 ExtensionsService* GetExtensionsService() { | 22 ExtensionService* GetExtensionService() { |
| 23 return browser()->profile()->GetExtensionsService(); | 23 return browser()->profile()->GetExtensionService(); |
| 24 } | 24 } |
| 25 | 25 |
| 26 ExtensionProcessManager* GetExtensionProcessManager() { | 26 ExtensionProcessManager* GetExtensionProcessManager() { |
| 27 return browser()->profile()->GetExtensionProcessManager(); | 27 return browser()->profile()->GetExtensionProcessManager(); |
| 28 } | 28 } |
| 29 | 29 |
| 30 CrashedExtensionInfoBarDelegate* GetCrashedExtensionInfoBarDelegate( | 30 CrashedExtensionInfoBarDelegate* GetCrashedExtensionInfoBarDelegate( |
| 31 int index) { | 31 int index) { |
| 32 TabContents* current_tab = browser()->GetSelectedTabContents(); | 32 TabContents* current_tab = browser()->GetSelectedTabContents(); |
| 33 EXPECT_LT(index, current_tab->infobar_delegate_count()); | 33 EXPECT_LT(index, current_tab->infobar_delegate_count()); |
| (...skipping 10 matching lines...) Expand all Loading... |
| 44 } | 44 } |
| 45 | 45 |
| 46 void CancelCrashedExtensionInfobar(int index) { | 46 void CancelCrashedExtensionInfobar(int index) { |
| 47 CrashedExtensionInfoBarDelegate* infobar = | 47 CrashedExtensionInfoBarDelegate* infobar = |
| 48 GetCrashedExtensionInfoBarDelegate(index); | 48 GetCrashedExtensionInfoBarDelegate(index); |
| 49 ASSERT_TRUE(infobar); | 49 ASSERT_TRUE(infobar); |
| 50 infobar->Cancel(); | 50 infobar->Cancel(); |
| 51 } | 51 } |
| 52 | 52 |
| 53 void CrashExtension(size_t index) { | 53 void CrashExtension(size_t index) { |
| 54 ASSERT_LT(index, GetExtensionsService()->extensions()->size()); | 54 ASSERT_LT(index, GetExtensionService()->extensions()->size()); |
| 55 const Extension* extension = | 55 const Extension* extension = |
| 56 GetExtensionsService()->extensions()->at(index); | 56 GetExtensionService()->extensions()->at(index); |
| 57 ASSERT_TRUE(extension); | 57 ASSERT_TRUE(extension); |
| 58 std::string extension_id(extension->id()); | 58 std::string extension_id(extension->id()); |
| 59 ExtensionHost* extension_host = | 59 ExtensionHost* extension_host = |
| 60 GetExtensionProcessManager()->GetBackgroundHostForExtension(extension); | 60 GetExtensionProcessManager()->GetBackgroundHostForExtension(extension); |
| 61 ASSERT_TRUE(extension_host); | 61 ASSERT_TRUE(extension_host); |
| 62 | 62 |
| 63 RenderProcessHost* extension_rph = | 63 RenderProcessHost* extension_rph = |
| 64 extension_host->render_view_host()->process(); | 64 extension_host->render_view_host()->process(); |
| 65 base::KillProcess(extension_rph->GetHandle(), ResultCodes::KILLED, false); | 65 base::KillProcess(extension_rph->GetHandle(), ResultCodes::KILLED, false); |
| 66 ASSERT_TRUE(WaitForExtensionCrash(extension_id)); | 66 ASSERT_TRUE(WaitForExtensionCrash(extension_id)); |
| 67 ASSERT_FALSE( | 67 ASSERT_FALSE( |
| 68 GetExtensionProcessManager()->GetBackgroundHostForExtension(extension)); | 68 GetExtensionProcessManager()->GetBackgroundHostForExtension(extension)); |
| 69 } | 69 } |
| 70 | 70 |
| 71 void CheckExtensionConsistency(size_t index) { | 71 void CheckExtensionConsistency(size_t index) { |
| 72 ASSERT_LT(index, GetExtensionsService()->extensions()->size()); | 72 ASSERT_LT(index, GetExtensionService()->extensions()->size()); |
| 73 const Extension* extension = | 73 const Extension* extension = |
| 74 GetExtensionsService()->extensions()->at(index); | 74 GetExtensionService()->extensions()->at(index); |
| 75 ASSERT_TRUE(extension); | 75 ASSERT_TRUE(extension); |
| 76 ExtensionHost* extension_host = | 76 ExtensionHost* extension_host = |
| 77 GetExtensionProcessManager()->GetBackgroundHostForExtension(extension); | 77 GetExtensionProcessManager()->GetBackgroundHostForExtension(extension); |
| 78 ASSERT_TRUE(extension_host); | 78 ASSERT_TRUE(extension_host); |
| 79 ASSERT_TRUE(GetExtensionProcessManager()->HasExtensionHost(extension_host)); | 79 ASSERT_TRUE(GetExtensionProcessManager()->HasExtensionHost(extension_host)); |
| 80 ASSERT_TRUE(extension_host->IsRenderViewLive()); | 80 ASSERT_TRUE(extension_host->IsRenderViewLive()); |
| 81 ASSERT_EQ(extension_host->render_view_host()->process(), | 81 ASSERT_EQ(extension_host->render_view_host()->process(), |
| 82 GetExtensionProcessManager()->GetExtensionProcess(extension->id())); | 82 GetExtensionProcessManager()->GetExtensionProcess(extension->id())); |
| 83 } | 83 } |
| 84 | 84 |
| 85 void LoadTestExtension() { | 85 void LoadTestExtension() { |
| 86 ExtensionBrowserTest::SetUpInProcessBrowserTestFixture(); | 86 ExtensionBrowserTest::SetUpInProcessBrowserTestFixture(); |
| 87 const size_t size_before = GetExtensionsService()->extensions()->size(); | 87 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 88 ASSERT_TRUE(LoadExtension( | 88 ASSERT_TRUE(LoadExtension( |
| 89 test_data_dir_.AppendASCII("common").AppendASCII("background_page"))); | 89 test_data_dir_.AppendASCII("common").AppendASCII("background_page"))); |
| 90 const Extension* extension = GetExtensionsService()->extensions()->back(); | 90 const Extension* extension = GetExtensionService()->extensions()->back(); |
| 91 ASSERT_TRUE(extension); | 91 ASSERT_TRUE(extension); |
| 92 first_extension_id_ = extension->id(); | 92 first_extension_id_ = extension->id(); |
| 93 CheckExtensionConsistency(size_before); | 93 CheckExtensionConsistency(size_before); |
| 94 } | 94 } |
| 95 | 95 |
| 96 void LoadSecondExtension() { | 96 void LoadSecondExtension() { |
| 97 int offset = GetExtensionsService()->extensions()->size(); | 97 int offset = GetExtensionService()->extensions()->size(); |
| 98 ASSERT_TRUE(LoadExtension( | 98 ASSERT_TRUE(LoadExtension( |
| 99 test_data_dir_.AppendASCII("install").AppendASCII("install"))); | 99 test_data_dir_.AppendASCII("install").AppendASCII("install"))); |
| 100 const Extension* extension = | 100 const Extension* extension = |
| 101 GetExtensionsService()->extensions()->at(offset); | 101 GetExtensionService()->extensions()->at(offset); |
| 102 ASSERT_TRUE(extension); | 102 ASSERT_TRUE(extension); |
| 103 second_extension_id_ = extension->id(); | 103 second_extension_id_ = extension->id(); |
| 104 CheckExtensionConsistency(offset); | 104 CheckExtensionConsistency(offset); |
| 105 } | 105 } |
| 106 | 106 |
| 107 std::string first_extension_id_; | 107 std::string first_extension_id_; |
| 108 std::string second_extension_id_; | 108 std::string second_extension_id_; |
| 109 }; | 109 }; |
| 110 | 110 |
| 111 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, Basic) { | 111 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, Basic) { |
| 112 const size_t size_before = GetExtensionsService()->extensions()->size(); | 112 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 113 LoadTestExtension(); | 113 LoadTestExtension(); |
| 114 CrashExtension(size_before); | 114 CrashExtension(size_before); |
| 115 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 115 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 116 AcceptCrashedExtensionInfobar(0); | 116 AcceptCrashedExtensionInfobar(0); |
| 117 | 117 |
| 118 SCOPED_TRACE("after clicking the infobar"); | 118 SCOPED_TRACE("after clicking the infobar"); |
| 119 CheckExtensionConsistency(size_before); | 119 CheckExtensionConsistency(size_before); |
| 120 } | 120 } |
| 121 | 121 |
| 122 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, CloseAndReload) { | 122 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, CloseAndReload) { |
| 123 const size_t size_before = GetExtensionsService()->extensions()->size(); | 123 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 124 LoadTestExtension(); | 124 LoadTestExtension(); |
| 125 CrashExtension(size_before); | 125 CrashExtension(size_before); |
| 126 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 126 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 127 CancelCrashedExtensionInfobar(0); | 127 CancelCrashedExtensionInfobar(0); |
| 128 ReloadExtension(first_extension_id_); | 128 ReloadExtension(first_extension_id_); |
| 129 | 129 |
| 130 SCOPED_TRACE("after reloading"); | 130 SCOPED_TRACE("after reloading"); |
| 131 CheckExtensionConsistency(size_before); | 131 CheckExtensionConsistency(size_before); |
| 132 } | 132 } |
| 133 | 133 |
| 134 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, ReloadIndependently) { | 134 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, ReloadIndependently) { |
| 135 const size_t size_before = GetExtensionsService()->extensions()->size(); | 135 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 136 LoadTestExtension(); | 136 LoadTestExtension(); |
| 137 CrashExtension(size_before); | 137 CrashExtension(size_before); |
| 138 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 138 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 139 | 139 |
| 140 ReloadExtension(first_extension_id_); | 140 ReloadExtension(first_extension_id_); |
| 141 | 141 |
| 142 SCOPED_TRACE("after reloading"); | 142 SCOPED_TRACE("after reloading"); |
| 143 CheckExtensionConsistency(size_before); | 143 CheckExtensionConsistency(size_before); |
| 144 | 144 |
| 145 TabContents* current_tab = browser()->GetSelectedTabContents(); | 145 TabContents* current_tab = browser()->GetSelectedTabContents(); |
| 146 ASSERT_TRUE(current_tab); | 146 ASSERT_TRUE(current_tab); |
| 147 | 147 |
| 148 // The infobar should automatically hide after the extension is successfully | 148 // The infobar should automatically hide after the extension is successfully |
| 149 // reloaded. | 149 // reloaded. |
| 150 ASSERT_EQ(0, current_tab->infobar_delegate_count()); | 150 ASSERT_EQ(0, current_tab->infobar_delegate_count()); |
| 151 } | 151 } |
| 152 | 152 |
| 153 // Make sure that when we don't do anything about the crashed extension | 153 // Make sure that when we don't do anything about the crashed extension |
| 154 // and close the browser, it doesn't crash. The browser is closed implicitly | 154 // and close the browser, it doesn't crash. The browser is closed implicitly |
| 155 // at the end of each browser test. | 155 // at the end of each browser test. |
| 156 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, ShutdownWhileCrashed) { | 156 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, ShutdownWhileCrashed) { |
| 157 const size_t size_before = GetExtensionsService()->extensions()->size(); | 157 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 158 LoadTestExtension(); | 158 LoadTestExtension(); |
| 159 CrashExtension(size_before); | 159 CrashExtension(size_before); |
| 160 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 160 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 161 } | 161 } |
| 162 | 162 |
| 163 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, TwoExtensionsCrashFirst) { | 163 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, TwoExtensionsCrashFirst) { |
| 164 const size_t size_before = GetExtensionsService()->extensions()->size(); | 164 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 165 LoadTestExtension(); | 165 LoadTestExtension(); |
| 166 LoadSecondExtension(); | 166 LoadSecondExtension(); |
| 167 CrashExtension(size_before); | 167 CrashExtension(size_before); |
| 168 ASSERT_EQ(size_before + 1, GetExtensionsService()->extensions()->size()); | 168 ASSERT_EQ(size_before + 1, GetExtensionService()->extensions()->size()); |
| 169 AcceptCrashedExtensionInfobar(0); | 169 AcceptCrashedExtensionInfobar(0); |
| 170 | 170 |
| 171 SCOPED_TRACE("after clicking the infobar"); | 171 SCOPED_TRACE("after clicking the infobar"); |
| 172 CheckExtensionConsistency(size_before); | 172 CheckExtensionConsistency(size_before); |
| 173 CheckExtensionConsistency(size_before + 1); | 173 CheckExtensionConsistency(size_before + 1); |
| 174 } | 174 } |
| 175 | 175 |
| 176 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, TwoExtensionsCrashSecond) { | 176 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, TwoExtensionsCrashSecond) { |
| 177 const size_t size_before = GetExtensionsService()->extensions()->size(); | 177 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 178 LoadTestExtension(); | 178 LoadTestExtension(); |
| 179 LoadSecondExtension(); | 179 LoadSecondExtension(); |
| 180 CrashExtension(size_before + 1); | 180 CrashExtension(size_before + 1); |
| 181 ASSERT_EQ(size_before + 1, GetExtensionsService()->extensions()->size()); | 181 ASSERT_EQ(size_before + 1, GetExtensionService()->extensions()->size()); |
| 182 AcceptCrashedExtensionInfobar(0); | 182 AcceptCrashedExtensionInfobar(0); |
| 183 | 183 |
| 184 SCOPED_TRACE("after clicking the infobar"); | 184 SCOPED_TRACE("after clicking the infobar"); |
| 185 CheckExtensionConsistency(size_before); | 185 CheckExtensionConsistency(size_before); |
| 186 CheckExtensionConsistency(size_before + 1); | 186 CheckExtensionConsistency(size_before + 1); |
| 187 } | 187 } |
| 188 | 188 |
| 189 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, | 189 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, |
| 190 TwoExtensionsCrashBothAtOnce) { | 190 TwoExtensionsCrashBothAtOnce) { |
| 191 const size_t size_before = GetExtensionsService()->extensions()->size(); | 191 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 192 LoadTestExtension(); | 192 LoadTestExtension(); |
| 193 LoadSecondExtension(); | 193 LoadSecondExtension(); |
| 194 CrashExtension(size_before); | 194 CrashExtension(size_before); |
| 195 ASSERT_EQ(size_before + 1, GetExtensionsService()->extensions()->size()); | 195 ASSERT_EQ(size_before + 1, GetExtensionService()->extensions()->size()); |
| 196 CrashExtension(size_before); | 196 CrashExtension(size_before); |
| 197 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 197 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 198 | 198 |
| 199 { | 199 { |
| 200 SCOPED_TRACE("first infobar"); | 200 SCOPED_TRACE("first infobar"); |
| 201 AcceptCrashedExtensionInfobar(0); | 201 AcceptCrashedExtensionInfobar(0); |
| 202 CheckExtensionConsistency(size_before); | 202 CheckExtensionConsistency(size_before); |
| 203 } | 203 } |
| 204 | 204 |
| 205 { | 205 { |
| 206 SCOPED_TRACE("second infobar"); | 206 SCOPED_TRACE("second infobar"); |
| 207 AcceptCrashedExtensionInfobar(0); | 207 AcceptCrashedExtensionInfobar(0); |
| 208 CheckExtensionConsistency(size_before); | 208 CheckExtensionConsistency(size_before); |
| 209 CheckExtensionConsistency(size_before + 1); | 209 CheckExtensionConsistency(size_before + 1); |
| 210 } | 210 } |
| 211 } | 211 } |
| 212 | 212 |
| 213 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, TwoExtensionsOneByOne) { | 213 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, TwoExtensionsOneByOne) { |
| 214 const size_t size_before = GetExtensionsService()->extensions()->size(); | 214 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 215 LoadTestExtension(); | 215 LoadTestExtension(); |
| 216 CrashExtension(size_before); | 216 CrashExtension(size_before); |
| 217 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 217 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 218 LoadSecondExtension(); | 218 LoadSecondExtension(); |
| 219 CrashExtension(size_before); | 219 CrashExtension(size_before); |
| 220 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 220 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 221 | 221 |
| 222 { | 222 { |
| 223 SCOPED_TRACE("first infobar"); | 223 SCOPED_TRACE("first infobar"); |
| 224 AcceptCrashedExtensionInfobar(0); | 224 AcceptCrashedExtensionInfobar(0); |
| 225 CheckExtensionConsistency(size_before); | 225 CheckExtensionConsistency(size_before); |
| 226 } | 226 } |
| 227 | 227 |
| 228 { | 228 { |
| 229 SCOPED_TRACE("second infobar"); | 229 SCOPED_TRACE("second infobar"); |
| 230 AcceptCrashedExtensionInfobar(0); | 230 AcceptCrashedExtensionInfobar(0); |
| 231 CheckExtensionConsistency(size_before); | 231 CheckExtensionConsistency(size_before); |
| 232 CheckExtensionConsistency(size_before + 1); | 232 CheckExtensionConsistency(size_before + 1); |
| 233 } | 233 } |
| 234 } | 234 } |
| 235 | 235 |
| 236 // Make sure that when we don't do anything about the crashed extensions | 236 // Make sure that when we don't do anything about the crashed extensions |
| 237 // and close the browser, it doesn't crash. The browser is closed implicitly | 237 // and close the browser, it doesn't crash. The browser is closed implicitly |
| 238 // at the end of each browser test. | 238 // at the end of each browser test. |
| 239 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, | 239 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, |
| 240 TwoExtensionsShutdownWhileCrashed) { | 240 TwoExtensionsShutdownWhileCrashed) { |
| 241 const size_t size_before = GetExtensionsService()->extensions()->size(); | 241 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 242 LoadTestExtension(); | 242 LoadTestExtension(); |
| 243 CrashExtension(size_before); | 243 CrashExtension(size_before); |
| 244 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 244 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 245 LoadSecondExtension(); | 245 LoadSecondExtension(); |
| 246 CrashExtension(size_before); | 246 CrashExtension(size_before); |
| 247 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 247 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 248 } | 248 } |
| 249 | 249 |
| 250 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, | 250 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, |
| 251 TwoExtensionsIgnoreFirst) { | 251 TwoExtensionsIgnoreFirst) { |
| 252 const size_t size_before = GetExtensionsService()->extensions()->size(); | 252 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 253 LoadTestExtension(); | 253 LoadTestExtension(); |
| 254 LoadSecondExtension(); | 254 LoadSecondExtension(); |
| 255 CrashExtension(size_before); | 255 CrashExtension(size_before); |
| 256 ASSERT_EQ(size_before + 1, GetExtensionsService()->extensions()->size()); | 256 ASSERT_EQ(size_before + 1, GetExtensionService()->extensions()->size()); |
| 257 CrashExtension(size_before); | 257 CrashExtension(size_before); |
| 258 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 258 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 259 | 259 |
| 260 CancelCrashedExtensionInfobar(0); | 260 CancelCrashedExtensionInfobar(0); |
| 261 AcceptCrashedExtensionInfobar(1); | 261 AcceptCrashedExtensionInfobar(1); |
| 262 | 262 |
| 263 SCOPED_TRACE("infobars done"); | 263 SCOPED_TRACE("infobars done"); |
| 264 ASSERT_EQ(size_before + 1, GetExtensionsService()->extensions()->size()); | 264 ASSERT_EQ(size_before + 1, GetExtensionService()->extensions()->size()); |
| 265 CheckExtensionConsistency(size_before); | 265 CheckExtensionConsistency(size_before); |
| 266 } | 266 } |
| 267 | 267 |
| 268 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, | 268 IN_PROC_BROWSER_TEST_F(ExtensionCrashRecoveryTest, |
| 269 TwoExtensionsReloadIndependently) { | 269 TwoExtensionsReloadIndependently) { |
| 270 const size_t size_before = GetExtensionsService()->extensions()->size(); | 270 const size_t size_before = GetExtensionService()->extensions()->size(); |
| 271 LoadTestExtension(); | 271 LoadTestExtension(); |
| 272 LoadSecondExtension(); | 272 LoadSecondExtension(); |
| 273 CrashExtension(size_before); | 273 CrashExtension(size_before); |
| 274 ASSERT_EQ(size_before + 1, GetExtensionsService()->extensions()->size()); | 274 ASSERT_EQ(size_before + 1, GetExtensionService()->extensions()->size()); |
| 275 CrashExtension(size_before); | 275 CrashExtension(size_before); |
| 276 ASSERT_EQ(size_before, GetExtensionsService()->extensions()->size()); | 276 ASSERT_EQ(size_before, GetExtensionService()->extensions()->size()); |
| 277 | 277 |
| 278 { | 278 { |
| 279 SCOPED_TRACE("first: reload"); | 279 SCOPED_TRACE("first: reload"); |
| 280 TabContents* current_tab = browser()->GetSelectedTabContents(); | 280 TabContents* current_tab = browser()->GetSelectedTabContents(); |
| 281 ASSERT_TRUE(current_tab); | 281 ASSERT_TRUE(current_tab); |
| 282 // At the beginning we should have one infobar displayed for each extension. | 282 // At the beginning we should have one infobar displayed for each extension. |
| 283 ASSERT_EQ(2, current_tab->infobar_delegate_count()); | 283 ASSERT_EQ(2, current_tab->infobar_delegate_count()); |
| 284 ReloadExtension(first_extension_id_); | 284 ReloadExtension(first_extension_id_); |
| 285 // One of the infobars should hide after the extension is reloaded. | 285 // One of the infobars should hide after the extension is reloaded. |
| 286 ASSERT_EQ(1, current_tab->infobar_delegate_count()); | 286 ASSERT_EQ(1, current_tab->infobar_delegate_count()); |
| 287 CheckExtensionConsistency(size_before); | 287 CheckExtensionConsistency(size_before); |
| 288 } | 288 } |
| 289 | 289 |
| 290 { | 290 { |
| 291 SCOPED_TRACE("second: infobar"); | 291 SCOPED_TRACE("second: infobar"); |
| 292 AcceptCrashedExtensionInfobar(0); | 292 AcceptCrashedExtensionInfobar(0); |
| 293 CheckExtensionConsistency(size_before); | 293 CheckExtensionConsistency(size_before); |
| 294 CheckExtensionConsistency(size_before + 1); | 294 CheckExtensionConsistency(size_before + 1); |
| 295 } | 295 } |
| 296 } | 296 } |
| OLD | NEW |