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

Unified Diff: content/browser/renderer_host/render_process_host_browsertest.cc

Issue 667943003: Standardize usage of virtual/override/final in content/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: content/browser/renderer_host/render_process_host_browsertest.cc
diff --git a/content/browser/renderer_host/render_process_host_browsertest.cc b/content/browser/renderer_host/render_process_host_browsertest.cc
index d684d7860649041576beeb8b7dce3b1aba645c6a..f2856bfdc8667e2f99d746ccbe941e1f53801a96 100644
--- a/content/browser/renderer_host/render_process_host_browsertest.cc
+++ b/content/browser/renderer_host/render_process_host_browsertest.cc
@@ -42,12 +42,12 @@ class RenderProcessHostTest : public ContentBrowserTest,
protected:
// RenderProcessHostObserver:
- virtual void RenderProcessExited(RenderProcessHost* host,
- base::TerminationStatus status,
- int exit_code) override {
+ void RenderProcessExited(RenderProcessHost* host,
+ base::TerminationStatus status,
+ int exit_code) override {
++process_exits_;
}
- virtual void RenderProcessHostDestroyed(RenderProcessHost* host) override {
+ void RenderProcessHostDestroyed(RenderProcessHost* host) override {
++host_destructions_;
}
@@ -120,14 +120,14 @@ class ShellCloser : public RenderProcessHostObserver {
protected:
// RenderProcessHostObserver:
- virtual void RenderProcessExited(RenderProcessHost* host,
- base::TerminationStatus status,
- int exit_code) override {
+ void RenderProcessExited(RenderProcessHost* host,
+ base::TerminationStatus status,
+ int exit_code) override {
logging_string_->append("ShellCloser::RenderProcessExited ");
shell_->Close();
}
- virtual void RenderProcessHostDestroyed(RenderProcessHost* host) override {
+ void RenderProcessHostDestroyed(RenderProcessHost* host) override {
logging_string_->append("ShellCloser::RenderProcessHostDestroyed ");
}
@@ -144,13 +144,13 @@ class ObserverLogger : public RenderProcessHostObserver {
protected:
// RenderProcessHostObserver:
- virtual void RenderProcessExited(RenderProcessHost* host,
- base::TerminationStatus status,
- int exit_code) override {
+ void RenderProcessExited(RenderProcessHost* host,
+ base::TerminationStatus status,
+ int exit_code) override {
logging_string_->append("ObserverLogger::RenderProcessExited ");
}
- virtual void RenderProcessHostDestroyed(RenderProcessHost* host) override {
+ void RenderProcessHostDestroyed(RenderProcessHost* host) override {
logging_string_->append("ObserverLogger::RenderProcessHostDestroyed ");
host_destroyed_ = true;
}
« no previous file with comments | « content/browser/renderer_host/render_message_filter.cc ('k') | content/browser/renderer_host/render_process_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698