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

Unified Diff: chrome/browser/extensions/api/alarms/alarm_manager.cc

Issue 666153002: Standardize usage of virtual/override/final in chrome/browser/extensions/ (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/extensions/api/alarms/alarm_manager.cc
diff --git a/chrome/browser/extensions/api/alarms/alarm_manager.cc b/chrome/browser/extensions/api/alarms/alarm_manager.cc
index 98705a820bcff009cf7f7224a9aa2dc6f694c4ff..bf688fcbae76e5c483c4c86e5af5bc2e7c63a0e7 100644
--- a/chrome/browser/extensions/api/alarms/alarm_manager.cc
+++ b/chrome/browser/extensions/api/alarms/alarm_manager.cc
@@ -39,10 +39,9 @@ class DefaultAlarmDelegate : public AlarmManager::Delegate {
public:
explicit DefaultAlarmDelegate(content::BrowserContext* context)
: browser_context_(context) {}
- virtual ~DefaultAlarmDelegate() {}
+ ~DefaultAlarmDelegate() override {}
- virtual void OnAlarm(const std::string& extension_id,
- const Alarm& alarm) override {
+ void OnAlarm(const std::string& extension_id, const Alarm& alarm) override {
scoped_ptr<base::ListValue> args(new base::ListValue());
args->Append(alarm.js_alarm->ToValue().release());
scoped_ptr<Event> event(new Event(alarms::OnAlarm::kEventName,
« no previous file with comments | « chrome/browser/extensions/api/alarms/alarm_manager.h ('k') | chrome/browser/extensions/api/alarms/alarms_api.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698