[New Tab Page] Change the mechanism to intercept online NTP errors

chrome/browser/browsing_data/browsing_data_remover_browsertest.cc

Index: chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc b/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
index 76246c938034c160719650570ca983f0d185f21d..110c28bae8045167728511c08c0567f1bd9902b9 100644
--- a/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
+++ b/chrome/browser/browsing_data/browsing_data_remover_browsertest.cc
@@ -29,8 +29,8 @@ using content::BrowserThread;
 
 namespace {
 void SetUrlRequestMock(const base::FilePath& path) {
-  net::URLRequestMockHTTPJob::AddUrlHandler(path,
-                                            BrowserThread::GetBlockingPool());
+  net::URLRequestMockHTTPJob::AddUrlHandlers(path,
+                                             BrowserThread::GetBlockingPool());

[mmenke, 2015/01/15 22:35:30]

Optional:  While you're here, could break the dependency on how mock handlers
are installed by using SetUrlRequestMocksEnabled(true) here instead.  This file
will still need to depend on URLRequestMockHTTPJob, though.  Still think it's an
improvement, though.

Could do it for PolicyBrowserTest as well, but not for the content/ tests.

[Mathieu, 2015/01/22 19:21:58]

Actually both this test and the PolicyBrowserTest need to serve from
content::DIR_TEST_DATA.

 }
 }