gtk: Remember which page user viewed last in the Certificate Manager window.

chrome/browser/gtk/certificate_manager.cc

 // Copyright (c) 2010 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.
 
 #include "chrome/browser/gtk/certificate_manager.h"
 
 #include <cert.h>
 #include <gtk/gtk.h>
 #include <pk11pub.h>
 
 #include <map>
 #include <string>
 
 #include "app/gtk_signal.h"
 #include "app/l10n_util.h"
 #include "app/l10n_util_collator.h"
 #include "base/i18n/time_formatting.h"
 #include "base/nss_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/gtk/certificate_viewer.h"
 #include "chrome/browser/gtk/gtk_util.h"
+#include "chrome/browser/pref_member.h"
+#include "chrome/browser/pref_service.h"
+#include "chrome/browser/profile.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/third_party/mozilla_security_manager/nsNSSCertHelper.h"
 #include "chrome/third_party/mozilla_security_manager/nsNSSCertificate.h"
 #include "grit/generated_resources.h"
 
 // PSM = Mozilla's Personal Security Manager.
 namespace psm = mozilla_security_manager;
 
 namespace {
 
 // Convert a char* return value from NSS into a std::string and free the NSS
@@ skipping 10 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // CertificatePage class definition.
 
 class CertificatePage {
  public:
   explicit CertificatePage(psm::CertType type);
 
   void PopulateTree(CERTCertList* cert_list);
 
   // Get the top-level widget of this page.
-  GtkWidget* widget() {
-    return vbox_;
-  }
+  GtkWidget* widget() { return vbox_; }
 
  private:
   // Columns of the tree store.
   enum {
     CERT_NAME,
     CERT_SECURITY_DEVICE,
     CERT_SERIAL_NUMBER,
     CERT_EXPIRES_ON,
     CERT_EXPIRES_ON_INT,
     CERT_ADDRESS,
@@ skipping 272 matching lines @@
   gtk_tree_model_get(model, &iter, CERT_POINTER, &cert, -1);
   if (cert)
     ShowCertificateViewer(GTK_WINDOW(gtk_widget_get_toplevel(widget())), cert);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // CertificateManager class definition.
 
 class CertificateManager {
  public:
-  explicit CertificateManager(gfx::NativeWindow parent);
+  explicit CertificateManager(gfx::NativeWindow parent, Profile* profile);
   ~CertificateManager();
 
-  void Show();
+  // Shows the Tab corresponding to the specified |page|.
+  void ShowCertificatePage(CertificateManagerPage page);
 
  private:
+  CHROMEGTK_CALLBACK_2(CertificateManager, void, OnSwitchPage,
+                       GtkNotebookPage*, guint);
+
   CERTCertList* cert_list_;
 
   CertificatePage user_page_;
   CertificatePage email_page_;
   CertificatePage server_page_;
   CertificatePage ca_page_;
   CertificatePage unknown_page_;
 
   GtkWidget* dialog_;
+
+  GtkWidget* notebook_;
+
+  // The last page the user was on when they opened the CertificateManager
+  // window.
+  IntegerPrefMember last_selected_page_;
+
+  DISALLOW_COPY_AND_ASSIGN(CertificateManager);
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // CertificateManager implementation.
 
 void OnDestroy(GtkDialog* dialog, CertificateManager* cert_manager) {
   delete cert_manager;
 }
 
-CertificateManager::CertificateManager(gfx::NativeWindow parent)
+CertificateManager::CertificateManager(gfx::NativeWindow parent,
+                                       Profile* profile)
     : user_page_(psm::USER_CERT),
       email_page_(psm::EMAIL_CERT),
       server_page_(psm::SERVER_CERT),
       ca_page_(psm::CA_CERT),
       unknown_page_(psm::UNKNOWN_CERT) {
+  // We don't need to observe changes in this value.
+  last_selected_page_.Init(prefs::kCertificateManagerWindowLastTabIndex,
+                           profile->GetPrefs(), NULL);
+
   dialog_ = gtk_dialog_new_with_buttons(
       l10n_util::GetStringUTF8(IDS_CERTIFICATE_MANAGER_TITLE).c_str(),
       parent,
       // Non-modal.
       GTK_DIALOG_NO_SEPARATOR,
       GTK_STOCK_CLOSE,
       GTK_RESPONSE_CLOSE,
       NULL);
   gtk_box_set_spacing(GTK_BOX(GTK_DIALOG(dialog_)->vbox),
                       gtk_util::kContentAreaSpacing);
   gtk_window_set_default_size(GTK_WINDOW(dialog_), 600, 440);
 
-  GtkWidget* notebook = gtk_notebook_new();
-  gtk_container_add(GTK_CONTAINER(GTK_DIALOG(dialog_)->vbox), notebook);
+  notebook_ = gtk_notebook_new();
+  gtk_container_add(GTK_CONTAINER(GTK_DIALOG(dialog_)->vbox), notebook_);
 
-  // TODO(mattm): Remember which page user viewed last.
   gtk_notebook_append_page(
-      GTK_NOTEBOOK(notebook),
+      GTK_NOTEBOOK(notebook_),
       user_page_.widget(),
       gtk_label_new_with_mnemonic(
           l10n_util::GetStringUTF8(
               IDS_CERT_MANAGER_PERSONAL_CERTS_TAB_LABEL).c_str()));
 
   gtk_notebook_append_page(
-      GTK_NOTEBOOK(notebook),
+      GTK_NOTEBOOK(notebook_),
       email_page_.widget(),
       gtk_label_new_with_mnemonic(
           l10n_util::GetStringUTF8(
               IDS_CERT_MANAGER_OTHER_PEOPLES_CERTS_TAB_LABEL).c_str()));
 
   gtk_notebook_append_page(
-      GTK_NOTEBOOK(notebook),
+      GTK_NOTEBOOK(notebook_),
       server_page_.widget(),
       gtk_label_new_with_mnemonic(
           l10n_util::GetStringUTF8(
               IDS_CERT_MANAGER_SERVER_CERTS_TAB_LABEL).c_str()));
 
   gtk_notebook_append_page(
-      GTK_NOTEBOOK(notebook),
+      GTK_NOTEBOOK(notebook_),
       ca_page_.widget(),
       gtk_label_new_with_mnemonic(
           l10n_util::GetStringUTF8(
               IDS_CERT_MANAGER_CERT_AUTHORITIES_TAB_LABEL).c_str()));
 
   gtk_notebook_append_page(
-      GTK_NOTEBOOK(notebook),
+      GTK_NOTEBOOK(notebook_),
       unknown_page_.widget(),
       gtk_label_new_with_mnemonic(
           l10n_util::GetStringUTF8(
               IDS_CERT_MANAGER_UNKNOWN_TAB_LABEL).c_str()));
 
   cert_list_ = PK11_ListCerts(PK11CertListUnique, NULL);
   user_page_.PopulateTree(cert_list_);
   email_page_.PopulateTree(cert_list_);
   server_page_.PopulateTree(cert_list_);
   ca_page_.PopulateTree(cert_list_);
   unknown_page_.PopulateTree(cert_list_);
 
+  // Need to show the notebook before connecting switch-page signal, otherwise
+  // we'll immediately get a signal switching to page 0 and overwrite our
+  // last_selected_page_ value.
+  gtk_util::ShowDialogWithLocalizedSize(dialog_, -1, -1, true);
+
+  g_signal_connect(notebook_, "switch-page",
+                   G_CALLBACK(OnSwitchPageThunk), this);
+
   g_signal_connect(dialog_, "response", G_CALLBACK(gtk_widget_destroy), NULL);
   g_signal_connect(dialog_, "destroy", G_CALLBACK(OnDestroy), this);
 }
 
 CertificateManager::~CertificateManager() {
   CERT_DestroyCertList(cert_list_);
 }
 
-void CertificateManager::Show() {
-  gtk_util::ShowDialog(dialog_);
+void CertificateManager::OnSwitchPage(GtkWidget* notebook,
+                                      GtkNotebookPage* page,
+                                      guint page_num) {
+  int index = static_cast<int>(page_num);
+  DCHECK(index > PAGE_DEFAULT && index < PAGE_COUNT);
+  last_selected_page_.SetValue(index);
 }
 
-} // namespace
+void CertificateManager::ShowCertificatePage(CertificateManagerPage page) {
+  // Bring options window to front if it already existed and isn't already
+  // in front
+  gtk_window_present_with_time(GTK_WINDOW(dialog_),
+                               gtk_get_current_event_time());
 
-void ShowCertificateManager(gfx::NativeWindow parent) {
+  if (page == PAGE_DEFAULT) {
+    // Remember the last visited page from local state.
+    page = static_cast<CertificateManagerPage>(last_selected_page_.GetValue());
+    if (page == PAGE_DEFAULT)
+      page = PAGE_USER;
+  }
+  // If the page number is out of bounds, reset to the first tab.
+  if (page < 0 || page >= gtk_notebook_get_n_pages(GTK_NOTEBOOK(notebook_)))
+    page = PAGE_USER;
+
+  gtk_notebook_set_current_page(GTK_NOTEBOOK(notebook_), page);
+}
+
+}  // namespace
+
+namespace certificate_manager_util {
+
+void RegisterUserPrefs(PrefService* prefs) {
+  prefs->RegisterIntegerPref(prefs::kCertificateManagerWindowLastTabIndex, 0);
+}
+
+}  // namespace certificate_manager_util
+
+void ShowCertificateManager(gfx::NativeWindow parent, Profile* profile,
+                            CertificateManagerPage page) {
   base::EnsureNSSInit();
-  (new CertificateManager(parent))->Show();
+  CertificateManager* manager = new CertificateManager(parent, profile);
+  manager->ShowCertificatePage(page);
 }