From 3a0015c3578083514987b254b2b6fac34a2eb237 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matthias=20Bartelme=C3=9F?= <mba@fourplusone.de>
Date: Sun, 8 Jul 2012 11:51:04 +0200
Subject: [PATCH 01/17] remove unused _opt in favor of code size/readability

---
 src/static/js/Changeset.js | 245 ++++++++++++++-----------------------
 1 file changed, 89 insertions(+), 156 deletions(-)

diff --git a/src/static/js/Changeset.js b/src/static/js/Changeset.js
index 738ee1bab..cfea43625 100644
--- a/src/static/js/Changeset.js
+++ b/src/static/js/Changeset.js
@@ -27,8 +27,6 @@
 
 var AttributePool = require("./AttributePool");
 
-var _opt = null;
-
 /**
  * ==================== General Util Functions =======================
  */
@@ -127,22 +125,13 @@ exports.opIterator = function (opsStr, optStartIndex) {
   function nextRegexMatch() {
     prevIndex = curIndex;
     var result;
-    if (_opt) {
-      result = _opt.nextOpInString(opsStr, curIndex);
-      if (result) {
-        if (result.opcode() == '?') {
-          exports.error("Hit error opcode in op stream");
-        }
-        curIndex = result.lastIndex();
-      }
-    } else {
-      regex.lastIndex = curIndex;
-      result = regex.exec(opsStr);
-      curIndex = regex.lastIndex;
-      if (result[0] == '?') {
-        exports.error("Hit error opcode in op stream");
-      }
+    regex.lastIndex = curIndex;
+    result = regex.exec(opsStr);
+    curIndex = regex.lastIndex;
+    if (result[0] == '?') {
+      exports.error("Hit error opcode in op stream");
     }
+  
     return result;
   }
   var regexResult = nextRegexMatch();
@@ -150,13 +139,7 @@ exports.opIterator = function (opsStr, optStartIndex) {
 
   function next(optObj) {
     var op = (optObj || obj);
-    if (_opt && regexResult) {
-      op.attribs = regexResult.attribs();
-      op.lines = regexResult.lines();
-      op.chars = regexResult.chars();
-      op.opcode = regexResult.opcode();
-      regexResult = nextRegexMatch();
-    } else if ((!_opt) && regexResult[0]) {
+    if (regexResult[0]) {
       op.attribs = regexResult[1];
       op.lines = exports.parseNum(regexResult[2] || 0);
       op.opcode = regexResult[3];
@@ -169,7 +152,7 @@ exports.opIterator = function (opsStr, optStartIndex) {
   }
 
   function hasNext() {
-    return !!(_opt ? regexResult : regexResult[0]);
+    return !!(regexResult[0]);
   }
 
   function lastIndex() {
@@ -414,159 +397,109 @@ exports.smartOpAssembler = function () {
   };
 };
 
-if (_opt) {
-  exports.mergingOpAssembler = function () {
-    var assem = _opt.mergingOpAssembler();
 
-    function append(op) {
-      assem.append(op.opcode, op.chars, op.lines, op.attribs);
-    }
+exports.mergingOpAssembler = function () {
+  // This assembler can be used in production; it efficiently
+  // merges consecutive operations that are mergeable, ignores
+  // no-ops, and drops final pure "keeps".  It does not re-order
+  // operations.
+  var assem = exports.opAssembler();
+  var bufOp = exports.newOp();
 
-    function toString() {
-      return assem.toString();
-    }
+  // If we get, for example, insertions [xxx\n,yyy], those don't merge,
+  // but if we get [xxx\n,yyy,zzz\n], that merges to [xxx\nyyyzzz\n].
+  // This variable stores the length of yyy and any other newline-less
+  // ops immediately after it.
+  var bufOpAdditionalCharsAfterNewline = 0;
 
-    function clear() {
-      assem.clear();
-    }
-
-    function endDocument() {
-      assem.endDocument();
-    }
-
-    return {
-      append: append,
-      toString: toString,
-      clear: clear,
-      endDocument: endDocument
-    };
-  };
-} else {
-  exports.mergingOpAssembler = function () {
-    // This assembler can be used in production; it efficiently
-    // merges consecutive operations that are mergeable, ignores
-    // no-ops, and drops final pure "keeps".  It does not re-order
-    // operations.
-    var assem = exports.opAssembler();
-    var bufOp = exports.newOp();
-
-    // If we get, for example, insertions [xxx\n,yyy], those don't merge,
-    // but if we get [xxx\n,yyy,zzz\n], that merges to [xxx\nyyyzzz\n].
-    // This variable stores the length of yyy and any other newline-less
-    // ops immediately after it.
-    var bufOpAdditionalCharsAfterNewline = 0;
-
-    function flush(isEndDocument) {
-      if (bufOp.opcode) {
-        if (isEndDocument && bufOp.opcode == '=' && !bufOp.attribs) {
-          // final merged keep, leave it implicit
-        } else {
+  function flush(isEndDocument) {
+    if (bufOp.opcode) {
+      if (isEndDocument && bufOp.opcode == '=' && !bufOp.attribs) {
+        // final merged keep, leave it implicit
+      } else {
+        assem.append(bufOp);
+        if (bufOpAdditionalCharsAfterNewline) {
+          bufOp.chars = bufOpAdditionalCharsAfterNewline;
+          bufOp.lines = 0;
           assem.append(bufOp);
-          if (bufOpAdditionalCharsAfterNewline) {
-            bufOp.chars = bufOpAdditionalCharsAfterNewline;
-            bufOp.lines = 0;
-            assem.append(bufOp);
-            bufOpAdditionalCharsAfterNewline = 0;
-          }
+          bufOpAdditionalCharsAfterNewline = 0;
         }
-        bufOp.opcode = '';
       }
+      bufOp.opcode = '';
     }
+  }
 
-    function append(op) {
-      if (op.chars > 0) {
-        if (bufOp.opcode == op.opcode && bufOp.attribs == op.attribs) {
-          if (op.lines > 0) {
-            // bufOp and additional chars are all mergeable into a multi-line op
-            bufOp.chars += bufOpAdditionalCharsAfterNewline + op.chars;
-            bufOp.lines += op.lines;
-            bufOpAdditionalCharsAfterNewline = 0;
-          } else if (bufOp.lines == 0) {
-            // both bufOp and op are in-line
-            bufOp.chars += op.chars;
-          } else {
-            // append in-line text to multi-line bufOp
-            bufOpAdditionalCharsAfterNewline += op.chars;
-          }
+  function append(op) {
+    if (op.chars > 0) {
+      if (bufOp.opcode == op.opcode && bufOp.attribs == op.attribs) {
+        if (op.lines > 0) {
+          // bufOp and additional chars are all mergeable into a multi-line op
+          bufOp.chars += bufOpAdditionalCharsAfterNewline + op.chars;
+          bufOp.lines += op.lines;
+          bufOpAdditionalCharsAfterNewline = 0;
+        } else if (bufOp.lines == 0) {
+          // both bufOp and op are in-line
+          bufOp.chars += op.chars;
         } else {
-          flush();
-          exports.copyOp(op, bufOp);
+          // append in-line text to multi-line bufOp
+          bufOpAdditionalCharsAfterNewline += op.chars;
         }
+      } else {
+        flush();
+        exports.copyOp(op, bufOp);
       }
     }
+  }
 
-    function endDocument() {
-      flush(true);
-    }
+  function endDocument() {
+    flush(true);
+  }
 
-    function toString() {
-      flush();
-      return assem.toString();
-    }
+  function toString() {
+    flush();
+    return assem.toString();
+  }
 
-    function clear() {
-      assem.clear();
-      exports.clearOp(bufOp);
-    }
-    return {
-      append: append,
-      toString: toString,
-      clear: clear,
-      endDocument: endDocument
-    };
+  function clear() {
+    assem.clear();
+    exports.clearOp(bufOp);
+  }
+  return {
+    append: append,
+    toString: toString,
+    clear: clear,
+    endDocument: endDocument
   };
-}
+};
 
-if (_opt) {
-  exports.opAssembler = function () {
-    var assem = _opt.opAssembler();
-    // this function allows op to be mutated later (doesn't keep a ref)
 
-    function append(op) {
-      assem.append(op.opcode, op.chars, op.lines, op.attribs);
+
+exports.opAssembler = function () {
+  var pieces = [];
+  // this function allows op to be mutated later (doesn't keep a ref)
+
+  function append(op) {
+    pieces.push(op.attribs);
+    if (op.lines) {
+      pieces.push('|', exports.numToString(op.lines));
     }
+    pieces.push(op.opcode);
+    pieces.push(exports.numToString(op.chars));
+  }
 
-    function toString() {
-      return assem.toString();
-    }
+  function toString() {
+    return pieces.join('');
+  }
 
-    function clear() {
-      assem.clear();
-    }
-    return {
-      append: append,
-      toString: toString,
-      clear: clear
-    };
+  function clear() {
+    pieces.length = 0;
+  }
+  return {
+    append: append,
+    toString: toString,
+    clear: clear
   };
-} else {
-  exports.opAssembler = function () {
-    var pieces = [];
-    // this function allows op to be mutated later (doesn't keep a ref)
-
-    function append(op) {
-      pieces.push(op.attribs);
-      if (op.lines) {
-        pieces.push('|', exports.numToString(op.lines));
-      }
-      pieces.push(op.opcode);
-      pieces.push(exports.numToString(op.chars));
-    }
-
-    function toString() {
-      return pieces.join('');
-    }
-
-    function clear() {
-      pieces.length = 0;
-    }
-    return {
-      append: append,
-      toString: toString,
-      clear: clear
-    };
-  };
-}
+};
 
 /**
  * A custom made String Iterator

From 4c8f69b7c55f1a8ba8e8d338c2cee6bac0cc85c4 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Sun, 8 Jul 2012 18:59:46 +0200
Subject: [PATCH 02/17] Use v8 to parse settings.json

---
 src/node/utils/Settings.js | 32 +++++++++++++-------------------
 1 file changed, 13 insertions(+), 19 deletions(-)

diff --git a/src/node/utils/Settings.js b/src/node/utils/Settings.js
index e60446df4..aeeb9015e 100644
--- a/src/node/utils/Settings.js
+++ b/src/node/utils/Settings.js
@@ -24,6 +24,7 @@ var os = require("os");
 var path = require('path');
 var argv = require('./Cli').argv;
 var npm = require("npm/lib/npm.js");
+var vm = require('vm');
 
 /* Root path of the installation */
 exports.root = path.normalize(path.join(npm.dir, ".."));
@@ -45,6 +46,7 @@ exports.dbType = "dirty";
  * This setting is passed with dbType to ueberDB to set up the database
  */
 exports.dbSettings = { "filename" : path.join(exports.root, "dirty.db") };
+
 /**
  * The default Text of a new pad
  */
@@ -102,33 +104,25 @@ exports.abiwordAvailable = function()
 
 // Discover where the settings file lives
 var settingsFilename = argv.settings || "settings.json";
-if (settingsFilename.charAt(0) != '/') {
-    settingsFilename = path.normalize(path.join(root, settingsFilename));
-}
+settingsFilename = path.resolve(path.join(root, settingsFilename));
 
-var settingsStr
+var settingsStr;
 try{
   //read the settings sync
-  settingsStr = fs.readFileSync(settingsFilename).toString();
+  settingsStr = fs.readFileSync(settingsFilename);
 } catch(e){
   console.warn('No settings file found. Using defaults.');
-  settingsStr = '{}';
 }
-  
-//remove all comments
-settingsStr = settingsStr.replace(/\*([^*]|[\r\n]|(\*+([^*/]|[\r\n])))*\*+/gm,"").replace(/#.*/g,"").replace(/\/\/.*/g,"");
 
-//try to parse the settings
+// try to parse the settings
 var settings;
-try
-{
-  settings = JSON.parse(settingsStr);
-}
-catch(e)
-{
-  console.error("There is a syntax error in your settings.json file");
-  console.error(e.message);
-  process.exit(1);
+try {
+  if(settingsStr) {
+    settings = vm.runInContext('exports = '+settingsStr, vm.createContext(), "settings.json");
+  }
+}catch(e){
+  console.warn('There was an error processing your settings.json file. Using defaults.');
+  console.warn(e.message);
 }
 
 //loop trough the settings

From 975171a86b1c3eae8bebb91e1b87f6efcf990ec3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matthias=20Bartelme=C3=9F?= <mba@fourplusone.de>
Date: Sun, 8 Jul 2012 21:06:19 +0200
Subject: [PATCH 03/17] Make handleMessage async

---
 src/node/handler/PadMessageHandler.js | 34 ++++++++++++++++++++-------
 1 file changed, 26 insertions(+), 8 deletions(-)

diff --git a/src/node/handler/PadMessageHandler.js b/src/node/handler/PadMessageHandler.js
index a0aef6648..d02e65377 100644
--- a/src/node/handler/PadMessageHandler.js
+++ b/src/node/handler/PadMessageHandler.js
@@ -159,11 +159,7 @@ exports.handleDisconnect = function(client)
  */
 exports.handleMessage = function(client, message)
 { 
-  _.map(hooks.callAll( "handleMessage", { client: client, message: message }), function ( newmessage ) {
-    if ( newmessage || newmessage === null ) {
-      message = newmessage;
-    }
-  });
+
   if(message == null)
   {
     messageLogger.warn("Message is null!");
@@ -175,6 +171,23 @@ exports.handleMessage = function(client, message)
     return;
   }
 
+  var handleMessageHook = function(callback){
+    var dropMessage = false;
+    
+    // Call handleMessage hook. If a plugin returns null, the message will be dropped. Note that for all messages 
+    // handleMessage will be called, even if the client is not authorized
+    hooks.aCallAll("handleMessage", { client: client, message: message }, function ( messages ) {
+      _.each(messages, function(newMessage){
+        if ( newmessage === null ) {
+          dropMessage = true;
+        }
+      });
+      
+      // If no plugins explicitly told us to drop the message, its ok to proceed
+      if(!dropMessage){ callback() };
+    });
+  }
+
   var finalHandler = function () {
     //Check what type of message we get and delegate to the other methodes
     if(message.type == "CLIENT_READY") {
@@ -203,11 +216,18 @@ exports.handleMessage = function(client, message)
     }
   };
 
-  if (message && message.padId) {
+  if (message) {
     async.series([
+      handleMessageHook,
       //check permissions
       function(callback)
       {
+        
+        if(!message.padId){
+          // If the message has a padId we assume the client is already known to the server and needs no re-authorization
+          callback();
+          return;
+        }
         // Note: message.sessionID is an entirely different kind of
         // session from the sessions we use here! Beware! FIXME: Call
         // our "sessions" "connections".
@@ -231,8 +251,6 @@ exports.handleMessage = function(client, message)
       },
       finalHandler
     ]);
-  } else {
-    finalHandler();
   }
 }
 

From 885844667887411c8fc2822ab1914b7277bcb4ca Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Tue, 10 Jul 2012 21:38:14 +0200
Subject: [PATCH 04/17] Exit on error.

---
 src/node/utils/Settings.js | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/node/utils/Settings.js b/src/node/utils/Settings.js
index aeeb9015e..68e58eb22 100644
--- a/src/node/utils/Settings.js
+++ b/src/node/utils/Settings.js
@@ -111,7 +111,8 @@ try{
   //read the settings sync
   settingsStr = fs.readFileSync(settingsFilename);
 } catch(e){
-  console.warn('No settings file found. Using defaults.');
+  console.error('No settings file found.');
+  process.exit(1);
 }
 
 // try to parse the settings
@@ -121,8 +122,8 @@ try {
     settings = vm.runInContext('exports = '+settingsStr, vm.createContext(), "settings.json");
   }
 }catch(e){
-  console.warn('There was an error processing your settings.json file. Using defaults.');
-  console.warn(e.message);
+  console.error('There was an error processing your settings.json file: '+e.message);
+  process.exit(1);
 }
 
 //loop trough the settings
@@ -142,8 +143,7 @@ for(var i in settings)
   //this setting is unkown, output a warning and throw it away
   else
   {
-    console.warn("Unkown Setting: '" + i + "'");
-    console.warn("This setting doesn't exist or it was removed");
+    console.warn("Unkown Setting: '" + i + "'. This setting doesn't exist or it was removed");
   }
 }
 

From 87f26334d1259f6422c3a2b7d9325b611c72316b Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Tue, 10 Jul 2012 21:55:35 +0200
Subject: [PATCH 05/17] Fix typo.

---
 src/node/utils/Settings.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/node/utils/Settings.js b/src/node/utils/Settings.js
index 68e58eb22..205b675f2 100644
--- a/src/node/utils/Settings.js
+++ b/src/node/utils/Settings.js
@@ -143,7 +143,7 @@ for(var i in settings)
   //this setting is unkown, output a warning and throw it away
   else
   {
-    console.warn("Unkown Setting: '" + i + "'. This setting doesn't exist or it was removed");
+    console.warn("Unknown Setting: '" + i + "'. This setting doesn't exist or it was removed");
   }
 }
 

From f09dd0f3fba8630d221dba3a652dad0a229aa509 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Wed, 11 Jul 2012 15:34:33 +0200
Subject: [PATCH 06/17] Put toString() back in.

---
 src/node/utils/Settings.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/node/utils/Settings.js b/src/node/utils/Settings.js
index 205b675f2..1b7c3f46c 100644
--- a/src/node/utils/Settings.js
+++ b/src/node/utils/Settings.js
@@ -109,7 +109,7 @@ settingsFilename = path.resolve(path.join(root, settingsFilename));
 var settingsStr;
 try{
   //read the settings sync
-  settingsStr = fs.readFileSync(settingsFilename);
+  settingsStr = fs.readFileSync(settingsFilename).toString();
 } catch(e){
   console.error('No settings file found.');
   process.exit(1);

From dc09323d8f6ee5a65f301bb677d9e18a54672322 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Wed, 11 Jul 2012 15:36:41 +0200
Subject: [PATCH 07/17] Don't exit if no settings file was found.

---
 src/node/utils/Settings.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/node/utils/Settings.js b/src/node/utils/Settings.js
index 1b7c3f46c..dd34ac5ee 100644
--- a/src/node/utils/Settings.js
+++ b/src/node/utils/Settings.js
@@ -111,8 +111,7 @@ try{
   //read the settings sync
   settingsStr = fs.readFileSync(settingsFilename).toString();
 } catch(e){
-  console.error('No settings file found.');
-  process.exit(1);
+  console.warn('No settings file found. Continuing using defaults!');
 }
 
 // try to parse the settings

From 6726ea66320e3075451027d5362d35d77bebaf5d Mon Sep 17 00:00:00 2001
From: John McLear <john@mclear.co.uk>
Date: Wed, 11 Jul 2012 17:42:59 +0100
Subject: [PATCH 08/17] option to stop autoscroll

---
 src/static/js/chat.js  | 7 +++++--
 src/templates/pad.html | 4 ++++
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/static/js/chat.js b/src/static/js/chat.js
index 47b0ae3ca..b2a7d2737 100644
--- a/src/static/js/chat.js
+++ b/src/static/js/chat.js
@@ -62,8 +62,11 @@ var chat = (function()
     },
     scrollDown: function()
     {
-      if($('#chatbox').css("display") != "none")
-        $('#chattext').animate({scrollTop: $('#chattext')[0].scrollHeight}, "slow");
+      if($('#options-scrollchat').is(':checked')){
+        if($('#chatbox').css("display") != "none"){
+          $('#chattext').animate({scrollTop: $('#chattext')[0].scrollHeight}, "slow");
+        }
+      }
     }, 
     send: function()
     {
diff --git a/src/templates/pad.html b/src/templates/pad.html
index 02af9b107..a25133472 100644
--- a/src/templates/pad.html
+++ b/src/templates/pad.html
@@ -168,6 +168,10 @@
                     <input type="checkbox" id="options-linenoscheck" checked>
                     <label for="options-linenoscheck">Line numbers</label>
                 </p>
+                <p>
+                    <input type="checkbox" id="options-scrollchat" checked>
+                    <label for="options-scrollchat">Automatically Scroll Chat</label>
+                </p>
                 <p>
                     Font type:
                     <select id="viewfontmenu">

From 8a471e590a38c0b453d00ae474e729b72cb2b7c1 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Wed, 11 Jul 2012 22:30:03 +0200
Subject: [PATCH 09/17] Fix #581 Don't scroll chat automatically if user is not
 at the bottom

---
 src/static/js/chat.js | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/static/js/chat.js b/src/static/js/chat.js
index b2a7d2737..28ceda975 100644
--- a/src/static/js/chat.js
+++ b/src/static/js/chat.js
@@ -62,9 +62,10 @@ var chat = (function()
     },
     scrollDown: function()
     {
-      if($('#options-scrollchat').is(':checked')){
-        if($('#chatbox').css("display") != "none"){
+      if($('#chatbox').css("display") != "none"){
+        if(!self.lastMessage || self.lastMessage.position().top < $('#chattext').height()) {
           $('#chattext').animate({scrollTop: $('#chattext')[0].scrollHeight}, "slow");
+          self.lastMessage = $('#chattext > p').eq(-1);
         }
       }
     }, 

From 7653db36fe0cb362f6775ba8be76440e4bb72585 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Thu, 12 Jul 2012 10:23:55 +0200
Subject: [PATCH 10/17] Revert "option to stop autoscroll"

This reverts commit 6726ea66320e3075451027d5362d35d77bebaf5d.

Conflicts:

	src/static/js/chat.js
---
 src/templates/pad.html | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/templates/pad.html b/src/templates/pad.html
index a25133472..02af9b107 100644
--- a/src/templates/pad.html
+++ b/src/templates/pad.html
@@ -168,10 +168,6 @@
                     <input type="checkbox" id="options-linenoscheck" checked>
                     <label for="options-linenoscheck">Line numbers</label>
                 </p>
-                <p>
-                    <input type="checkbox" id="options-scrollchat" checked>
-                    <label for="options-scrollchat">Automatically Scroll Chat</label>
-                </p>
                 <p>
                     Font type:
                     <select id="viewfontmenu">

From f12cb0dd58c98cd08f3c30b4a21d9e7289d5414e Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Thu, 12 Jul 2012 10:34:11 +0200
Subject: [PATCH 11/17] Don't screw up, if chat is empty.

---
 src/static/js/chat.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/static/js/chat.js b/src/static/js/chat.js
index 28ceda975..18904352f 100644
--- a/src/static/js/chat.js
+++ b/src/static/js/chat.js
@@ -63,7 +63,7 @@ var chat = (function()
     scrollDown: function()
     {
       if($('#chatbox').css("display") != "none"){
-        if(!self.lastMessage || self.lastMessage.position().top < $('#chattext').height()) {
+        if(!self.lastMessage || !self.lastMessage.position() || self.lastMessage.position().top < $('#chattext').height()) {
           $('#chattext').animate({scrollTop: $('#chattext')[0].scrollHeight}, "slow");
           self.lastMessage = $('#chattext > p').eq(-1);
         }

From dc9eda9364eec93983ba95ea3560c5c72296af0b Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Thu, 12 Jul 2012 20:18:33 +0200
Subject: [PATCH 12/17] Use Tinycon to display chat mentions in favicon.

---
 src/static/js/chat.js    |  12 +-
 src/static/js/tinycon.js | 237 +++++++++++++++++++++++++++++++++++++++
 2 files changed, 242 insertions(+), 7 deletions(-)
 create mode 100644 src/static/js/tinycon.js

diff --git a/src/static/js/chat.js b/src/static/js/chat.js
index 18904352f..9dfc9dc1f 100644
--- a/src/static/js/chat.js
+++ b/src/static/js/chat.js
@@ -23,11 +23,12 @@
 var padutils = require('./pad_utils').padutils;
 var padcookie = require('./pad_cookie').padcookie;
 
+require('./tinycon');
+
 var chat = (function()
 {
   var isStuck = false;
   var chatMentions = 0;
-  var title = document.title;
   var self = {
     show: function () 
     {      
@@ -35,7 +36,7 @@ var chat = (function()
       $("#chatbox").show();
       self.scrollDown();
       chatMentions = 0;
-      document.title = title;
+      Tinycon.setBubble(0);
     },
     stickToScreen: function(fromInitialCall) // Make chat stick to right hand side of screen
     {
@@ -126,12 +127,9 @@ var chat = (function()
         // chat throb stuff -- Just make it throw for twice as long
         if(wasMentioned && !alreadyFocused)
         { // If the user was mentioned show for twice as long and flash the browser window
-          if (chatMentions == 0){
-            title = document.title;
-          }
           $('#chatthrob').html("<b>"+authorName+"</b>" + ": " + text).show().delay(4000).hide(400);
           chatMentions++;
-          document.title = "("+chatMentions+") " + title;
+          Tinycon.setBubble(chatMentions);
         }
         else
         {
@@ -141,7 +139,7 @@ var chat = (function()
        // Clear the chat mentions when the user clicks on the chat input box
       $('#chatinput').click(function(){
         chatMentions = 0;
-        document.title = title;
+        Tinycon.setBubble(0);
       });
       self.scrollDown();
 
diff --git a/src/static/js/tinycon.js b/src/static/js/tinycon.js
new file mode 100644
index 000000000..0e6a10a10
--- /dev/null
+++ b/src/static/js/tinycon.js
@@ -0,0 +1,237 @@
+/*!
+ * Tinycon - A small library for manipulating the Favicon
+ * Tom Moor, http://tommoor.com
+ * Copyright (c) 2012 Tom Moor
+ * MIT Licensed
+ * @version 0.2.6
+*/
+
+(function(){
+	
+	var Tinycon = {};
+	var currentFavicon = null;
+	var originalFavicon = null;
+	var originalTitle = document.title;
+	var faviconImage = null;
+	var canvas = null;
+	var options = {};
+	var defaults = {
+		width: 7,
+		height: 9,
+		font: '10px arial',
+		colour: '#ffffff',
+		background: '#F03D25',
+		fallback: true
+	};
+	
+	var ua = (function () {
+		var agent = navigator.userAgent.toLowerCase();
+		// New function has access to 'agent' via closure
+		return function (browser) {
+			return agent.indexOf(browser) !== -1;
+		};
+	}());
+
+	var browser = {
+		ie: ua('msie'),
+		chrome: ua('chrome'),
+		webkit: ua('chrome') || ua('safari'),
+		safari: ua('safari') && !ua('chrome'),
+		mozilla: ua('mozilla') && !ua('chrome') && !ua('safari')
+	};
+	
+	// private methods
+	var getFaviconTag = function(){
+		
+		var links = document.getElementsByTagName('link');
+		
+		for(var i=0, len=links.length; i < len; i++) {
+			if ((links[i].getAttribute('rel') || '').match(/\bicon\b/)) {
+				return links[i];
+			}
+		}
+		
+		return false;
+	};
+	
+	var removeFaviconTag = function(){
+	
+		var links = document.getElementsByTagName('link');
+		var head = document.getElementsByTagName('head')[0];
+		
+		for(var i=0, len=links.length; i < len; i++) {
+			var exists = (typeof(links[i]) !== 'undefined');
+			if (exists && links[i].getAttribute('rel') === 'icon') {
+				head.removeChild(links[i]);
+			}
+		}
+	};
+	
+	var getCurrentFavicon = function(){
+		
+		if (!originalFavicon || !currentFavicon) {
+			var tag = getFaviconTag();
+			originalFavicon = currentFavicon = tag ? tag.getAttribute('href') : '/favicon.ico';
+		}
+
+		return currentFavicon;
+	};
+	
+	var getCanvas = function (){
+		
+		if (!canvas) {
+			canvas = document.createElement("canvas");
+			canvas.width = 16;
+			canvas.height = 16;
+		}
+		
+		return canvas;
+	};
+	
+	var setFaviconTag = function(url){
+		removeFaviconTag();
+		
+		var link = document.createElement('link');
+		link.type = 'image/x-icon';
+		link.rel = 'icon';
+		link.href = url;
+		document.getElementsByTagName('head')[0].appendChild(link);
+	};
+	
+	var log = function(message){
+		if (window.console) window.console.log(message);
+	};
+	
+	var drawFavicon = function(num, colour) {
+
+		// fallback to updating the browser title if unsupported
+		if (!getCanvas().getContext || browser.ie || browser.safari || options.fallback === 'force') {
+			return updateTitle(num);
+		}
+		
+		var context = getCanvas().getContext("2d");
+		var colour = colour || '#000000';
+		var num = num || 0;
+		var src = getCurrentFavicon();
+		
+		faviconImage = new Image();
+		faviconImage.onload = function() {
+			
+			// clear canvas  
+			context.clearRect(0, 0, 16, 16);
+
+			// draw original favicon
+			context.drawImage(faviconImage, 0, 0, faviconImage.width, faviconImage.height, 0, 0, 16, 16);
+			
+			// draw bubble over the top
+			if (num > 0) drawBubble(context, num, colour);
+			
+			// refresh tag in page
+			refreshFavicon();
+		};
+		
+		// allow cross origin resource requests if the image is not a data:uri
+		// as detailed here: https://github.com/mrdoob/three.js/issues/1305
+		if (!src.match(/^data/)) {
+			faviconImage.crossOrigin = 'anonymous';
+		}
+		
+		faviconImage.src = src;
+	};
+	
+	var updateTitle = function(num) {
+		
+		if (options.fallback) {
+			if (num > 0) {
+				document.title = '('+num+') ' + originalTitle;
+			} else {
+				document.title = originalTitle;
+			}
+		}
+	};
+	
+	var drawBubble = function(context, num, colour) {
+		
+		// bubble needs to be larger for double digits
+		var len = (num+"").length-1;
+		var width = options.width + (6*len);
+		var w = 16-width;
+		var h = 16-options.height;
+
+		// webkit seems to render fonts lighter than firefox
+		context.font = (browser.webkit ? 'bold ' : '') + options.font;
+		context.fillStyle = options.background;
+		context.strokeStyle = options.background;
+		context.lineWidth = 1;
+		
+		// bubble
+		context.fillRect(w,h,width-1,options.height);
+		
+		// rounded left
+		context.beginPath();
+		context.moveTo(w-0.5,h+1);
+		context.lineTo(w-0.5,15);
+		context.stroke();
+		
+		// rounded right
+		context.beginPath();
+		context.moveTo(15.5,h+1);
+		context.lineTo(15.5,15);
+		context.stroke();
+		
+		// bottom shadow
+		context.beginPath();
+		context.strokeStyle = "rgba(0,0,0,0.3)";
+		context.moveTo(w,16);
+		context.lineTo(15,16);
+		context.stroke();
+		
+		// number
+		context.fillStyle = options.colour;
+		context.textAlign = "right";
+		context.textBaseline = "top";
+		
+		// unfortunately webkit/mozilla are a pixel different in text positioning
+		context.fillText(num, 15, browser.mozilla ? 7 : 6);  
+	};
+	
+	var refreshFavicon = function(){
+		// check support
+		if (!getCanvas().getContext) return;
+		
+		setFaviconTag(getCanvas().toDataURL());
+	};
+	
+	
+	// public methods
+	Tinycon.setOptions = function(custom){
+		options = {};
+		
+		for(var key in defaults){
+			options[key] = custom.hasOwnProperty(key) ? custom[key] : defaults[key];
+		}
+		return this;
+	};
+	
+	Tinycon.setImage = function(url){
+		currentFavicon = url;
+		refreshFavicon();
+		return this;
+	};
+	
+	Tinycon.setBubble = function(num, colour){
+		
+		// validate
+		if(isNaN(parseFloat(num)) || !isFinite(num)) return log('Bubble must be a number');
+		
+		drawFavicon(num, colour);
+		return this;
+	};
+	
+	Tinycon.reset = function(){
+		Tinycon.setImage(originalFavicon);
+	};
+	
+	Tinycon.setOptions(defaults);
+	window.Tinycon = Tinycon;
+})();

From b1123d11b60b6325e0f3dc5cd343814cd2c264bf Mon Sep 17 00:00:00 2001
From: 0ip <me@factor.cc>
Date: Sat, 14 Jul 2012 20:35:26 +0300
Subject: [PATCH 13/17] Correct path

---
 src/templates/admin/plugins.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/templates/admin/plugins.html b/src/templates/admin/plugins.html
index 104f6b986..3dad3bd01 100644
--- a/src/templates/admin/plugins.html
+++ b/src/templates/admin/plugins.html
@@ -21,7 +21,7 @@
 
       <h1>Etherpad Lite</h1>
 
-      <a href="/admin/plugins/info">Technical information on installed plugins</a>
+      <a href="plugins/info">Technical information on installed plugins</a>
 
       <div class="separator"></div>
       <h2>Installed plugins</h2>

From 9bd23acb3dd14e381b7c97041a2a70712890761d Mon Sep 17 00:00:00 2001
From: Mark Holmquist <mtraceur@member.fsf.org>
Date: Tue, 17 Jul 2012 10:12:10 -0700
Subject: [PATCH 14/17] Add userJoinOrUpdate hook

This hook fires on the client side, whenever a user joins or
updates (hence the name). It will pass one thing, the user's info,
into the context. This is mostly just for notification purposes.
---
 src/static/js/pad_userlist.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/static/js/pad_userlist.js b/src/static/js/pad_userlist.js
index d83825854..474f9b0e1 100644
--- a/src/static/js/pad_userlist.js
+++ b/src/static/js/pad_userlist.js
@@ -21,6 +21,7 @@
  */
 
 var padutils = require('./pad_utils').padutils;
+var hooks = require('./pluginfw/hooks');
 
 var myUserInfo = {};
 
@@ -529,6 +530,10 @@ var paduserlist = (function()
         return;
       }
 
+      hooks.callAll('userJoinOrUpdate', {
+        userInfo: info
+      });
+
       var userData = {};
       userData.color = typeof info.colorId == "number" ? clientVars.colorPalette[info.colorId] : info.colorId;
       userData.name = info.name;

From 9aed433ad5947fbbeeac1bf94c0e4961f0833087 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Wed, 18 Jul 2012 15:54:53 +0200
Subject: [PATCH 15/17] Fix #880 Remove call to
 padsavedrevs.handleIsFullyConnected

---
 src/static/js/pad.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/static/js/pad.js b/src/static/js/pad.js
index df6342e2d..a22e181a2 100644
--- a/src/static/js/pad.js
+++ b/src/static/js/pad.js
@@ -790,8 +790,6 @@ var pad = {
       }, 1000);
     }
 
-    padsavedrevs.handleIsFullyConnected(isConnected);
-
     // pad.determineSidebarVisibility(isConnected && !isInitialConnect);
     pad.determineChatVisibility(isConnected && !isInitialConnect);
     pad.determineAuthorshipColorsVisibility();

From c3ddff3fa061a48ae7178c62038e143ab5304ec6 Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Thu, 19 Jul 2012 11:34:14 +0200
Subject: [PATCH 16/17] Notify the user, if JavaScript is disabled.

---
 src/templates/pad.html | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/templates/pad.html b/src/templates/pad.html
index 02af9b107..36d6334c2 100644
--- a/src/templates/pad.html
+++ b/src/templates/pad.html
@@ -148,7 +148,10 @@
 
         <div id="editorcontainerbox">
             <div id="editorcontainer"></div>
-            <div id="editorloadingbox">Loading...</div>
+            <div id="editorloadingbox">
+              <p>Loading...</p>
+              <noscript><strong>Sorry, you have to enable Javascript in order to use this.</strong></noscript>
+            </div>
         </div>
 
         <div id="settings" class="popup">

From fa3020dffdab4a69bba045965ca038ca3d21575a Mon Sep 17 00:00:00 2001
From: Marcel Klehr <mklehr@gmx.net>
Date: Mon, 23 Jul 2012 14:12:36 +0200
Subject: [PATCH 17/17] Display errors occuring on page load to the user.

---
 src/templates/pad.html | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/templates/pad.html b/src/templates/pad.html
index 36d6334c2..6821b5e6c 100644
--- a/src/templates/pad.html
+++ b/src/templates/pad.html
@@ -316,6 +316,19 @@
         <% e.end_block(); %>
 
         <% e.begin_block("scripts"); %>
+        <script type="text/javascript">
+            /* Display errors on page load to the user 
+               (Gets overridden by padutils.setupGlobalExceptionHandler)
+            */
+            window.onerror = function(msg, url, line) {
+              console.log('error', arguments);
+              var box   = document.getElementById('editorloadingbox');
+              box.innerHTML = '<p><b>An error occured while loading the pad</b></p>'
+                            + '<p><b>'+msg+'</b> '
+                            + '<small>in '+ url +' (line '+ line +')</small></p>'
+            };
+        </script>
+
         <script type="text/javascript" src="../static/js/require-kernel.js"></script>
         <script type="text/javascript" src="../socket.io/socket.io.js"></script>