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

Unified Diff: chrome/browser/extensions/suspicious_extension_bubble_controller.h

Issue 95133002: Add an extension bubble explaining which extensions are in dev mode. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to head Created 7 years 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/suspicious_extension_bubble_controller.h
diff --git a/chrome/browser/extensions/suspicious_extension_bubble_controller.h b/chrome/browser/extensions/suspicious_extension_bubble_controller.h
index 304f7c85bb7154d19ec72b734b89c22a82beb960..6ae75a238461e8d232cc07b85437b76a1679a108 100644
--- a/chrome/browser/extensions/suspicious_extension_bubble_controller.h
+++ b/chrome/browser/extensions/suspicious_extension_bubble_controller.h
@@ -7,6 +7,7 @@
#include <string>
#include "chrome/browser/extensions/api/profile_keyed_api_factory.h"
+#include "chrome/browser/extensions/extension_message_bubble_controller.h"
#include "extensions/common/extension.h"
class Browser;
@@ -16,7 +17,10 @@ namespace extensions {
class SuspiciousExtensionBubble;
-class SuspiciousExtensionBubbleController : public ProfileKeyedAPI {
+class SuspiciousExtensionBubbleController
+ : public ProfileKeyedAPI,
+ public ExtensionMessageBubbleController,
+ public ExtensionMessageBubbleController::Delegate {
public:
explicit SuspiciousExtensionBubbleController(Profile* profile);
virtual ~SuspiciousExtensionBubbleController();
@@ -25,8 +29,8 @@ class SuspiciousExtensionBubbleController : public ProfileKeyedAPI {
static ProfileKeyedAPIFactory<
SuspiciousExtensionBubbleController>* GetFactoryInstance();
- // Convenience method to get the SuspiciousExtensionBubbleController for a
- // profile.
+ // Convenience method to get the SuspiciousExtensionBubbleController
+ // for a profile.
static SuspiciousExtensionBubbleController* Get(Profile* profile);
// Check for suspicious extensions, returns true if found.
@@ -50,7 +54,8 @@ class SuspiciousExtensionBubbleController : public ProfileKeyedAPI {
virtual void OnLinkClicked();
private:
- friend class ProfileKeyedAPIFactory<SuspiciousExtensionBubbleController>;
+ friend class ProfileKeyedAPIFactory<
+ SuspiciousExtensionBubbleController>;
// ProfileKeyedAPI implementation.
static const char* service_name() {
@@ -58,24 +63,12 @@ class SuspiciousExtensionBubbleController : public ProfileKeyedAPI {
}
static const bool kServiceRedirectedInIncognito = true;
- // Mark all extensions found as acknowledged (don't need to warn about them
- // again).
- void AcknowledgeWipeout();
-
- // The list of suspicious extensions found. Reset at the beginning of each
- // call to FoundSuspiciousExtensions.
- ExtensionIdList suspicious_extensions_;
-
// Our extension service. Weak, not owned by us.
ExtensionService* service_;
// A weak pointer to the profile we are associated with. Not owned by us.
Profile* profile_;
- // This object only checks once for suspicious extensions because the dataset
- // doesn't change after startup.
- bool has_notified_;
-
DISALLOW_COPY_AND_ASSIGN(SuspiciousExtensionBubbleController);
};

Powered by Google App Engine
This is Rietveld 408576698