Force XML comments to maintain whitespace
authortony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Aug 2012 01:26:49 +0000 (01:26 +0000)
committertony@chromium.org <tony@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Aug 2012 01:26:49 +0000 (01:26 +0000)
https://bugs.webkit.org/show_bug.cgi?id=94620

Reviewed by Pavel Feldman.

Source/WebCore:

Fix a typo in XMLViewer.css for white-space formatting. This seems
like the desired behavior so you can see, e.g., code formatted in an
XML comment.

Covered by existing xmlviewer tests in http/tests/xmlviewer/dumpAsText

* xml/XMLViewer.css:
(.comment): Fix typo.
* xml/XMLViewer.js:
(createComment): Add style to comment nodes.

LayoutTests:

Update comments to include whitespace formatting.

* http/tests/xmlviewer/dumpAsText/xlink-expected.txt:
* http/tests/xmlviewer/dumpAsText/xmlviewer-expected.txt:
* http/tests/xmlviewer/dumpAsText/xul-expected.txt:
* platform/chromium/http/tests/xmlviewer/dumpAsText/mathml-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@126241 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/http/tests/xmlviewer/dumpAsText/xlink-expected.txt
LayoutTests/http/tests/xmlviewer/dumpAsText/xmlviewer-expected.txt
LayoutTests/http/tests/xmlviewer/dumpAsText/xul-expected.txt
LayoutTests/platform/chromium/http/tests/xmlviewer/dumpAsText/mathml-expected.txt
LayoutTests/svg/hixie/error/dumpAsText/004-expected.txt
LayoutTests/svg/hixie/error/dumpAsText/005-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/xml/XMLViewer.css
Source/WebCore/xml/XMLViewer.js

index 593fa82..db4fe1a 100644 (file)
@@ -1,3 +1,17 @@
+2012-08-21  Tony Chang  <tony@chromium.org>
+
+        Force XML comments to maintain whitespace
+        https://bugs.webkit.org/show_bug.cgi?id=94620
+
+        Reviewed by Pavel Feldman.
+
+        Update comments to include whitespace formatting.
+
+        * http/tests/xmlviewer/dumpAsText/xlink-expected.txt:
+        * http/tests/xmlviewer/dumpAsText/xmlviewer-expected.txt:
+        * http/tests/xmlviewer/dumpAsText/xul-expected.txt:
+        * platform/chromium/http/tests/xmlviewer/dumpAsText/mathml-expected.txt:
+
 2012-08-21  David Grogan  <dgrogan@chromium.org>
 
         IndexedDB: Fire error at request when abort is called in upgradeneeded
index 3593698..0ca26c5 100644 (file)
@@ -1,5 +1,5 @@
 This XML file does not appear to have any style information associated with it. The document tree is shown below.
-<!-- If you can see this, this test has failed. -->
+<!--  If you can see this, this test has failed.  -->
 <test xmlns="http://still-unknown.example.org/">
 <header>
 This tests that xml viewer is not used when there is an attribute XLink namespace.
index 3705c2f..dfe510b 100644 (file)
@@ -1,7 +1,13 @@
 This XML file does not appear to have any style information associated with it. The document tree is shown below.
 <?some-short-processing-instruction short-params?>
 <?some-long-processing-instruction
-The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog.
+The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy
+                dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps
+                over the lazy dog. The quick brown fox jumps over the lazy dog.  The quick brown fox jumps over the lazy dog. The quick brown fox
+                jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick
+                brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog.
+                The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy
+                dog. The quick brown fox jumps over the lazy dog.
 ?>
 <root-tag-with-mixed-content>
 <empty-tag/>
@@ -17,10 +23,19 @@ The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the
 The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog.
 </tag-with-long-text-and-attrs>
 And now some text in root element. And now some other text in root element.
-<!-- some short comment -->
+<!--  some short comment  -->
 And now some long text in root element. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog.
 <!--
-some long comment The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog.
+ some long comment
+                 The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the
+                 lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox
+                 jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The
+                 quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the
+                 lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox
+                 jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The quick brown fox jumps over the lazy dog. The
+                 quick brown fox jumps over the lazy dog. The quick brown fox
+                 jumps over the lazy dog.
+    
 -->
 <tag-with-short-cdata>
 <![CDATA[ some short text ]]>
