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

Unified Diff: content/browser/battery_status/battery_status_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/battery_status/battery_status_browsertest.cc
diff --git a/content/browser/battery_status/battery_status_browsertest.cc b/content/browser/battery_status/battery_status_browsertest.cc
index b2e90e7c67c71c01e74b28de91930eaf4a5e2953..880e62a8bdecabe063cb7fe4031393a4b5094a4a 100644
--- a/content/browser/battery_status/battery_status_browsertest.cc
+++ b/content/browser/battery_status/battery_status_browsertest.cc
@@ -23,17 +23,17 @@ class FakeBatteryManager : public BatteryStatusManager {
explicit FakeBatteryManager(
const BatteryStatusService::BatteryUpdateCallback& callback)
: callback_(callback), battery_status_available_(true), started_(false) {}
- virtual ~FakeBatteryManager() { }
+ ~FakeBatteryManager() override {}
// Methods from BatteryStatusManager.
- virtual bool StartListeningBatteryChange() override {
+ bool StartListeningBatteryChange() override {
started_ = true;
if (battery_status_available_)
InvokeUpdateCallback();
return battery_status_available_;
}
- virtual void StopListeningBatteryChange() override { }
+ void StopListeningBatteryChange() override {}
void InvokeUpdateCallback() {
callback_.Run(status_);
@@ -68,12 +68,12 @@ class BatteryStatusBrowserTest : public ContentBrowserTest {
io_loop_finished_event_(false, false) {
}
- virtual void SetUpCommandLine(CommandLine* command_line) override {
+ void SetUpCommandLine(CommandLine* command_line) override {
command_line->AppendSwitch(
switches::kEnableExperimentalWebPlatformFeatures);
}
- virtual void SetUpOnMainThread() override {
+ void SetUpOnMainThread() override {
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
base::Bind(&BatteryStatusBrowserTest::SetUpOnIOThread, this));
« no previous file with comments | « content/browser/appcache/view_appcache_internals_job.cc ('k') | content/browser/battery_status/battery_status_manager_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698