Reviewers: MikeSamuel, DavidSarah,

Description:
Fix problems found by David-Sarah Hopwood after the following change was
already submitted:

  http://codereview.appspot.com/136053

The changes made here are:

1. Documentation fixes.

2. Parsing of directive prologues no longer tries to do any semicolon
insertion.

3. Fixed bug in conditional rendering of Directive -- the Directive
should not render itself if it contains escape sequences, but the
original code did this check incorrectly.

Please review this at http://codereview.appspot.com/156090

Affected files:
  M     src/com/google/caja/parser/js/Directive.java
  M     src/com/google/caja/parser/js/DirectivePrologue.java
  M     src/com/google/caja/parser/js/Parser.java
  M     tests/com/google/caja/parser/js/ParserTest.java
  M     tests/com/google/caja/parser/js/parsergolden10.txt
  M     tests/com/google/caja/parser/js/parsertest10.js
  M     tests/com/google/caja/parser/js/parsertest6.js


Reply via email to