Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(440)

Unified Diff: components/test_runner/web_frame_test_client.cc

Issue 2425663002: Add an error page for resources blocked via XSS Auditor. (Closed)
Patch Set: frameNavigation test. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/test_runner/web_frame_test_client.cc
diff --git a/components/test_runner/web_frame_test_client.cc b/components/test_runner/web_frame_test_client.cc
index f155dd223e68660d01f0d7256581ad69d3c3882d..5a49332a2644d1a3af74982a0709b54035c8b7bb 100644
--- a/components/test_runner/web_frame_test_client.cc
+++ b/components/test_runner/web_frame_test_client.cc
@@ -23,6 +23,7 @@
#include "components/test_runner/web_test_delegate.h"
#include "components/test_runner/web_view_test_proxy.h"
#include "components/test_runner/web_widget_test_proxy.h"
+#include "net/base/net_errors.h"
#include "third_party/WebKit/public/platform/WebString.h"
#include "third_party/WebKit/public/platform/WebURL.h"
#include "third_party/WebKit/public/platform/WebURLRequest.h"
@@ -397,6 +398,13 @@ void WebFrameTestClient::loadURLExternally(
}
}
+void WebFrameTestClient::loadErrorPage(int reason) {
+ if (test_runner()->shouldDumpFrameLoadCallbacks()) {
+ delegate_->PrintMessage(base::StringPrintf(
+ "- loadErrorPage: %s\n", net::ErrorToString(reason).c_str()));
+ }
+}
+
void WebFrameTestClient::didStartProvisionalLoad(blink::WebLocalFrame* frame,
double trigering_event_time) {
test_runner()->tryToSetTopLoadingFrame(frame);

Powered by Google App Engine
This is Rietveld 408576698