Add "Get Themes" button to gtk options dialog.

chrome/browser/views/options/content_page_view.cc

 // Copyright (c) 2009 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/views/options/content_page_view.h"
 
 #include <windows.h>
 #include <shlobj.h>
 #include <vsstyle.h>
 #include <vssym32.h>
@@ skipping 71 matching lines @@
     }
     ask_to_save_form_autofill_.SetValue(enabled);
   } else if (sender == passwords_exceptions_button_) {
     UserMetricsRecordAction(L"Options_ShowPasswordsExceptions", NULL);
     PasswordsExceptionsWindowView::Show(profile());
   } else if (sender == themes_reset_button_) {
     UserMetricsRecordAction(L"Options_ThemesReset", profile()->GetPrefs());
     profile()->ClearTheme();
   } else if (sender == themes_gallery_button_) {
     UserMetricsRecordAction(L"Options_ThemesGallery", profile()->GetPrefs());
-    Browser* browser =
-        BrowserList::FindBrowserWithType(profile(), Browser::TYPE_NORMAL);
-
-    if (!browser || !browser->GetSelectedTabContents()) {
-      browser = Browser::Create(profile());
-      browser->OpenURL(GURL(l10n_util::GetString(IDS_THEMES_GALLERY_URL)),
-                       GURL(), NEW_WINDOW, PageTransition::LINK);
-    } else {
-      browser->AddTabWithURL(GURL(l10n_util::GetString(IDS_THEMES_GALLERY_URL)),
-          GURL(), PageTransition::LINK, true, -1, false, NULL);
-    }
+    BrowserList::GetLastActive()->OpenURL(
+        GURL(l10n_util::GetString(IDS_THEMES_GALLERY_URL)),
+        GURL(), NEW_FOREGROUND_TAB, PageTransition::LINK);

[Evan Stade, 2009/07/30 21:23:49]

correct me if I'm wrong, but I believe this does exactly what you want (if the
last active browser doesn't support tabs then it finds one that does)

   } else if (sender == import_button_) {
     views::Window::CreateChromeWindow(
       GetWindow()->GetNativeWindow(),
       gfx::Rect(),
       new ImporterView(profile()))->Show();
   } else if (sender == clear_data_button_) {
     views::Window::CreateChromeWindow(
       GetWindow()->GetNativeWindow(),
       gfx::Rect(),
       new ClearBrowsingDataView(profile()))->Show();
@@ skipping 220 matching lines @@
   double_col_set->AddColumn(GridLayout::LEADING, GridLayout::CENTER, 0,
                             GridLayout::USE_PREF, 0, 0);
   layout->StartRow(0, double_column_view_set_id);
   layout->AddView(import_button_);
   layout->AddView(clear_data_button_);
 
   browsing_data_group_ = new OptionsGroupView(
       contents, l10n_util::GetString(IDS_OPTIONS_BROWSING_DATA_GROUP_NAME),
       L"", true);
 }