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

Side by Side Diff: chrome/browser/background/background_application_list_model.h

Issue 648653003: Standardize usage of virtual/override/final in chrome/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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_BACKGROUND_BACKGROUND_APPLICATION_LIST_MODEL_H_ 5 #ifndef CHROME_BROWSER_BACKGROUND_BACKGROUND_APPLICATION_LIST_MODEL_H_
6 #define CHROME_BROWSER_BACKGROUND_BACKGROUND_APPLICATION_LIST_MODEL_H_ 6 #define CHROME_BROWSER_BACKGROUND_BACKGROUND_APPLICATION_LIST_MODEL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 10
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 // it no longer detects a previously known extension. 42 // it no longer detects a previously known extension.
43 virtual void OnApplicationListChanged(Profile* profile); 43 virtual void OnApplicationListChanged(Profile* profile);
44 44
45 protected: 45 protected:
46 virtual ~Observer(); 46 virtual ~Observer();
47 }; 47 };
48 48
49 // Create a new model associated with profile. 49 // Create a new model associated with profile.
50 explicit BackgroundApplicationListModel(Profile* profile); 50 explicit BackgroundApplicationListModel(Profile* profile);
51 51
52 virtual ~BackgroundApplicationListModel(); 52 ~BackgroundApplicationListModel() override;
53 53
54 // Associate observer with this model. 54 // Associate observer with this model.
55 void AddObserver(Observer* observer); 55 void AddObserver(Observer* observer);
56 56
57 // Return the icon associated with |extension| or NULL. NULL indicates either 57 // Return the icon associated with |extension| or NULL. NULL indicates either
58 // that there is no icon associated with the extension, or that a pending 58 // that there is no icon associated with the extension, or that a pending
59 // task to retrieve the icon has not completed. See the Observer class above. 59 // task to retrieve the icon has not completed. See the Observer class above.
60 // 60 //
61 // NOTE: The model manages the ImageSkia result, that is it "owns" the memory, 61 // NOTE: The model manages the ImageSkia result, that is it "owns" the memory,
62 // releasing it if the associated background application is unloaded. 62 // releasing it if the associated background application is unloaded.
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 void DissociateApplicationData(const extensions::Extension* extension); 109 void DissociateApplicationData(const extensions::Extension* extension);
110 110
111 // Returns the Application associated with |extension| or NULL. 111 // Returns the Application associated with |extension| or NULL.
112 const Application* FindApplication( 112 const Application* FindApplication(
113 const extensions::Extension* extension) const; 113 const extensions::Extension* extension) const;
114 114
115 // Returns the Application associated with |extension| or NULL. 115 // Returns the Application associated with |extension| or NULL.
116 Application* FindApplication(const extensions::Extension* extension); 116 Application* FindApplication(const extensions::Extension* extension);
117 117
118 // content::NotificationObserver implementation. 118 // content::NotificationObserver implementation.
119 virtual void Observe(int type, 119 void Observe(int type,
120 const content::NotificationSource& source, 120 const content::NotificationSource& source,
121 const content::NotificationDetails& details) override; 121 const content::NotificationDetails& details) override;
122 122
123 // Notifies observers that some of the data associated with this background 123 // Notifies observers that some of the data associated with this background
124 // application, e. g. the Icon, has changed. 124 // application, e. g. the Icon, has changed.
125 void SendApplicationDataChangedNotifications( 125 void SendApplicationDataChangedNotifications(
126 const extensions::Extension* extension); 126 const extensions::Extension* extension);
127 127
128 // Notifies observers that at least one background application has been added 128 // Notifies observers that at least one background application has been added
129 // or removed. 129 // or removed.
130 void SendApplicationListChangedNotifications(); 130 void SendApplicationListChangedNotifications();
131 131
(...skipping 23 matching lines...) Expand all
155 extensions::ExtensionList extensions_; 155 extensions::ExtensionList extensions_;
156 ObserverList<Observer, true> observers_; 156 ObserverList<Observer, true> observers_;
157 Profile* profile_; 157 Profile* profile_;
158 content::NotificationRegistrar registrar_; 158 content::NotificationRegistrar registrar_;
159 bool ready_; 159 bool ready_;
160 160
161 DISALLOW_COPY_AND_ASSIGN(BackgroundApplicationListModel); 161 DISALLOW_COPY_AND_ASSIGN(BackgroundApplicationListModel);
162 }; 162 };
163 163
164 #endif // CHROME_BROWSER_BACKGROUND_BACKGROUND_APPLICATION_LIST_MODEL_H_ 164 #endif // CHROME_BROWSER_BACKGROUND_BACKGROUND_APPLICATION_LIST_MODEL_H_
OLDNEW
« no previous file with comments | « chrome/browser/autofill/personal_data_manager_factory.h ('k') | chrome/browser/background/background_contents_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698