From f3a08e286d0719a388a56eae5e396fc86c223ddc Mon Sep 17 00:00:00 2001
From: "stuartmorgan@chromium.org"
 <stuartmorgan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Tue, 11 Jan 2011 00:22:45 +0000
Subject: DOMUI Prefs: Fix the add page overlay

Add an explicit list height now that we don't inherit one by default.
Clear list selection when clearing state, so re-opening will give a fresh dialog.

BUG=60842
TEST=Open the Add dialog for startup pages, select something, cancel, and then re-open. The seection should be gone.

Review URL: http://codereview.chromium.org/6155004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70968 0039d316-1c4b-4281-b951-d872f2087c98
---
 chrome/browser/resources/options/add_startup_page_overlay.css | 4 ++++
 chrome/browser/resources/options/add_startup_page_overlay.js  | 1 +
 2 files changed, 5 insertions(+)

diff --git a/chrome/browser/resources/options/add_startup_page_overlay.css b/chrome/browser/resources/options/add_startup_page_overlay.css
index a863eb6..7929ba1 100644
--- a/chrome/browser/resources/options/add_startup_page_overlay.css
+++ b/chrome/browser/resources/options/add_startup_page_overlay.css
@@ -19,6 +19,10 @@
   -webkit-box-flex: 1;
 }
 
+#addStartupRecentPageList {
+  height: 320px;
+}
+
 #addStartupRecentPageList .url {
   display: none;
   color: hsl(0, 0%, 70%);
diff --git a/chrome/browser/resources/options/add_startup_page_overlay.js b/chrome/browser/resources/options/add_startup_page_overlay.js
index 659ef3b..d27f266 100644
--- a/chrome/browser/resources/options/add_startup_page_overlay.js
+++ b/chrome/browser/resources/options/add_startup_page_overlay.js
@@ -65,6 +65,7 @@ cr.define('options', function() {
      */
     dismissOverlay_: function() {
       $('addStartupPageURL').value = '';
+      $('addStartupRecentPageList').selectionModel.unselectAll();
       this.updateAddButtonState_();
       OptionsPage.clearOverlays();
     },
-- 
cgit v1.1