Enable logging from the client to the cloud.

remoting/webapp/me2mom/log_to_server.js

Index: remoting/webapp/me2mom/log_to_server.js
diff --git a/remoting/webapp/me2mom/log_to_server.js b/remoting/webapp/me2mom/log_to_server.js
index 2d2d96e75d1829d8e5ebe34c6afcd50e953ecf43..b6c65def898a5fd7ce9f009b92d03302a9dd453f 100644
--- a/remoting/webapp/me2mom/log_to_server.js
+++ b/remoting/webapp/me2mom/log_to_server.js
@@ -28,10 +28,6 @@ remoting.LogToServer = function() {
   this.sessionStartTime = 0;
 };
 
-// Local storage key.
-/** @private */
-remoting.LogToServer.KEY_ENABLED_ = 'remoting.LogToServer.enabled';
-
 // Constants used for generating a session ID.
 /** @private */
 remoting.LogToServer.SESSION_ID_ALPHABET_ =
@@ -47,16 +43,6 @@ remoting.LogToServer.MAX_SESSION_ID_AGE = 24 * 60 * 60 * 1000;
 remoting.LogToServer.CONNECTION_STATS_ACCUMULATE_TIME = 60 * 1000;
 
 /**
- * Enables or disables logging.
- *
- * @param {boolean} enabled whether logging is enabled
- */
-remoting.LogToServer.prototype.setEnabled = function(enabled) {
-  window.localStorage.setItem(remoting.LogToServer.KEY_ENABLED_,
-     enabled ? 'true' : 'false');
-}
-
-/**
  * Logs a client session state change.
  *
  * @param {remoting.ClientSession.State} state
@@ -200,8 +186,7 @@ remoting.LogToServer.prototype.log = function(entry) {
  * @return {boolean} whether logging is enabled
  */
 remoting.LogToServer.prototype.isEnabled = function() {
-  var value = window.localStorage.getItem(remoting.LogToServer.KEY_ENABLED_);
-  return (value == 'true');
+  return true;

[Jamie, 2012/01/11 20:52:51]

Is it worth keeping isEnabled at all? The only reason I can see is to make it
easy to turn it off in the future if we wanted to, and unless isEnabled is
called in a lot of places, I don't see that being a big win.

[simonmorris, 2012/01/11 21:23:34]

Done.

 };
 
 /**