From bcec4017ac26d4916d57088fc1cdf3085096f761 Mon Sep 17 00:00:00 2001
From: Anthony Lieuallen <arantius@gmail.com>
Date: Mon, 28 May 2007 14:34:28 +0000
Subject: [PATCH] Closes #41

 * Disable the cache-selection interface while a synchronous API call is being made.
---
 content/netError.xhtml              |  4 ++--
 content/resurrect-select-mirror.xul |  2 +-
 content/resurrect.js                | 26 ++++++++++++++++++++++----
 3 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/content/netError.xhtml b/content/netError.xhtml
index e12ecbe..9804023 100644
--- a/content/netError.xhtml
+++ b/content/netError.xhtml
@@ -196,7 +196,7 @@
 	<div id="errorTitle">
 		<h1 id="errorTitleText" />
 	</div>
-	
+
 	<!-- LONG CONTENT (the section most likely to require scrolling) -->
 	<div id="errorLongContent">
 		<!-- Short Description -->
@@ -219,7 +219,7 @@
 		<div id='targetWin' style='display: none;'></div>
 
 		<p>&resurrect.thispage;</p>
-		<select id='mirror' size='5'>
+		<select id='mirror' size='6'>
 			<option value='coralcdn'>&resurrect.coralcdn;</option>
 			<option value='google'>&resurrect.google;</option>
 			<option value='googletext'>&resurrect.google;&resurrect.textonly;</option>
diff --git a/content/resurrect-select-mirror.xul b/content/resurrect-select-mirror.xul
index 79f2c29..f59e075 100644
--- a/content/resurrect-select-mirror.xul
+++ b/content/resurrect-select-mirror.xul
@@ -22,7 +22,7 @@
 		</radiogroup>
 
 		<label value='&resurrect.fromMirror;' />
-		<listbox id='mirror' rows='5' 
+		<listbox id='mirror' rows='6'
 			onclick='resurrect.clickedXul(event);'
 		>
 			<listitem value='coralcdn' label='&resurrect.coralcdn;' />
diff --git a/content/resurrect.js b/content/resurrect.js
index 8047dbc..eb67f6d 100644
--- a/content/resurrect.js
+++ b/content/resurrect.js
@@ -99,7 +99,7 @@ var resurrect={
 	},
 
 	clickedHtml:function(event) {
-		resurrect.selectMirror(
+		return resurrect.clickHandler(
 			event.target.ownerDocument,
 			event.target.ownerDocument,
 			event.target.ownerDocument.location.href
@@ -107,17 +107,35 @@ var resurrect={
 	},
 
 	clickedXul:function(event) {
-		resurrect.selectMirror(
+		return resurrect.clickHandler(
 			event.target.ownerDocument,
 			window.arguments[0],
 			window.arguments[1]
 		);
 	},
 
+	clickHandler:function(ownerDoc, contentDoc, rawUrl) {
+		// Check, set, disabled status.
+		var listbox=ownerDoc.getElementById('mirror');
+
+		if (-1==listbox.selectedIndex) return false;
+		if (resurrect.disabled) return false;
+
+		resurrect.disabled=true;
+		listbox.setAttribute('disabled', true);
+
+		var button=contentDoc.getElementById('mirrorSelect');
+		if (!button && ownerDoc.documentElement.getButton) {
+			button=ownerDoc.documentElement.getButton('accept');
+		}
+		button.setAttribute('disabled', true);
+
+		// Run the actual code.  After timeout for UI repaint.
+		setTimeout(resurrect.selectMirror, 1, ownerDoc, contentDoc, rawUrl);
+	},
+
 	selectMirror:function(ownerDoc, contentDoc, rawUrl) {
 		var listbox=ownerDoc.getElementById('mirror');
-		if (resurrect.disabled) return false;
-		resurrect.disabled=true;
 
 		var gotoUrl=null;
 		var encUrl=encodeURIComponent(rawUrl);