Added extra text to 'host started' dialog.

remoting/webapp/host_setup_dialog.js

Index: remoting/webapp/host_setup_dialog.js
diff --git a/remoting/webapp/host_setup_dialog.js b/remoting/webapp/host_setup_dialog.js
index 8bee46614b0ddfcc23e3f8d4e3bc2475e2f1bc16..3b24caf0ffa16af86828f09883e265e215122540 100644
--- a/remoting/webapp/host_setup_dialog.js
+++ b/remoting/webapp/host_setup_dialog.js
@@ -219,10 +219,17 @@ remoting.HostSetupDialog.prototype.updateState_ = function() {
     l10n.localizeElementFromTag(messageDiv, tag);
     remoting.setMode(remoting.AppMode.HOST_SETUP_PROCESSING);
   }
-  /** @param {string} tag */
-  function showDoneMessage(tag) {
+  /** @param {string} tag1
+   *  @param {string=} opt_tag2 */
+  function showDoneMessage(tag1, opt_tag2) {
     var messageDiv = document.getElementById('host-setup-done-message');
-    l10n.localizeElementFromTag(messageDiv, tag);
+    l10n.localizeElementFromTag(messageDiv, tag1);
+    messageDiv = document.getElementById('host-setup-done-message-2');
+    if (opt_tag2) {
+      l10n.localizeElementFromTag(messageDiv, opt_tag2);
+    } else {
+      messageDiv.innerHTML = '';
+    }
     remoting.setMode(remoting.AppMode.HOST_SETUP_DONE);
   }
   /** @param {string} tag */
@@ -251,7 +258,8 @@ remoting.HostSetupDialog.prototype.updateState_ = function() {
     showProcessingMessage(/*i18n-content*/'HOST_SETUP_STOPPING');
     this.stopHost_();
   } else if (state == remoting.HostSetupFlow.State.HOST_STARTED) {
-    showDoneMessage(/*i18n-content*/'HOST_SETUP_STARTED');
+    showDoneMessage(/*i18n-content*/'HOST_SETUP_STARTED',
+                    /*i18n-content*/'HOST_SETUP_STARTED_DISABLE_SLEEP');

[Sergey Ulanov, 2012/05/01 01:16:32]

Potentially the host plugin could verify power settings, and then we could show
this message only when necessary.

[Sergey Ulanov, 2012/05/01 01:16:32]

Why can't this message be part of HOST_SETUP_STARTED? Is it just so that they
are separated visually? Then maybe we could just put <br> in that resource?

[Jamie, 2012/05/01 01:35:23]

That's a nice idea, but I don't think it's feasible for M20 in the time we have
left. I've added a TODO.

[Jamie, 2012/05/01 01:35:23]

Firstly, I try to avoid putting HTML in translated strings (although it's
occasionally necessary). Secondly, using a <br> to insert vertical space isn't
the right way to do it--we'd need separate <div>s for the two components as the
HTML currently has, which means that the string we send for translation becomes
quite complicated. Finally, if we ever did want to let the plugin control this
programmatically, it's a lot easier with two <div>s.

   } else if (state == remoting.HostSetupFlow.State.UPDATED_PIN) {
     showDoneMessage(/*i18n-content*/'HOST_SETUP_UPDATED_PIN');
   } else if (state == remoting.HostSetupFlow.State.HOST_STOPPED) {