Refactor favicon to use C++11 loop and fix style violation

components/favicon/core/browser/favicon_client.h

Index: components/favicon/core/browser/favicon_client.h
diff --git a/components/favicon/core/browser/favicon_client.h b/components/favicon/core/browser/favicon_client.h
index eeeeac030db9bb56a4f57d1e5310f9f28bf65c5f..d2c276e66687844efe6b691d2664c84c98754028 100644
--- a/components/favicon/core/browser/favicon_client.h
+++ b/components/favicon/core/browser/favicon_client.h
@@ -5,6 +5,7 @@
 #ifndef COMPONENTS_FAVICON_CORE_BROWSER_FAVICON_CLIENT_H_
 #define COMPONENTS_FAVICON_CORE_BROWSER_FAVICON_CLIENT_H_
 
+#include "base/macros.h"
 #include "components/keyed_service/core/keyed_service.h"
 
 class GURL;
@@ -13,10 +14,15 @@ class GURL;
 // e.g. Chrome.
 class FaviconClient : public KeyedService {
  public:
-  ~FaviconClient() override{};
-
   // Returns true if the specified URL is bookmarked.
   virtual bool IsBookmarked(const GURL& url) = 0;
+
+ protected:
+  FaviconClient() {}
+  ~FaviconClient() override {}
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(FaviconClient);
 };
 
 #endif  // COMPONENTS_FAVICON_CORE_BROWSER_FAVICON_CLIENT_H_