[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[or-cvs] r20117: {torbutton} Bump version and improve external app dialog based on patch (in torbutton/trunk/src: . components)
Author: mikeperry
Date: 2009-07-23 17:12:07 -0400 (Thu, 23 Jul 2009)
New Revision: 20117
Modified:
torbutton/trunk/src/components/external-app-blocker.js
torbutton/trunk/src/install.rdf
Log:
Bump version and improve external app dialog based on patch
from optimist.
Modified: torbutton/trunk/src/components/external-app-blocker.js
===================================================================
--- torbutton/trunk/src/components/external-app-blocker.js 2009-07-23 02:27:54 UTC (rev 20116)
+++ torbutton/trunk/src/components/external-app-blocker.js 2009-07-23 21:12:07 UTC (rev 20117)
@@ -136,10 +136,16 @@
.getService(Components.interfaces.nsIPromptService);
var check = {value: false};
// XXX: Localize
- var result = prompts.confirmCheck(chrome, "Load external content?",
- "Do you want to launch an external app to handle"
- +"\n"+aUri.spec+
- "\n\nNote: External apps are NOT Tor safe by default and can unmask you!\n",
+ var flags = prompts.BUTTON_POS_0 * prompts.BUTTON_TITLE_IS_STRING +
+ prompts.BUTTON_POS_1 * prompts.BUTTON_TITLE_IS_STRING +
+ prompts.BUTTON_DELAY_ENABLE +
+ prompts.BUTTON_POS_1_DEFAULT;
+ var result = prompts.confirmEx(chrome, "Load external content?",
+ "An external application is needed to handle"
+ +"\n"+aUri.spec
+ +"\n\nNOTE: External applications are NOT Tor safe by default and can unmask you!\n ",
+ flags,
+ "Launch application", "Cancel", "",
"Do not ask me again", check);
// do something check.value / result
@@ -147,7 +153,7 @@
// XXX: Set a pref...
}
- if (!result) {
+ if (result != 0) {
return null;
}
@@ -168,17 +174,23 @@
.getService(Components.interfaces.nsIPromptService);
var check = {value: false};
// XXX: Localize
- var result = prompts.confirmCheck(chrome, "Load external content?",
- "Do you want to launch an external app to handle"
- +"\n"+aRequest.name+
- "\n\nNote: External apps are NOT Tor safe by default and can unmask you!\n",
+ var flags = prompts.BUTTON_POS_0 * prompts.BUTTON_TITLE_IS_STRING +
+ prompts.BUTTON_POS_1 * prompts.BUTTON_TITLE_IS_STRING +
+ prompts.BUTTON_DELAY_ENABLE +
+ prompts.BUTTON_POS_1_DEFAULT;
+ var result = prompts.confirmEx(chrome, "Load external content?",
+ "An external application is needed to handle"
+ +"\n"+aRequest.name
+ +"\n\nNOTE: External applications are NOT Tor safe by default and can unmask you!\n ",
+ flags,
+ "Launch application", "Cancel", "",
"Do not ask me again", check);
// do something check.value / result
if (check.value) {
// XXX: Set a pref...
}
- if (!result) {
+ if (result != 0) {
return null;
}
}
Modified: torbutton/trunk/src/install.rdf
===================================================================
--- torbutton/trunk/src/install.rdf 2009-07-23 02:27:54 UTC (rev 20116)
+++ torbutton/trunk/src/install.rdf 2009-07-23 21:12:07 UTC (rev 20117)
@@ -6,7 +6,7 @@
<em:name>Torbutton</em:name>
<em:creator>Mike Perry & Scott Squires</em:creator>
<em:id>{e0204bd5-9d31-402b-a99d-a6aa8ffebdca}</em:id>
- <em:version>1.2.1</em:version>
+ <em:version>1.2.1-dev</em:version>
<em:homepageURL>https://torbutton.torproject.org/dev/</em:homepageURL>
<em:optionsURL>chrome://torbutton/content/preferences.xul</em:optionsURL>
<em:iconURL>chrome://torbutton/skin/tor.png</em:iconURL>