Bump version to 2.9.2

Signed-off-by: Paris Kasidiaris <paris@sourcelair.com>

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "xterm.js",
-  "version": "2.9.1",
+  "version": "2.9.2",
   "ignore": ["demo", "test", ".gitignore"],
   "main": [
     "dist/xterm.js",

dist/xterm.js

@@ -41,7 +41,7 @@ var Buffer = (function () {
             var ch = [this._terminal.defAttr, ' ', 1];
             for (var i = 0; i < this._lines.length; i++) {
                 if (this._lines.get(i) === undefined) {
-                    this._lines.set(i, this._terminal.blankLine());
+                    this._lines.set(i, this._terminal.blankLine(undefined, undefined, newCols));
                 }
                 while (this._lines.get(i).length < newCols) {
                     this._lines.get(i).push(ch);
@@ -60,7 +60,7 @@ var Buffer = (function () {
                         }
                     }
                     else {
-                        this._lines.push(this._terminal.blankLine());
+                        this._lines.push(this._terminal.blankLine(undefined, undefined, newCols));
                     }
                 }
             }
@@ -4946,7 +4946,7 @@ Terminal.prototype.clear = function () {
 Terminal.prototype.eraseLine = function (y) {
     this.eraseRight(0, y);
 };
-Terminal.prototype.blankLine = function (cur, isWrapped) {
+Terminal.prototype.blankLine = function (cur, isWrapped, cols) {
     var attr = cur
         ? this.eraseAttr()
         : this.defAttr;
@@ -4954,7 +4954,8 @@ Terminal.prototype.blankLine = function (cur, isWrapped) {
     if (isWrapped) {
         line.isWrapped = isWrapped;
     }
-    for (; i < this.cols; i++) {
+    cols = cols || this.cols;
+    for (; i < cols; i++) {
         line[i] = ch;
     }
     return line;

package.json

@@ -1,7 +1,7 @@
 {
   "name": "xterm",
   "description": "Full xterm terminal, in your browser",
-  "version": "2.9.1",
+  "version": "2.9.2",
   "ignore": [
     "demo",
     "test",

src/Buffer.test.ts

@@ -143,5 +143,16 @@ describe('Buffer', () => {
         });
       });
     });
+
+    describe('row and column increased', () => {
+      it('should resize properly', () => {
+        buffer.fillViewportRows();
+        buffer.resize(INIT_COLS + 5, INIT_ROWS + 5);
+        assert.equal(buffer.lines.length, INIT_ROWS + 5);
+        for (let i = 0; i < INIT_ROWS + 5; i++) {
+          assert.equal(buffer.lines.get(i).length, INIT_COLS + 5);
+        }
+      });
+    });
   });
 });

src/Buffer.ts

@@ -85,8 +85,10 @@ export class Buffer implements IBuffer {
     if (this._terminal.cols < newCols) {
       const ch: [number, string, number] = [this._terminal.defAttr, ' ', 1]; // does xterm use the default attr?
       for (let i = 0; i < this._lines.length; i++) {
+        // TODO: This should be removed, with tests setup for the case that was
+        // causing the underlying bug, see https://github.com/sourcelair/xterm.js/issues/824
         if (this._lines.get(i) === undefined) {
-          this._lines.set(i, this._terminal.blankLine());
+          this._lines.set(i, this._terminal.blankLine(undefined, undefined, newCols));
         }
         while (this._lines.get(i).length < newCols) {
           this._lines.get(i).push(ch);
@@ -111,7 +113,7 @@ export class Buffer implements IBuffer {
           } else {
             // Add a blank line if there is no buffer left at the top to scroll to, or if there
             // are blank lines after the cursor
-            this._lines.push(this._terminal.blankLine());
+            this._lines.push(this._terminal.blankLine(undefined, undefined, newCols));
           }
         }
       }

src/Interfaces.ts

@@ -48,7 +48,7 @@ export interface ITerminal {
   emit(event: string, data: any);
   reset(): void;
   showCursor(): void;
-  blankLine(cur?: boolean, isWrapped?: boolean);
+  blankLine(cur?: boolean, isWrapped?: boolean, cols?: number);
 }
 
 export interface IBuffer {

src/utils/TestUtils.ts

@@ -43,9 +43,10 @@ export class MockTerminal implements ITerminal {
   showCursor(): void {
     throw new Error('Method not implemented.');
   }
-  blankLine(cur?: boolean, isWrapped?: boolean) {
+  blankLine(cur?: boolean, isWrapped?: boolean, cols?: number) {
     const line = [];
-    for (let i = 0; i < this.cols; i++) {
+    cols = cols || this.cols;
+    for (let i = 0; i < cols; i++) {
       line.push([0, ' ', 1]);
     }
     return line;

src/xterm.js

@@ -2091,7 +2091,7 @@ Terminal.prototype.eraseLine = function(y) {
  * @param {number} cur First bunch of data for each "blank" character.
  * @param {boolean} isWrapped Whether the new line is wrapped from the previous line.
  */
-Terminal.prototype.blankLine = function(cur, isWrapped) {
+Terminal.prototype.blankLine = function(cur, isWrapped, cols) {
   var attr = cur
   ? this.eraseAttr()
   : this.defAttr;
@@ -2106,7 +2106,8 @@ Terminal.prototype.blankLine = function(cur, isWrapped) {
     line.isWrapped = isWrapped;
   }
 
-  for (; i < this.cols; i++) {
+  cols = cols || this.cols;
+  for (; i < cols; i++) {
     line[i] = ch;
   }