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

Side by Side Diff: chrome/browser/task_manager/web_contents_information.h

Issue 663023009: Standardize usage of virtual/override/final in chrome/browser/task_manager/ (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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_TASK_MANAGER_WEB_CONTENTS_INFORMATION_H_ 5 #ifndef CHROME_BROWSER_TASK_MANAGER_WEB_CONTENTS_INFORMATION_H_
6 #define CHROME_BROWSER_TASK_MANAGER_WEB_CONTENTS_INFORMATION_H_ 6 #define CHROME_BROWSER_TASK_MANAGER_WEB_CONTENTS_INFORMATION_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/callback.h" 9 #include "base/callback.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 }; 54 };
55 55
56 // Implements the observer methods (StartObservingCreation() / 56 // Implements the observer methods (StartObservingCreation() /
57 // StopObservingCreation()) of WebContentsInformation using 57 // StopObservingCreation()) of WebContentsInformation using
58 // NOTIFICATION_WEB_CONTENTS_CONNECTED. 58 // NOTIFICATION_WEB_CONTENTS_CONNECTED.
59 class NotificationObservingWebContentsInformation 59 class NotificationObservingWebContentsInformation
60 : public WebContentsInformation, 60 : public WebContentsInformation,
61 public content::NotificationObserver { 61 public content::NotificationObserver {
62 public: 62 public:
63 NotificationObservingWebContentsInformation(); 63 NotificationObservingWebContentsInformation();
64 virtual ~NotificationObservingWebContentsInformation(); 64 ~NotificationObservingWebContentsInformation() override;
65 65
66 // WebContentsInformation partial implementation. 66 // WebContentsInformation partial implementation.
67 virtual void StartObservingCreation(const NewWebContentsCallback& callback) 67 void StartObservingCreation(const NewWebContentsCallback& callback) override;
68 override; 68 void StopObservingCreation() override;
69 virtual void StopObservingCreation() override;
70 69
71 // content::NotificationObserver implementation. 70 // content::NotificationObserver implementation.
72 virtual void Observe(int type, 71 void Observe(int type,
73 const content::NotificationSource& source, 72 const content::NotificationSource& source,
74 const content::NotificationDetails& details) override; 73 const content::NotificationDetails& details) override;
75 74
76 private: 75 private:
77 content::NotificationRegistrar registrar_; 76 content::NotificationRegistrar registrar_;
78 NewWebContentsCallback observer_callback_; 77 NewWebContentsCallback observer_callback_;
79 DISALLOW_COPY_AND_ASSIGN(NotificationObservingWebContentsInformation); 78 DISALLOW_COPY_AND_ASSIGN(NotificationObservingWebContentsInformation);
80 }; 79 };
81 80
82 } // namespace task_manager 81 } // namespace task_manager
83 82
84 #endif // CHROME_BROWSER_TASK_MANAGER_WEB_CONTENTS_INFORMATION_H_ 83 #endif // CHROME_BROWSER_TASK_MANAGER_WEB_CONTENTS_INFORMATION_H_
OLDNEW
« no previous file with comments | « chrome/browser/task_manager/task_manager_unittest.cc ('k') | chrome/browser/task_manager/web_contents_resource_provider.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698