Avoid null dereference in updateStatistics().

remoting/webapp/me2mom/remoting.js

Index: remoting/webapp/me2mom/remoting.js
diff --git a/remoting/webapp/me2mom/remoting.js b/remoting/webapp/me2mom/remoting.js
index acd1d1327eb2368a7be41cc799657ea62aa8249f..9c0760a045c15efd84cce98332f3581a56d554c5 100644
--- a/remoting/webapp/me2mom/remoting.js
+++ b/remoting/webapp/me2mom/remoting.js
@@ -358,6 +358,8 @@ remoting.cancelShare = function() {
 }
 
 function updateStatistics() {
+  if (!remoting.session)
+    return;

[Jamie, 2011/08/25 22:57:14]

I think I'd combine these with || rather than having two tests.

[Lambros, 2011/08/25 23:11:30]

I prefer to leave as-is, if that's OK.  I find constructs such as
  if (!X || !Y)
hard to understand.  Also, this would push the single if.. test over the 80-char
limit.

   if (remoting.session.state != remoting.ClientSession.State.CONNECTED)
     return;
   var stats = remoting.session.stats();