From d69231f5703724a02f19d69d098f2d97f1a85588 Mon Sep 17 00:00:00 2001
From: Marina Samuel <msamuel@mozilla.com>
Date: Fri, 29 Apr 2011 01:20:35 -0700
Subject: [PATCH 1/4] Bug 621517 - Sync UI: Add a Device wizard pixel-shifts
 while validating J-PAKE pin. r=rnewman

---
 browser/base/content/syncAddDevice.xul | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/browser/base/content/syncAddDevice.xul b/browser/base/content/syncAddDevice.xul
index ccc051d5a306..24ae401d740e 100644
--- a/browser/base/content/syncAddDevice.xul
+++ b/browser/base/content/syncAddDevice.xul
@@ -77,11 +77,11 @@
              value="&addDevice.showMeHow.label;"
              href="https://services.mozilla.com/sync/help/add-device"/>
     </description>
-    <spacer flex="1"/>
+    <separator class="groove-thin"/>
     <description>
       &addDevice.dialog.enterCode.label;
     </description>
-    <spacer flex="1"/>
+    <separator class="groove-thin"/>
     <vbox align="center">
       <textbox id="pin1"
                class="pin"
@@ -99,7 +99,7 @@
                onfocus="this.select();" 
               />
     </vbox>
-    <spacer flex="1"/>
+    <separator class="groove-thin"/>
     <vbox id="add-device-throbber" align="center" hidden="true">
       <image/>
     </vbox>

From 81975c5596eed650d6d3b72ccaadd3aff8e2587f Mon Sep 17 00:00:00 2001
From: Marina Samuel <msamuel@mozilla.com>
Date: Fri, 29 Apr 2011 01:21:06 -0700
Subject: [PATCH 2/4] Bug 645883 - "Deactivate this device" should not default
 to action. r=rnewman

---
 browser/components/preferences/sync.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/browser/components/preferences/sync.js b/browser/components/preferences/sync.js
