http://codereview.appspot.com/98051/diff/1/2 File src/com/google/caja/plugin/domita.js (right): http://codereview.appspot.com/98051/diff/1/2#newcode2586 Line 2586: return defaultTameNode(this.node___.insertCell(0 + index), this.editable___); What do you want (0 + index) to do? It definitely does not guarantee that the result is an integer or that it is in a sensible range. http://codereview.appspot.com/98051/diff/1/2#newcode2627 Line 2627: if (!this.editable___) { throw new Error(NOT_EDITABLE); } Why are create methods contingent on editability? http://codereview.appspot.com/98051
- [Caja] Re: Adds some table manipulation methods that were miss... mikesamuel
- [Caja] Re: Adds some table manipulation methods that were... metaweta
- [Caja] Re: Adds some table manipulation methods that were... mikesamuel
- [Caja] Re: Adds some table manipulation methods that were... mikesamuel
- [Caja] Re: Adds some table manipulation methods that were... metaweta
- [Caja] Re: Adds some table manipulation methods that were... metaweta
- [Caja] Re: Adds some table manipulation methods that were... metaweta
- [Caja] Re: Adds some table manipulation methods that were... mikesamuel
- [Caja] Re: Adds some table manipulation methods that were... metaweta
- [Caja] Re: Adds some table manipulation methods that were... mikesamuel
