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

Unified Diff: content/browser/webui/web_ui_data_source_impl.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/webui/web_ui_data_source_impl.cc
diff --git a/content/browser/webui/web_ui_data_source_impl.cc b/content/browser/webui/web_ui_data_source_impl.cc
index 1ab38d1b85b3a94fb8f338dd517198e6fa0c0b48..6fbe577b4d75951413ac657c71e68e2cb62b2f57 100644
--- a/content/browser/webui/web_ui_data_source_impl.cc
+++ b/content/browser/webui/web_ui_data_source_impl.cc
@@ -59,17 +59,14 @@ class WebUIDataSourceImpl::InternalDataSource : public URLDataSource {
InternalDataSource(WebUIDataSourceImpl* parent) : parent_(parent) {
}
- virtual ~InternalDataSource() {
- }
+ ~InternalDataSource() override {}
// URLDataSource implementation.
- virtual std::string GetSource() const override {
- return parent_->GetSource();
- }
- virtual std::string GetMimeType(const std::string& path) const override {
+ std::string GetSource() const override { return parent_->GetSource(); }
+ std::string GetMimeType(const std::string& path) const override {
return parent_->GetMimeType(path);
}
- virtual void StartDataRequest(
+ void StartDataRequest(
const std::string& path,
int render_process_id,
int render_frame_id,
@@ -77,26 +74,24 @@ class WebUIDataSourceImpl::InternalDataSource : public URLDataSource {
return parent_->StartDataRequest(path, render_process_id, render_frame_id,
callback);
}
- virtual bool ShouldReplaceExistingSource() const override {
+ bool ShouldReplaceExistingSource() const override {
return parent_->replace_existing_source_;
}
- virtual bool AllowCaching() const override {
- return false;
- }
- virtual bool ShouldAddContentSecurityPolicy() const override {
+ bool AllowCaching() const override { return false; }
+ bool ShouldAddContentSecurityPolicy() const override {
return parent_->add_csp_;
}
- virtual std::string GetContentSecurityPolicyObjectSrc() const override {
+ std::string GetContentSecurityPolicyObjectSrc() const override {
if (parent_->object_src_set_)
return parent_->object_src_;
return URLDataSource::GetContentSecurityPolicyObjectSrc();
}
- virtual std::string GetContentSecurityPolicyFrameSrc() const override {
+ std::string GetContentSecurityPolicyFrameSrc() const override {
if (parent_->frame_src_set_)
return parent_->frame_src_;
return URLDataSource::GetContentSecurityPolicyFrameSrc();
}
- virtual bool ShouldDenyXFrameOptions() const override {
+ bool ShouldDenyXFrameOptions() const override {
return parent_->deny_xframe_options_;
}
« no previous file with comments | « content/browser/webui/web_ui_data_source_impl.h ('k') | content/browser/webui/web_ui_data_source_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698