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

Unified Diff: chrome/browser/ui/webui/app_launcher_page_ui.h

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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/ui/webui/app_launcher_page_ui.h
diff --git a/chrome/browser/ui/webui/app_launcher_page_ui.h b/chrome/browser/ui/webui/app_launcher_page_ui.h
index 4b6310e93c1ea8f43c591f3caa7d51811d230e8b..8b1a4459e0f70296e04c5cb275c432d279b4e134 100644
--- a/chrome/browser/ui/webui/app_launcher_page_ui.h
+++ b/chrome/browser/ui/webui/app_launcher_page_ui.h
@@ -19,7 +19,7 @@ class RefCountedMemory;
class AppLauncherPageUI : public content::WebUIController {
public:
explicit AppLauncherPageUI(content::WebUI* web_ui);
- virtual ~AppLauncherPageUI();
+ ~AppLauncherPageUI() override;
static base::RefCountedMemory* GetFaviconResourceBytes(
ui::ScaleFactor scale_factor);
@@ -28,18 +28,18 @@ class AppLauncherPageUI : public content::WebUIController {
class HTMLSource : public content::URLDataSource {
public:
explicit HTMLSource(Profile* profile);
- virtual ~HTMLSource();
+ ~HTMLSource() override;
// content::URLDataSource implementation.
- virtual std::string GetSource() const override;
- virtual void StartDataRequest(
+ std::string GetSource() const override;
+ void StartDataRequest(
const std::string& path,
int render_process_id,
int render_frame_id,
const content::URLDataSource::GotDataCallback& callback) override;
- virtual std::string GetMimeType(const std::string&) const override;
- virtual bool ShouldReplaceExistingSource() const override;
- virtual bool ShouldAddContentSecurityPolicy() const override;
+ std::string GetMimeType(const std::string&) const override;
+ bool ShouldReplaceExistingSource() const override;
+ bool ShouldAddContentSecurityPolicy() const override;
private:

Powered by Google App Engine
This is Rietveld 408576698