index e9abc46dfc64..40e696634437 100644
--- a/browser/components/preferences/sync.js
+++ b/browser/components/preferences/sync.js
@@ -110,7 +110,8 @@ let gSyncPane = {
   startOver: function (showDialog) {
     if (showDialog) {
       let flags = Services.prompt.BUTTON_POS_0 * Services.prompt.BUTTON_TITLE_IS_STRING +
-                  Services.prompt.BUTTON_POS_1 * Services.prompt.BUTTON_TITLE_CANCEL;
+                  Services.prompt.BUTTON_POS_1 * Services.prompt.BUTTON_TITLE_CANCEL + 
+                  Services.prompt.BUTTON_POS_1_DEFAULT;
       let buttonChoice =
         Services.prompt.confirmEx(window,
                                   this._stringBundle.GetStringFromName("stopUsingAccount.title"),

From cc7e36b18a02e639050382ba24b4c6ee81d47005 Mon Sep 17 00:00:00 2001
From: Richard Newman <rnewman@mozilla.com>
Date: Fri, 29 Apr 2011 10:57:59 -0700
Subject: [PATCH 3/4] Bug 653307: Adjust reCAPTCHA NoScript hack in Sync.
 r=philiKON

---
 browser/base/content/syncSetup.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/browser/base/content/syncSetup.js b/browser/base/content/syncSetup.js
index 514d2d6f99a6..7f2ef409fb7c 100644
--- a/browser/base/content/syncSetup.js
+++ b/browser/base/content/syncSetup.js
@@ -56,6 +56,11 @@ const OPTIONS_PAGE                  = 6;
 const OPTIONS_CONFIRM_PAGE          = 7;
 const SETUP_SUCCESS_PAGE            = 8;
 
+// Broader than we'd like, but after this changed from api-secure.recaptcha.net
+// we had no choice. At least we only do this for the duration of setup.
+// See discussion in Bugs 508112 and 653307.
+const RECAPTCHA_DOMAIN = "https://www.google.com";
+
 Cu.import("resource://services-sync/main.js");
 Cu.import("resource://gre/modules/XPCOMUtils.jsm");
 Cu.import("resource://gre/modules/Services.jsm");
@@ -69,7 +74,6 @@ var gSyncSetup = {
   captchaBrowser: null,
   wizard: null,
   _disabledSites: [],
-  _remoteSites: [Weave.Service.serverURL, "https://api-secure.recaptcha.net"],
 
   status: {
     password: false,
@@ -77,6 +81,8 @@ var gSyncSetup = {
     server: false
   },
 
+  get _remoteSites() [Weave.Service.serverURL, RECAPTCHA_DOMAIN],
+
   get _usingMainServers() {
     if (this._settingUpNew)
       return document.getElementById("server").selectedIndex == 0;

From 63cf42e132914ebd59f491f7a71a3cf5291bb797 Mon Sep 17 00:00:00 2001
From: Marina Samuel <msamuel@mozilla.com>
Date: Mon, 2 May 2011 09:26:31 -0700
Subject: [PATCH 4/4] Bug 652483 - Sync UI: When printing or saving a sync key,
 the dialog prompts you to change it. r=philikon

---
 browser/base/content/syncGenericChange.js                       | 2 +-
 browser/base/content/syncGenericChange.xul                      | 2 +-
 .../locales/en-US/chrome/browser/syncGenericChange.properties   | 2 +-
 browser/locales/en-US/chrome/browser/syncSetup.dtd              | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/browser/base/content/syncGenericChange.js b/browser/base/content/syncGenericChange.js
index 2cd16f6c3e38..90ab64197fdc 100644
--- a/browser/base/content/syncGenericChange.js
+++ b/browser/base/content/syncGenericChange.js
@@ -111,7 +111,7 @@ let Change = {
              pp = Weave.Utils.hyphenatePassphrase(pp);
           this._passphraseBox.value = pp;
           this._passphraseBox.focus();
-          document.title = this._str("change.synckey.title");
+          document.title = this._str("change.synckey2.title");
           introText.textContent = this._str("change.synckey.introText2");
           warningText.textContent = this._str("change.synckey2.warningText");
           this._dialog.getButton("finish").label
diff --git a/browser/base/content/syncGenericChange.xul b/browser/base/content/syncGenericChange.xul
index 1e76c16520d6..391a29c26221 100644
--- a/browser/base/content/syncGenericChange.xul
+++ b/browser/base/content/syncGenericChange.xul
@@ -100,7 +100,7 @@
           <spacer flex="1"/>
           <label id="generatePassphraseButton"
                  hidden="true"
-                 value="&syncKeyGenerate.label;"
+                 value="&syncGenerateNewKey.label;"
                  class="text-link inline-link"
                  onclick="event.stopPropagation();
                           Change.doGeneratePassphrase();"/>
diff --git a/browser/locales/en-US/chrome/browser/syncGenericChange.properties b/browser/locales/en-US/chrome/browser/syncGenericChange.properties
index c7da1ac1b0cb..32310a307a7c 100644
--- a/browser/locales/en-US/chrome/browser/syncGenericChange.properties
+++ b/browser/locales/en-US/chrome/browser/syncGenericChange.properties
@@ -8,7 +8,7 @@ change.password.status.error = There was an error changing your password.
 change.password2.introText = Your password must be at least 8 characters long.  It cannot be the same as either your user name or your Sync Key.
 change.password.warningText = Note: All of your other devices will be unable to connect to your account once you change this password.
 
-change.synckey.title = Change your Sync Key
+change.synckey2.title = My Sync Key
 change.synckey.acceptButton = Change Sync Key
 change.synckey.label = Changing Sync Key and uploading local data, please wait…
 change.synckey2.error = There was an error while changing your Sync Key!
diff --git a/browser/locales/en-US/chrome/browser/syncSetup.dtd b/browser/locales/en-US/chrome/browser/syncSetup.dtd
index c8023bbf599d..020ac966e263 100644
--- a/browser/locales/en-US/chrome/browser/syncSetup.dtd
+++ b/browser/locales/en-US/chrome/browser/syncSetup.dtd
@@ -43,7 +43,7 @@
 <!ENTITY setup.newSyncKeyPage.description.label "To ensure your total privacy, all of your data is encrypted prior to being uploaded. The Sync Key which is necessary to decrypt your data is not uploaded.">
 <!ENTITY syncKeyEntry.label        "Your Sync Key">
 <!ENTITY syncKeyEntry.accesskey    "K">
-<!ENTITY syncKeyGenerate.label     "Generate">
+<!ENTITY syncGenerateNewKey.label  "Generate a new key">
 <!ENTITY syncKeyBackup.description "Your Sync Key is required to access &syncBrand.fullName.label; on other machines. Please create a backup copy. We cannot help you recover your Sync Key.">
 
 <!ENTITY button.syncKeyBackup.print.label     "Print…">