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

Unified Diff: chrome/common/notification_source.h

Issue 384103: This patch is the rest of the lint cleanup minus the other open issue. (Closed)
Patch Set: Created 11 years, 1 month 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
« no previous file with comments | « chrome/common/notification_details.h ('k') | chrome/common/notification_type.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/notification_source.h
diff --git a/chrome/common/notification_source.h b/chrome/common/notification_source.h
index fb371241fdbdcdeaf59f1cbe8d3e8e87d0821355..5cdedb188e597ff6f1cb85c56857499590484a18 100644
--- a/chrome/common/notification_source.h
+++ b/chrome/common/notification_source.h
@@ -31,7 +31,7 @@ class NotificationSource {
}
protected:
- NotificationSource(const void* ptr) : ptr_(ptr) {}
+ explicit NotificationSource(const void* ptr) : ptr_(ptr) {}
// Declaring this const allows Source<T> to be used with both T = Foo and
// T = const Foo.
@@ -41,9 +41,9 @@ class NotificationSource {
template <class T>
class Source : public NotificationSource {
public:
- Source(T* ptr) : NotificationSource(ptr) {}
+ explicit Source(T* ptr) : NotificationSource(ptr) {}
- Source(const NotificationSource& other)
+ explicit Source(const NotificationSource& other)
: NotificationSource(other) {}
T* operator->() const { return ptr(); }
« no previous file with comments | « chrome/common/notification_details.h ('k') | chrome/common/notification_type.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698