Changeset 15526 in webkit for trunk/JavaScriptCore/kjs/nodes2string.cpp

Timestamp:

Jul 19, 2006, 10:32:15 AM (19 years ago)

Author:

andersca

Message:

JavaScriptCore:

2006-07-19 Anders Carlsson <​[email protected]>

Reviewed by Darin.

<rdar://problem/4620655> REGRESSION(10.4.7-10.5): preview button for a blogger.com post doesn't work

• kjs/nodes2string.cpp: (StringNode::streamTo): Return the escaped string.

(RegExpNode::streamTo):
Use the correct syntax.

• kjs/function.cpp: (KJS::escapeStringForPrettyPrinting):
• kjs/function.h: Add escape function which escapes a string for pretty-printing so it can be parsed again.

• wtf/unicode/icu/UnicodeIcu.h: (WTF::Unicode::isPrintableChar): New function.

LayoutTests:

2006-07-19 Anders Carlsson <​[email protected]>

Reviewed by Darin.

<rdar://problem/4620655> REGRESSION(10.4.7-10.5): preview button for a blogger.com post doesn't work

• fast/js/pretty-print-expected.txt: Added.
• fast/js/pretty-print.html: Added.
• fast/js/resources/pretty-print.js: Added.

File:

• trunk/JavaScriptCore/kjs/nodes2string.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/nodes2string.cpp

@@ -23 +23 @@
 #include "config.h"
 #include "nodes.h"
+#include "function.h"
 
 namespace KJS {
@@ -117 +118 @@
 void StringNode::streamTo(SourceStream &s) const
 {
-  s << '"' << value << '"';
-}
-
-void RegExpNode::streamTo(SourceStream &s) const { s <<  pattern; }
+  s << '"' << escapeStringForPrettyPrinting(value) << '"';
+}
+
+void RegExpNode::streamTo(SourceStream &s) const
+{ 
+    s << "/" <<  pattern << "/" << flags; 
+}
 
 void ThisNode::streamTo(SourceStream &s) const { s << "this"; }