Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(128)

Issue 88116: Support TR.sectionRowIndex (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
16 years, 11 months ago by felix8a
Modified:
16 years, 10 months ago
Reviewers:
ihab.awad
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Domita doesn't support TR.sectionRowIndex. This change fixes that.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -1 line) Patch
M src/com/google/caja/plugin/domita.js View 2 chunks +4 lines, -1 line 0 comments Download
M tests/com/google/caja/plugin/domita_test_untrusted.html View 2 chunks +27 lines, -0 lines 1 comment Download

Messages

Total messages: 2
felix8a
16 years, 11 months ago (2009-07-07 00:53:09 UTC) #1
ihab.awad
16 years, 11 months ago (2009-07-08 05:58:18 UTC) #2
LGTM++ (comments attached)

http://codereview.appspot.com/88116/diff/1/2
File tests/com/google/caja/plugin/domita_test_untrusted.html (right):

http://codereview.appspot.com/88116/diff/1/2#newcode308
Line 308: <td id="test-table-elements-td-b11">b11</td>
You never end up using the ID test-table-elements-td-b11; to avoid confusion,
please remove it.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b