Provide UI for per-extension enabling/disabling of error collection.

chrome/browser/extensions/error_console/error_console.h

 // Copyright 2013 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_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_
 #define CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_
 
 #include <deque>
 #include <map>
 
@@ skipping 12 matching lines @@
 namespace content {
 class NotificationDetails;
 class NotificationSource;
 class RenderViewHost;
 }
 
 class ExtensionService;
 class Profile;
 
 namespace extensions {
-class ErrorConsoleUnitTest;
 class Extension;
+class ExtensionPrefs;
 
 // The ErrorConsole is a central object to which all extension errors are
 // reported. This includes errors detected in extensions core, as well as
 // runtime Javascript errors. If FeatureSwitch::error_console() is enabled these
 // errors can be viewed at chrome://extensions in developer mode.
 // This class is owned by ExtensionSystem, making it, in effect, a
 // BrowserContext-keyed service.
 class ErrorConsole : public content::NotificationObserver,
                      public ExtensionRegistryObserver {
  public:
@@ skipping 13 matching lines @@
   // Convenience method to return the ErrorConsole for a given profile.
   static ErrorConsole* Get(Profile* profile);
 
   // Set whether or not errors of the specified |type| are stored for the
   // extension with the given |extension_id|. This will be stored in the
   // preferences.
   void SetReportingForExtension(const std::string& extension_id,
                                 ExtensionError::Type type,
                                 bool enabled);
 
+  // Set whether or not errors of all types are stored for the extension with
+  // the given |extension_id|.
+  void SetReportingAllForExtension(const std::string& extension_id,
+                                           bool enabled);
+
+  // Returns true if reporting for either manifest or runtime errors is enabled
+  // for the extension with the given |extension_id|.
+  bool IsReportingEnabledForExtension(const std::string& extension_id) const;
+
   // Restore default reporting to the given extension.
   void UseDefaultReportingForExtension(const std::string& extension_id);
 
   // Report an extension error, and add it to the list.
   void ReportError(scoped_ptr<ExtensionError> error);
 
   // Get a collection of weak pointers to all errors relating to the extension
   // with the given |extension_id|.
   const ErrorList& GetErrorsForExtension(const std::string& extension_id) const;
 
@@ skipping 17 matching lines @@
   // Return the number of entries (extensions) in the error map.
   size_t get_num_entries_for_test() const { return errors_.size(); }
 
   // Set the default reporting for all extensions.
   void set_default_reporting_for_test(ExtensionError::Type type, bool enabled) {
     default_mask_ =
         enabled ? default_mask_ | (1 << type) : default_mask_ & ~(1 << type);
   }
 
  private:
-  // A map which stores the reporting preferences for each Extension. If there
-  // is no entry in the map, it signals that the |default_mask_| should be used.
-  typedef std::map<std::string, int32> ErrorPreferenceMap;
-
   // Checks whether or not the ErrorConsole should be enabled or disabled. If it
   // is in the wrong state, enables or disables it appropriately.
   void CheckEnabled();
 
   // Enable the error console for error collection and retention. This involves
   // subscribing to the appropriate notifications and fetching manifest errors.
   void Enable();
 
   // Disable the error console, removing the subscriptions to notifications and
   // removing all current errors.
@@ skipping 12 matching lines @@
                                  const Extension* extension) OVERRIDE;
 
   // Add manifest errors from an extension's install warnings.
   void AddManifestErrorsForExtension(const Extension* extension);
 
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
+  // Returns true if we should report the error of the given type for the
+  // extension.
+  bool ShouldReportErrorForExtension(const std::string& extension_id,
+                                     ExtensionError::Type type) const;
+
   // Whether or not the error console should record errors. This is true if
   // the user is in developer mode, and at least one of the following is true:
   // - The Chrome Apps Developer Tools are installed.
   // - FeatureSwitch::error_console() is enabled.
   // - This is a Dev Channel release.
   bool enabled_;
 
   // Needed because ObserverList is not thread-safe.
   base::ThreadChecker thread_checker_;
 
   // The list of all observers for the ErrorConsole.
   ObserverList<Observer> observers_;
 
   // The errors which we have received so far.
   ErrorMap errors_;
 
-  // The mapping of Extension's error-reporting preferences.
-  ErrorPreferenceMap pref_map_;
-
   // The default mask to use if an Extension does not have specific settings.
   int32 default_mask_;
 
   // The profile with which the ErrorConsole is associated. Only collect errors
   // from extensions and RenderViews associated with this Profile (and it's
   // incognito fellow).
   Profile* profile_;
 
+  // The ExtensionPrefs with which the ErrorConsole is associated. This weak
+  // pointer is safe because ErrorConsole is owned by ExtensionSystem, which
+  // is dependent on ExtensionPrefs.
+  ExtensionPrefs* prefs_;
+
   content::NotificationRegistrar notification_registrar_;
   PrefChangeRegistrar pref_registrar_;
 
   ScopedObserver<ExtensionRegistry, ExtensionRegistryObserver>
       registry_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(ErrorConsole);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_ERROR_CONSOLE_ERROR_CONSOLE_H_