Include the session id in feedback logs.

remoting/webapp/app_remoting/js/application_context_menu.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
  * Class representing the application's context menu.
  */
 
 'use strict';
 
 /** @suppress {duplicate} */
 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'),
       false);
   this.adapter_.create(
       remoting.ApplicationContextMenu.kShowStatsId,
       l10n.getTranslationOrError(/*i18n-content*/'SHOW_STATS'),
       true);
 
   // TODO(kelvinp):Unhook this event on shutdown.
@@ skipping 52 matching lines @@
 
       /** @type {remoting.ApplicationContextMenu} */
       var that = this;
 
       /** @param {AppWindow} consentWindow */
       var onCreate = function(consentWindow) {
         var onLoad = function() {
           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, '*');
         };
         consentWindow.contentWindow.addEventListener('load', onLoad, false);
       };
       chrome.app.window.create(
           'feedback_consent.html', windowAttributes, onCreate);
       break;
 
     case remoting.ApplicationContextMenu.kShowStatsId:
       this.stats_.show(info.checked);
       break;
   }
 };
 
 
 /** @type {string} */
 remoting.ApplicationContextMenu.kSendFeedbackId = 'send-feedback';
 
 /** @type {string} */
 remoting.ApplicationContextMenu.kShowStatsId = 'show-stats';