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

Unified Diff: chrome/browser/extensions/component_loader_unittest.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/component_loader_unittest.cc
diff --git a/chrome/browser/extensions/component_loader_unittest.cc b/chrome/browser/extensions/component_loader_unittest.cc
index dbe01a17e36f6dcfe6d7c408282c0b3a77a4e4a8..9a8cfb841ed29a0a76a08272c0daa4e8307c32e7 100644
--- a/chrome/browser/extensions/component_loader_unittest.cc
+++ b/chrome/browser/extensions/component_loader_unittest.cc
@@ -36,33 +36,27 @@ class MockExtensionService : public TestExtensionService {
MockExtensionService() : ready_(false), unloaded_count_(0) {
}
- virtual void AddComponentExtension(const Extension* extension) override {
+ void AddComponentExtension(const Extension* extension) override {
EXPECT_FALSE(extension_set_.Contains(extension->id()));
// ExtensionService must become the owner of the extension object.
extension_set_.Insert(extension);
}
- virtual void UnloadExtension(
- const std::string& extension_id,
- UnloadedExtensionInfo::Reason reason) override {
+ void UnloadExtension(const std::string& extension_id,
+ UnloadedExtensionInfo::Reason reason) override {
ASSERT_TRUE(extension_set_.Contains(extension_id));
// Remove the extension with the matching id.
extension_set_.Remove(extension_id);
unloaded_count_++;
}
- virtual void RemoveComponentExtension(const std::string & extension_id)
- override {
+ void RemoveComponentExtension(const std::string& extension_id) override {
UnloadExtension(extension_id, UnloadedExtensionInfo::REASON_DISABLE);
}
- virtual bool is_ready() override {
- return ready_;
- }
+ bool is_ready() override { return ready_; }
- virtual const ExtensionSet* extensions() const override {
- return &extension_set_;
- }
+ const ExtensionSet* extensions() const override { return &extension_set_; }
void set_ready(bool ready) {
ready_ = ready;
« no previous file with comments | « chrome/browser/extensions/chrome_url_request_util.cc ('k') | chrome/browser/extensions/content_script_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698