Auto select the connection tab of the Website Settings UI in case of an https error or mixed content

chrome/browser/ui/website_settings/website_settings_ui.h

Index: chrome/browser/ui/website_settings/website_settings_ui.h
diff --git a/chrome/browser/ui/website_settings/website_settings_ui.h b/chrome/browser/ui/website_settings/website_settings_ui.h
index c6a9d220a423aa264a555f2c60a624a26954a6f2..5d3ff2c2a887c3717daeea74befac12eab1a94a3 100644
--- a/chrome/browser/ui/website_settings/website_settings_ui.h
+++ b/chrome/browser/ui/website_settings/website_settings_ui.h
@@ -34,6 +34,14 @@ class Image;
 // etc.).
 class WebsiteSettingsUI {
  public:
+  // The Website Settings UI contains several tabs. Each tab is assiciated with
+  // a unique tab id. The enum |TabId| contains all the ids for the tabs.
+  enum TabId {
+    TAB_ID_PERMISSIONS,

[msw, 2012/08/23 01:01:35]

nit: explicitly specify TAB_ID_PERMISSIONS = 0 (especially if you're DCHECKing
this elsewhere), you can let the others implicitly inherit incremental values.

[markusheintz_, 2012/08/23 19:46:21]

Done.

+    TAB_ID_CONNECTION,
+    NUM_TAB_IDS

[msw, 2012/08/23 01:01:35]

nit: always use trailing commas in enum decls.

[markusheintz_, 2012/08/23 19:46:21]

Done

+  };
+
   // |CookieInfo| contains information about the cookies from a specific source.
   // A source can for example be a specific origin or an entire domain.
   struct CookieInfo {
@@ -133,6 +141,9 @@ class WebsiteSettingsUI {
 
   // Sets the first visited data. |first_visit| can be an empty string.
   virtual void SetFirstVisit(const string16& first_visit) = 0;
+
+  // Selects the tab with the given |tab_id|.
+  virtual void SetSelectedTab(TabId tab_id) = 0;
 };
 
 typedef WebsiteSettingsUI::CookieInfoList CookieInfoList;