Include the session id in feedback logs.

remoting/webapp/app_remoting/js/application_context_menu.js

Index: remoting/webapp/app_remoting/js/application_context_menu.js
diff --git a/remoting/webapp/app_remoting/js/application_context_menu.js b/remoting/webapp/app_remoting/js/application_context_menu.js
index ae1b4b0923a140b0046070ed2f318f02f395a60c..507987f3aa10429bc817f899e9c9a62d9d6ffb4d 100644
--- a/remoting/webapp/app_remoting/js/application_context_menu.js
+++ b/remoting/webapp/app_remoting/js/application_context_menu.js
@@ -15,13 +15,18 @@ var remoting = remoting || {};
 /**
  * @param {remoting.ContextMenuAdapter} adapter
  * @param {remoting.ClientPlugin} plugin
+ * @param {remoting.ClientSession} clientSession
+ *
  * @constructor
  * @implements {base.Disposable}
  */
-remoting.ApplicationContextMenu = function(adapter, plugin) {
-  /** @private {remoting.ContextMenuAdapter} */
+remoting.ApplicationContextMenu = function(adapter, plugin, clientSession) {

[kelvinp, 2015/05/13 22:36:53]

what about just passing the sessionId?

[Jamie, 2015/05/13 23:41:24]

That would work in the majority of cases, but since the session id is reset
every 24hrs, it's not perfect, and I don't think it simplifies the code very
much either.

[kelvinp, 2015/05/14 01:18:04]

Acknowledged.

+  /** @private */
   this.adapter_ = adapter;
 
+  /** @private */
+  this.clientSession_ = clientSession;
+
   this.adapter_.create(
       remoting.ApplicationContextMenu.kSendFeedbackId,
       l10n.getTranslationOrError(/*i18n-content*/'SEND_FEEDBACK'),
@@ -94,7 +99,8 @@ remoting.ApplicationContextMenu.prototype.onClicked_ = function(info) {
           var message = {
             method: 'init',
             hostId: that.hostId_,
-            connectionStats: JSON.stringify(that.stats_.mostRecent())
+            connectionStats: JSON.stringify(that.stats_.mostRecent()),
+            sessionId: that.clientSession_.getSessionId()
           };
           consentWindow.contentWindow.postMessage(message, '*');
         };