[protector] Support for collapsing multiple changes into a single one.

chrome/browser/protector/protector_service.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_PROTECTOR_PROTECTOR_SERVICE_H_
 #define CHROME_BROWSER_PROTECTOR_PROTECTOR_SERVICE_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 53 matching lines @@
 
   // Returns the most recent change instance or NULL if there are no changes.
   BaseSettingChange* GetLastChange();
 
   // Returns the Profile instance for this service.
   Profile* profile() { return profile_; }
 
  private:
   friend class ProtectorServiceTest;
 
-  // Pair of error and corresponding change instance. linked_ptr is used because
-  // Item instances are stored in a std::vector and must be copyable.
+  // Each item consists of an error and corresponding change instance.
+  // linked_ptr is used because Item instances are stored in a std::vector and
+  // must be copyable.
   struct Item {
     Item();
     ~Item();
     linked_ptr<BaseSettingChange> change;
     linked_ptr<SettingsChangeGlobalError> error;
+    // When true, this means |change| was merged with another instance and
+    // |error| is in process of being removed from GlobalErrorService.
+    bool was_merged;
+    // Meaningful only when |was_merged| is true. In that case, true means that
+    // the new merged GlobalError instance will be immediately shown.
+    bool show_when_merged;
   };
 
+  typedef std::vector<Item> Items;
+
   // Matches Item by |change| field.
   class MatchItemByChange {
    public:
     explicit MatchItemByChange(const BaseSettingChange* other);
 
     bool operator()(const Item& item);
 
    private:
     const BaseSettingChange* other_;
   };
 
   // Matches Item by |error| field.
   class MatchItemByError {
    public:
     explicit MatchItemByError(const SettingsChangeGlobalError* other);
 
     bool operator()(const Item& item);
 
    private:
     const SettingsChangeGlobalError* other_;
   };
 
+  // Returns an Item instance whose change can be merged with |change|, if any.
+  // Otherwise returns |NULL|. Provided that the merge strategy is transitive,
+  // there can be only one such instance.
+  Item* FindItemToMergeWith(const BaseSettingChange* change);
+
   // ProfileKeyedService implementation.
   virtual void Shutdown() OVERRIDE;
 
   // SettingsChangeGlobalErrorDelegate implementation.
   virtual void OnApplyChange(SettingsChangeGlobalError* error,
                              Browser* browser) OVERRIDE;
   virtual void OnDiscardChange(SettingsChangeGlobalError* error,
                                Browser* browser) OVERRIDE;
   virtual void OnDecisionTimeout(SettingsChangeGlobalError* error) OVERRIDE;
   virtual void OnRemovedFromProfile(SettingsChangeGlobalError* error) OVERRIDE;
 
   // Pointers to error bubble controllers and corresponding changes in the order
   // added.
-  std::vector<Item> items_;
+  Items items_;
 
   // Profile which settings we are protecting.
   Profile* profile_;
 
   // True if there is a change that has been shown and not yet accepted or
   // discarded by user.
   bool has_active_change_;
 
   // Observes changes to protected prefs and updates the backup.
   scoped_ptr<ProtectedPrefsWatcher> prefs_watcher_;
 
   DISALLOW_COPY_AND_ASSIGN(ProtectorService);
 };
 
 }  // namespace protector
 
 #endif  // CHROME_BROWSER_PROTECTOR_PROTECTOR_SERVICE_H_