ProtectorService supports multiple change instances.

chrome/browser/protector/settings_change_global_error.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_SETTINGS_CHANGE_GLOBAL_ERROR_H_
 #define CHROME_BROWSER_PROTECTOR_SETTINGS_CHANGE_GLOBAL_ERROR_H_
 #pragma once
 
+#include <bitset>
+
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
+#include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/protector/base_setting_change.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/global_error.h"
 
 class Browser;
 class Profile;
 
 namespace protector {
 
 class BaseSettingChange;
@@ skipping 10 matching lines @@
                             SettingsChangeGlobalErrorDelegate* delegate);
   virtual ~SettingsChangeGlobalError();
 
   // Displays a global error bubble for the given browser profile.
   // Can be called from any thread.
   void ShowForProfile(Profile* profile);
 
   // Removes global error from its profile.
   void RemoveFromProfile();
 
+  // Returns the change instance to which this error refers.
+  BaseSettingChange* change() { return change_; }
+
  private:
   // GlobalError implementation.
   virtual bool HasBadge() OVERRIDE;
   virtual int GetBadgeResourceID() OVERRIDE;
   virtual bool HasMenuItem() OVERRIDE;
   virtual int MenuItemCommandID() OVERRIDE;
   virtual string16 MenuItemLabel() OVERRIDE;
   virtual int MenuItemIconResourceID() OVERRIDE;
   virtual void ExecuteMenuItem(Browser* browser) OVERRIDE;
   virtual bool HasBubbleView() OVERRIDE;
@@ skipping 37 matching lines @@
   Profile* profile_;
 
   // True if user has dismissed the bubble by clicking on one of the buttons.
   bool closed_by_button_;
 
   // True if the bubble has to be shown on the next browser window activation.
   bool show_on_browser_activation_;
 
   base::WeakPtrFactory<SettingsChangeGlobalError> weak_factory_;
 
+  // Menu command ID assigned to |this| from the pool of available IDs.
+  int menu_id_;
+
+  // Unset bits indicate available command IDs.
+  static std::bitset<IDC_SHOW_SETTINGS_CHANGE_LAST -

[whywhat, 2012/02/29 18:55:14]

Using statis non-elementary types members is also forbidden. Could be a pointer
to bitset though.
Will static bitset work with multiple-profiles?

[Ivan Korotkov, 2012/02/29 19:23:44]

As discussed, it's used application-wide for vectors/maps.

+                     IDC_SHOW_SETTINGS_CHANGE_FIRST + 1> menu_ids_;
+
   DISALLOW_COPY_AND_ASSIGN(SettingsChangeGlobalError);
 };
 
 }  // namespace protector
 
 #endif  // CHROME_BROWSER_PROTECTOR_SETTINGS_CHANGE_GLOBAL_ERROR_H_