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

Unified Diff: components/favicon/core/favicon_driver.h

Issue 227153007: Remove dependency on Profile's IsOffTheRecord in favicon_handler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@FAVICON_create_core_dir
Patch Set: Created 6 years, 8 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: components/favicon/core/favicon_driver.h
diff --git a/components/favicon/core/favicon_driver.h b/components/favicon/core/favicon_driver.h
new file mode 100644
index 0000000000000000000000000000000000000000..fe39982de6e1d55234fdaed803fb86229b830ccc
--- /dev/null
+++ b/components/favicon/core/favicon_driver.h
@@ -0,0 +1,26 @@
+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
blundell 2014/04/08 10:01:19 nit: kill (c).
jif 2014/04/08 11:55:47 Done.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_FAVICON_CORE_FAVICON_DRIVER_H_
+#define COMPONENTS_FAVICON_CORE_FAVICON_DRIVER_H_
+
+#include "base/macros.h"
+
+class Profile;
+
+// Interface that allows Favicon core code to interact with its driver
+// (i.e., obtain information from it and give information to it).
+class FaviconDriver {
+ public:
+ FaviconDriver() {}
+ virtual ~FaviconDriver() {}
+
+ // If this browsing session should not be persisted.
blundell 2014/04/08 10:01:19 Returns whether the user is operating in an off-th
jif 2014/04/08 11:55:47 Done.
+ virtual bool IsOffTheRecord() = 0;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(FaviconDriver);
+};
+
+#endif // COMPONENTS_FAVICON_CORE_FAVICON_DRIVER_H_

Powered by Google App Engine
This is Rietveld 408576698