New post-sideload UI:

chrome/browser/ui/global_error/global_error.h

Index: chrome/browser/ui/global_error/global_error.h
diff --git a/chrome/browser/ui/global_error/global_error.h b/chrome/browser/ui/global_error/global_error.h
index f80e55ddce27256bd39832cd27d88dec85990fc3..f38372510e53f92425f2ab697ee1bbda99196e09 100644
--- a/chrome/browser/ui/global_error/global_error.h
+++ b/chrome/browser/ui/global_error/global_error.h
@@ -15,6 +15,14 @@ class GlobalErrorBubbleViewBase;
 // This object describes a single global error.
 class GlobalError : public base::SupportsWeakPtr<GlobalError> {
  public:
+  enum Severity {
+    SEVERITY_LOW,
+    SEVERITY_MEDIUM,
+    SEVERITY_HIGH,
+    SEVERITY_CRITICAL,
+    NUM_SEVERITY,

[sail, 2012/10/16 21:26:32]

It seems like this isn't used anywhere. Can we remove it?
In general, it's better not to have this. This allows us to have switch
statements without having to add a NOTREACHED() clause.

[Matt Perry, 2012/10/16 21:36:31]

Done.

+  };
+
   GlobalError();
   virtual ~GlobalError();
 
@@ -22,6 +30,10 @@ class GlobalError : public base::SupportsWeakPtr<GlobalError> {
   virtual bool HasBadge() = 0;
   // Returns the resource ID of the badge icon.
   virtual int GetBadgeResourceID();
+  // Returns the error's severity level. If there are multiple errors,
+  // the error with the highest severity will display in the menu. If not
+  // overridden, this is based on the badge resource ID.
+  virtual Severity GetSeverity();
 
   // Returns true if a menu item should be added to the wrench menu.
   virtual bool HasMenuItem() = 0;