Index: chrome/browser/ui/webui/ntp/new_tab_ui_browsertest.cc |
diff --git a/chrome/browser/ui/webui/ntp/new_tab_ui_browsertest.cc b/chrome/browser/ui/webui/ntp/new_tab_ui_browsertest.cc |
index 8ffe5a7498ecadb9be46dd12587910f683638c3f..e51da86d05ba78c81ae4937dca54663c529f64f1 100644 |
--- a/chrome/browser/ui/webui/ntp/new_tab_ui_browsertest.cc |
+++ b/chrome/browser/ui/webui/ntp/new_tab_ui_browsertest.cc |
@@ -27,30 +27,28 @@ namespace { |
static bool had_console_errors = false; |
-bool HandleMessage(int severity, |
- const char* file, |
- int line, |
- size_t message_start, |
- const std::string& str) { |
- if (severity == logging::LOG_ERROR && file && file == std::string("CONSOLE")) |
- had_console_errors = true; |
- return false; |
-} |
+class MessageListener : logging::LogMessageListener { |
+ public: |
+ void OnMessage(int severity, const char* file, int line, |
+ size_t message_start, const std::string& str) override { |
+ if (severity == logging::LOG_ERROR && file && |
+ file == std::string("CONSOLE")) |
+ had_console_errors = true; |
+ } |
+}; |
} // namespace |
class NewTabUIBrowserTest : public InProcessBrowserTest { |
public: |
- NewTabUIBrowserTest() { |
- logging::SetLogMessageHandler(&HandleMessage); |
- } |
- |
- ~NewTabUIBrowserTest() override { logging::SetLogMessageHandler(NULL); } |
+ NewTabUIBrowserTest() {} |
void TearDown() override { |
InProcessBrowserTest::TearDown(); |
ASSERT_FALSE(had_console_errors); |
} |
+ private: |
+ MessageListener log_listener_; |
}; |
// Navigate to incognito NTP. Fails if there are console errors. |