Fix misplaced directive prologues in rewriter output

Some Quasi Rewrite rules of the form
  { @ss* }  ->  { foo(); @ss* }
are problematic because when *ss; matches a directive prologue, the
output will include the prologue in a context where it is likely to be
misinterpreted.
E.g. if @ss* in the above matches "use strict"; bar(), then the output
would be
  { foo(); "use strict"; bar() }
and when rendered and reparsed, the "use strict"; is interpreted as a
string, not a directive prologue.


Submitted @3962

[MikeSamuel, 2010-01-08 17:09:48]

ping

[MikeSamuel, 2010-01-22 19:01:44]

ping

[ihab.awad, 2010-02-01 06:08:37]

LGTM++

http://codereview.appspot.com/181060/diff/1/2
File src/com/google/caja/parser/quasiliteral/SimpleQuasiNode.java (right):

http://codereview.appspot.com/181060/diff/1/2#newcode120
src/com/google/caja/parser/quasiliteral/SimpleQuasiNode.java:120: // assumption
here that everything is an AbstractParseTreeNode.
While you're at it, could you please delete the above comment since it's no
longer applicable? Tx.

[MikeSamuel, 2010-02-01 09:22:30]

http://codereview.appspot.com/181060/diff/1/2
File src/com/google/caja/parser/quasiliteral/SimpleQuasiNode.java (right):

http://codereview.appspot.com/181060/diff/1/2#newcode120
src/com/google/caja/parser/quasiliteral/SimpleQuasiNode.java:120: // assumption
here that everything is an AbstractParseTreeNode.
On 2010/02/01 06:08:37, ihab.awad wrote:
> While you're at it, could you please delete the above comment since it's no
> longer applicable? Tx.

Done.