| OLD | NEW |
| 1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 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 "chrome/browser/extensions/error_console/error_console.h" | 5 #include "chrome/browser/extensions/error_console/error_console.h" |
| 6 | 6 |
| 7 #include "base/files/file_path.h" | 7 #include "base/files/file_path.h" |
| 8 #include "base/prefs/pref_service.h" | 8 #include "base/prefs/pref_service.h" |
| 9 #include "base/strings/string16.h" | 9 #include "base/strings/string16.h" |
| 10 #include "base/strings/stringprintf.h" | 10 #include "base/strings/stringprintf.h" |
| (...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 147 waiting_(false), | 147 waiting_(false), |
| 148 error_console_(error_console) { | 148 error_console_(error_console) { |
| 149 error_console_->AddObserver(this); | 149 error_console_->AddObserver(this); |
| 150 } | 150 } |
| 151 virtual ~ErrorObserver() { | 151 virtual ~ErrorObserver() { |
| 152 if (error_console_) | 152 if (error_console_) |
| 153 error_console_->RemoveObserver(this); | 153 error_console_->RemoveObserver(this); |
| 154 } | 154 } |
| 155 | 155 |
| 156 // ErrorConsole::Observer implementation. | 156 // ErrorConsole::Observer implementation. |
| 157 virtual void OnErrorAdded(const ExtensionError* error) override { | 157 void OnErrorAdded(const ExtensionError* error) override { |
| 158 ++errors_observed_; | 158 ++errors_observed_; |
| 159 if (errors_observed_ >= errors_expected_) { | 159 if (errors_observed_ >= errors_expected_) { |
| 160 if (waiting_) | 160 if (waiting_) |
| 161 base::MessageLoopForUI::current()->Quit(); | 161 base::MessageLoopForUI::current()->Quit(); |
| 162 } | 162 } |
| 163 } | 163 } |
| 164 | 164 |
| 165 virtual void OnErrorConsoleDestroyed() override { | 165 void OnErrorConsoleDestroyed() override { error_console_ = NULL; } |
| 166 error_console_ = NULL; | |
| 167 } | |
| 168 | 166 |
| 169 // Spin until the appropriate number of errors have been observed. | 167 // Spin until the appropriate number of errors have been observed. |
| 170 void WaitForErrors() { | 168 void WaitForErrors() { |
| 171 if (errors_observed_ < errors_expected_) { | 169 if (errors_observed_ < errors_expected_) { |
| 172 waiting_ = true; | 170 waiting_ = true; |
| 173 content::RunMessageLoop(); | 171 content::RunMessageLoop(); |
| 174 waiting_ = false; | 172 waiting_ = false; |
| 175 } | 173 } |
| 176 } | 174 } |
| 177 | 175 |
| (...skipping 14 matching lines...) Expand all Loading... |
| 192 ACTION_NAVIGATE, | 190 ACTION_NAVIGATE, |
| 193 // Simulate a browser action click. | 191 // Simulate a browser action click. |
| 194 ACTION_BROWSER_ACTION, | 192 ACTION_BROWSER_ACTION, |
| 195 // Navigate to the new tab page. | 193 // Navigate to the new tab page. |
| 196 ACTION_NEW_TAB, | 194 ACTION_NEW_TAB, |
| 197 // Do nothing (errors will be caused by a background script, | 195 // Do nothing (errors will be caused by a background script, |
| 198 // or by a manifest/loading warning). | 196 // or by a manifest/loading warning). |
| 199 ACTION_NONE | 197 ACTION_NONE |
| 200 }; | 198 }; |
| 201 | 199 |
| 202 virtual void SetUpInProcessBrowserTestFixture() override { | 200 void SetUpInProcessBrowserTestFixture() override { |
| 203 ExtensionBrowserTest::SetUpInProcessBrowserTestFixture(); | 201 ExtensionBrowserTest::SetUpInProcessBrowserTestFixture(); |
| 204 | 202 |
| 205 // We need to enable the ErrorConsole FeatureSwitch in order to collect | 203 // We need to enable the ErrorConsole FeatureSwitch in order to collect |
| 206 // errors. This should be enabled on any channel <= Dev, but let's make | 204 // errors. This should be enabled on any channel <= Dev, but let's make |
| 207 // sure (in case a test is running on, e.g., a beta channel). | 205 // sure (in case a test is running on, e.g., a beta channel). |
| 208 FeatureSwitch::error_console()->SetOverrideValue( | 206 FeatureSwitch::error_console()->SetOverrideValue( |
| 209 FeatureSwitch::OVERRIDE_ENABLED); | 207 FeatureSwitch::OVERRIDE_ENABLED); |
| 210 } | 208 } |
| 211 | 209 |
| 212 virtual void SetUpOnMainThread() override { | 210 void SetUpOnMainThread() override { |
| 213 ExtensionBrowserTest::SetUpOnMainThread(); | 211 ExtensionBrowserTest::SetUpOnMainThread(); |
| 214 | 212 |
| 215 // Errors are only kept if we have Developer Mode enabled. | 213 // Errors are only kept if we have Developer Mode enabled. |
| 216 profile()->GetPrefs()->SetBoolean(prefs::kExtensionsUIDeveloperMode, true); | 214 profile()->GetPrefs()->SetBoolean(prefs::kExtensionsUIDeveloperMode, true); |
| 217 | 215 |
| 218 error_console_ = ErrorConsole::Get(profile()); | 216 error_console_ = ErrorConsole::Get(profile()); |
| 219 CHECK(error_console_); | 217 CHECK(error_console_); |
| 220 | 218 |
| 221 test_data_dir_ = test_data_dir_.AppendASCII("error_console"); | 219 test_data_dir_ = test_data_dir_.AppendASCII("error_console"); |
| 222 } | 220 } |
| (...skipping 352 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 575 | 573 |
| 576 const StackTrace& stack_trace = GetStackTraceFromError(errors[0]); | 574 const StackTrace& stack_trace = GetStackTraceFromError(errors[0]); |
| 577 ASSERT_EQ(1u, stack_trace.size()); | 575 ASSERT_EQ(1u, stack_trace.size()); |
| 578 CheckStackFrame(stack_trace[0], | 576 CheckStackFrame(stack_trace[0], |
| 579 script_url, | 577 script_url, |
| 580 kAnonymousFunction, | 578 kAnonymousFunction, |
| 581 12u, 20u); | 579 12u, 20u); |
| 582 } | 580 } |
| 583 | 581 |
| 584 } // namespace extensions | 582 } // namespace extensions |
| OLD | NEW |