Hangouts Remote Desktop Part VI - Show confirm dialog before retrieving access code

remoting/webapp/background/it2me_helpee_channel.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  *
  * It2MeHelpeeChannel relays messages between the Hangouts web page (Hangouts)
  * and the It2Me Native Messaging Host (It2MeHost) for the helpee (the Hangouts
  * participant who is receiving remoting assistance).
@@ skipping 257 matching lines @@
     this.connectToHost_.bind(this, email),
     this.sendErrorResponse_.bind(this, message)
   );
 };
 
 /**
  * Prompts the user before starting the It2Me Native Messaging Host.  This
  * ensures that even if Hangouts is compromised, an attacker cannot start the
  * host without explicit user confirmation.
  *
- * @return {Promise} A promise that resolves when the user clicks accept on the
- *     dialog.
+ * @return {Promise} A promise that resolves to a boolean value, indicating
+ *     whether the user accepts the remote assistance or not.
  * @private
  */
 remoting.It2MeHelpeeChannel.prototype.showConfirmDialog_ = function() {
+  if (base.isAppsV2()) {
+    return this.showConfirmDialogV2_();
+  } else {
+    return this.showConfirmDialogV1_();
+  }
+};
+
+/**
+ * @return {Promise} A promise that resolves to a boolean value, indicating
+ *     whether the user accepts the remote assistance or not.
+ * @private
+ */
+remoting.It2MeHelpeeChannel.prototype.showConfirmDialogV1_ = function() {
+  var messageHeader = l10n.getTranslationOrError(
+      /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_MESSAGE_1');
+  var message1 = l10n.getTranslationOrError(
+      /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_MESSAGE_2');
+  var message2 = l10n.getTranslationOrError(
+      /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_MESSAGE_3');
+  var message = base.escapeHTML(messageHeader) + '\n' +
+                '- ' + base.escapeHTML(message1) + '\n' +
+                '- ' + base.escapeHTML(message2) + '\n';
+
+  if(window.confirm(message)) {
+    return Promise.resolve();
+  } else {
+    return Promise.reject(new Error(remoting.Error.CANCELLED));
+  }
+};
+
+/**
+ * @return {Promise} A promise that resolves to a boolean value, indicating
+ *     whether the user accepts the remote assistance or not.
+ * @private
+ */
+remoting.It2MeHelpeeChannel.prototype.showConfirmDialogV2_ = function() {
+  var messageHeader = l10n.getTranslationOrError(
+      /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_MESSAGE_1');
+  var message1 = l10n.getTranslationOrError(
+      /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_MESSAGE_2');
+  var message2 = l10n.getTranslationOrError(
+      /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_MESSAGE_3');
+  var message = '<div>' + base.escapeHTML(messageHeader) + '</div>' +
+                '<ul class="insetList">' +
+                  '<li>' + base.escapeHTML(message1) + '</li>' +
+                  '<li>' + base.escapeHTML(message2) + '</li>' +
+                '</ul>';
   /**
    * @param {function(*=):void} resolve
    * @param {function(*=):void} reject
    */
   return new Promise(function(resolve, reject) {
-    // TODO(kelvinp): The existing implementation doesn't work in the v2 app as
-    // window.confirm is blacklisted. Implement the dialog using
-    // chrome.app.window instead (crbug.com/405139).
-    if (base.isAppsV2()) {
-      resolve();
-    // The unlocalized string will be replaced in (crbug.com/405139).
-    } else if (window.confirm('Accept help?')) {
-      resolve();
-    } else {
-      reject(new Error(remoting.Error.CANCELLED));
+    /** @param {number} result */
+    function confirmDialogCallback(result) {
+      if (result === 1) {
+        resolve();
+      } else {
+        reject(new Error(remoting.Error.CANCELLED));
+      }
     }
+    remoting.MessageWindow.showConfirmWindow(
+        '', // Empty string to use the package name as the dialog title.
+        message,
+        l10n.getTranslationOrError(
+            /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_ACCEPT'),
+        l10n.getTranslationOrError(
+            /*i18n-content*/'HANGOUTS_CONFIRM_DIALOG_DECLINE'),
+        confirmDialogCallback
+    );
   });
 };
 
 /**
  * @return {Promise} A promise that resolves when the host is initialized.
  * @private
  */
 remoting.It2MeHelpeeChannel.prototype.initializeHost_ = function() {
   /** @type {remoting.It2MeHostFacade} */
   var host = this.host_;
@@ skipping 110 matching lines @@
 
   console.error('Error responding to message method:' +
                 (incomingMessage ? incomingMessage.method : 'null') +
                 ' error:' + error);
   this.hangoutPort_.postMessage({
     method: remoting.It2MeHelpeeChannel.HangoutMessageTypes.ERROR,
     message: error,
     request: incomingMessage
   });
 };