Revert of [Extensions] Don't count bubble dismissal from focus loss as acknowledgment

chrome/browser/extensions/extension_message_bubble_controller.h

Index: chrome/browser/extensions/extension_message_bubble_controller.h
diff --git a/chrome/browser/extensions/extension_message_bubble_controller.h b/chrome/browser/extensions/extension_message_bubble_controller.h
index c3059c53efdcc9985387dd47ed2fc30de6e26d90..40e5b81bead2fba0d851d0c343d549b96ea3a94c 100644
--- a/chrome/browser/extensions/extension_message_bubble_controller.h
+++ b/chrome/browser/extensions/extension_message_bubble_controller.h
@@ -24,9 +24,8 @@
   enum BubbleAction {
     ACTION_LEARN_MORE = 0,
     ACTION_EXECUTE,
-    ACTION_DISMISS_USER_ACTION,
-    ACTION_DISMISS_DEACTIVATION,
-    ACTION_BOUNDARY,  // Must be the last value.
+    ACTION_DISMISS,
+    ACTION_BOUNDARY, // Must be the last value.
   };
 
   class Delegate {
@@ -75,22 +74,14 @@
     virtual void LogExtensionCount(size_t count) = 0;
     virtual void LogAction(BubbleAction action) = 0;
 
-    // Returns a key unique to the type of bubble that can be used to retrieve
-    // state specific to the type (e.g., shown for profiles).
-    virtual const char* GetKey() = 0;
+    // Has the user acknowledged info about the extension the bubble reports.
+    virtual bool HasBubbleInfoBeenAcknowledged(const std::string& extension_id);
+    virtual void SetBubbleInfoBeenAcknowledged(const std::string& extension_id,
+                                               bool value);
 
-    // Whether the "shown for profiles" set should be cleared if an action is
-    // taken on the bubble. This defaults to true, since once an action is
-    // taken, the extension will usually either be acknowledged or removed, and
-    // the bubble won't show for that extension.
-    // This should be false in cases where there is no acknowledgment option
-    // (as in the developer-mode extension warning).
-    virtual bool ClearProfileSetAfterAction();
-
-    // Has the user acknowledged info about the extension the bubble reports.
-    bool HasBubbleInfoBeenAcknowledged(const std::string& extension_id);
-    void SetBubbleInfoBeenAcknowledged(const std::string& extension_id,
-                                       bool value);
+    // Returns the set of profiles for which this bubble has been shown.
+    // If profiles are not tracked, returns null (default).
+    virtual std::set<Profile*>* GetProfileSet();
 
    protected:
     Profile* profile() { return profile_; }
@@ -150,10 +141,8 @@
 
   // Callbacks from bubble. Declared virtual for testing purposes.
   virtual void OnBubbleAction();
-  virtual void OnBubbleDismiss(bool dismissed_by_deactivation);
+  virtual void OnBubbleDismiss();
   virtual void OnLinkClicked();
-
-  void ClearProfileListForTesting();
 
   static void set_should_ignore_learn_more_for_testing(
       bool should_ignore_learn_more);
@@ -167,8 +156,6 @@
 
   // Performs cleanup after the bubble closes.
   void OnClose();
-
-  std::set<Profile*>* GetProfileSet();
 
   // A weak pointer to the Browser we are associated with. Not owned by us.
   Browser* browser_;