index 751d260..871d0ba 100644 (file)
@@ -1,5 +1,5 @@
 This XML file does not appear to have any style information associated with it. The document tree is shown below.
-<!-- If you can see this, this test has failed. -->
+<!--  If you can see this, this test has failed.  -->
 <FAIL xmlns:a="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
 <header>
 This tests that xml viewer is not used when there is an attribute in XUL namespace.
index 8628b1e..a33ef7d 100644 (file)
@@ -1,6 +1,7 @@
 This XML file does not appear to have any style information associated with it. The document tree is shown below.
 <!--
-If you can see this, this test has failed, except if you are not using a MathML-aware client.
+ If you can see this, this test has failed, except if you are
+     not using a MathML-aware client. 
 -->
 <FAIL xmlns:a="http://www.w3.org/1998/Math/MathML">
 <header>
index 86967b4..cf5d886 100644 (file)
@@ -1,9 +1,15 @@
 This XML file does not appear to have any style information associated with it. The document tree is shown below.
 <svg width="300" height="200">
-<!-- note: no namespace, but sent as image/svg+xml -->
+<!--  note: no namespace, but sent as image/svg+xml  -->
 <desc>There should be no red on this page.</desc>
 <rect x="10" y="10" width="280" height="180" fill="red"/>
 <!--
-This document is either not SVG, or is SVG but is in error. Section 5.1.1 of SVG 1.1 says: # In all cases, for compliance with the "Namespaces in XML" # Recommendation [XML-NS], an SVG namespace declaration must be # provided so that all SVG elements are identified as belonging # to the SVG namespace.
+ This document is either not SVG, or is SVG but is in error.
+      Section 5.1.1 of SVG 1.1 says:
+      # In all cases, for compliance with the "Namespaces in XML"
+      # Recommendation [XML-NS], an SVG namespace declaration must be
+      # provided so that all SVG elements are identified as belonging
+      # to the SVG namespace.
+  
 -->
 </svg>
index ea1d902..8a91ecf 100644 (file)
@@ -1,6 +1,6 @@
 This XML file does not appear to have any style information associated with it. The document tree is shown below.
 <svg xmlns="http://www.example.org/" width="300" height="200">
-<!-- wrong namespace, but sent as image/svg+xml -->
+<!--  wrong namespace, but sent as image/svg+xml  -->
 <desc>There should be no red on this page.</desc>
 <rect x="10" y="10" width="280" height="180" fill="red"/>
 </svg>
index 9f7affc..49ae972 100644 (file)
@@ -1,3 +1,21 @@
+2012-08-21  Tony Chang  <tony@chromium.org>
+
+        Force XML comments to maintain whitespace
+        https://bugs.webkit.org/show_bug.cgi?id=94620
+
+        Reviewed by Pavel Feldman.
+
+        Fix a typo in XMLViewer.css for white-space formatting. This seems
+        like the desired behavior so you can see, e.g., code formatted in an
+        XML comment.
+
+        Covered by existing xmlviewer tests in http/tests/xmlviewer/dumpAsText
+
+        * xml/XMLViewer.css:
+        (.comment): Fix typo.
+        * xml/XMLViewer.js:
+        (createComment): Add style to comment nodes.
+
 2012-08-21  Sheriff Bot  <webkit.review.bot@gmail.com>
 
         Unreviewed, rolling out r126202.
index 2a1d5fd..f96b0b1 100644 (file)
@@ -51,7 +51,7 @@ div.collapsible > div.hidden {
     margin-left: 1em;
 }
 .comment {
-    whitespace: pre;
+    white-space: pre;
 }
 
 .button {
index 22405e0..8615984 100644 (file)
@@ -289,6 +289,7 @@ function createExpandButton(str)
 function createComment(commentString)
 {
     var comment = createHTMLElement('span');
+    comment.classList.add('comment');
     comment.classList.add('webkit-html-comment');
     comment.textContent = commentString;
     return comment;