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

Unified Diff: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc

Issue 648653003: Standardize usage of virtual/override/final in chrome/browser/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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: chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc
diff --git a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc
index db468689504a5a1713ed9d7d780fb388d4293726..e504412d4ac7666c820d9d50b5ae642c368400c6 100644
--- a/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc
+++ b/chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc
@@ -57,9 +57,9 @@ class TestDispatcherHostDelegate : public ChromeResourceDispatcherHostDelegate {
: ChromeResourceDispatcherHostDelegate(prerender_tracker) {
}
- virtual ~TestDispatcherHostDelegate() {}
+ ~TestDispatcherHostDelegate() override {}
- virtual void RequestBeginning(
+ void RequestBeginning(
net::URLRequest* request,
content::ResourceContext* resource_context,
content::AppCacheService* appcache_service,
@@ -74,11 +74,10 @@ class TestDispatcherHostDelegate : public ChromeResourceDispatcherHostDelegate {
request_headers_.MergeFrom(request->extra_request_headers());
}
- virtual void OnRequestRedirected(
- const GURL& redirect_url,
- net::URLRequest* request,
- content::ResourceContext* resource_context,
- content::ResourceResponse* response) override {
+ void OnRequestRedirected(const GURL& redirect_url,
+ net::URLRequest* request,
+ content::ResourceContext* resource_context,
+ content::ResourceResponse* response) override {
ChromeResourceDispatcherHostDelegate::OnRequestRedirected(
redirect_url,
request,
@@ -100,7 +99,7 @@ class ChromeResourceDispatcherHostDelegateBrowserTest :
public:
ChromeResourceDispatcherHostDelegateBrowserTest() {}
- virtual void SetUpOnMainThread() override {
+ void SetUpOnMainThread() override {
InProcessBrowserTest::SetUpOnMainThread();
// Hook navigations with our delegate.
dispatcher_host_delegate_.reset(new TestDispatcherHostDelegate(
@@ -130,7 +129,7 @@ class ChromeResourceDispatcherHostDelegateBrowserTest :
}
}
- virtual void TearDownOnMainThread() override {
+ void TearDownOnMainThread() override {
content::ResourceDispatcherHost::Get()->SetDelegate(NULL);
dispatcher_host_delegate_.reset();
}

Powered by Google App Engine
This is Rietveld 408576698