Escaping text on hidden content dividers

Review Request #4330 — Created July 15, 2013 and submitted — Latest diff uploaded

Information

Review Board

Reviewers

ReviewBoard shows you a certain amount of context around the changes. Above
this you're provided with two options...

  [+] 1005 lines
  [+] public void titlesModified(List<Title> titlesAdd, List<String> titlesDelete,

This is all fine and well, except in the case of content like that shown above.
'Title' coincidently belongs to an html tag, and including it breaks the html
parser for the page (as it would with '</script>' and a few other things). HTML
escaping these lines to prevent this.

The main visible result is that diff hunks beyond the first fail to load. In
firebug you get the error of...

  Uncaught ReferenceError: setFileAttachmentComments is not defined

This is probably the cause of...

  http://code.google.com/p/reviewboard/issues/detail?id=2612

This is available in the divider_escape branch of my repository...

  https://github.com/atagar/ReviewBoard/commit/617f526af36d320910626eac6872714734c5d7dd
Tested against a review with the above divider. It renders just fine and the page source shows that it is now escaped.
    Loading...