[Chromoting] Improve the text on the buttons in the connection-failed dialog.

remoting/webapp/client_screen.js

Index: remoting/webapp/client_screen.js
diff --git a/remoting/webapp/client_screen.js b/remoting/webapp/client_screen.js
index 5890518400270bb50245507b595abeff723a057c..feacf85727fcf2b258dddfd1a7150b0a96064e36 100644
--- a/remoting/webapp/client_screen.js
+++ b/remoting/webapp/client_screen.js
@@ -212,6 +212,7 @@ function onClientStateChange_(oldState, newState) {
   } else if (newState == remoting.ClientSession.State.CONNECTED) {
     if (remoting.clientSession) {
       clearPin = true;
+      setConnectionInterruptedButtonsText_();
       remoting.setMode(remoting.AppMode.IN_SESSION);
       remoting.toolbar.center();
       remoting.toolbar.preview();
@@ -347,6 +348,20 @@ function showConnectError_(errorTag) {
 }
 
 /**
+ * Set the text on the buttons shown under the error message so that they are
+ * easy to understand in the case where a successful connection failed, as
+ * opposed to the case where a connection never succeeded.
+ */
+function setConnectionInterruptedButtonsText_() {
+  var button1 = document.getElementById('client-reconnect-button');
+  l10n.localizeElementFromTag(button1, 'RECONNECT');

[Jamie, 2012/03/22 04:24:05]

Please add /*i18n-content*/ to this so that the script will pick it up correctly
(you might also need to modify remoting.gyp to make sure that this file is
included in the checked set. The script is not very smart about spacing--look
elsewhere in JS files to see what it expects.

[simonmorris, 2012/03/22 17:08:25]

Done.

+  button1.removeAttribute('autofocus');
+  var button2 = document.getElementById('client-finished-me2me-button');
+  l10n.localizeElementFromTag(button2, 'OK');

[Jamie, 2012/03/22 04:24:05]

Ditto

[simonmorris, 2012/03/22 17:08:25]

Done.

+  button2.setAttribute('autofocus', 'autofocus');
+}
+
+/**
  * Parse the response from the server to a request to resolve a support id.
  *
  * @param {XMLHttpRequest} xhr The XMLHttpRequest object.