Title: [121564] trunk/Source/WebCore
Revision
121564
Author
[email protected]
Date
2012-06-29 08:27:45 -0700 (Fri, 29 Jun 2012)

Log Message

Update FIXME comment in XMLDocumentParser::wellFormed
https://bugs.webkit.org/show_bug.cgi?id=90223

Patch by Kwang Yul Seo <[email protected]> on 2012-06-29
Reviewed by Adam Barth.

XMLDocumentParser::wellFormed is still used by the XMLHttpRequest to check if the responseXML was well formed.
So it can't be removed.

* xml/parser/XMLDocumentParser.h:
(XMLDocumentParser):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (121563 => 121564)


--- trunk/Source/WebCore/ChangeLog	2012-06-29 15:23:40 UTC (rev 121563)
+++ trunk/Source/WebCore/ChangeLog	2012-06-29 15:27:45 UTC (rev 121564)
@@ -1,3 +1,16 @@
+2012-06-29  Kwang Yul Seo  <[email protected]>
+
+        Update FIXME comment in XMLDocumentParser::wellFormed
+        https://bugs.webkit.org/show_bug.cgi?id=90223
+
+        Reviewed by Adam Barth.
+
+        XMLDocumentParser::wellFormed is still used by the XMLHttpRequest to check if the responseXML was well formed.
+        So it can't be removed.
+
+        * xml/parser/XMLDocumentParser.h:
+        (XMLDocumentParser):
+
 2012-06-29  Andreas Kling  <[email protected]>
 
         Separate mutating CSSStyleDeclaration operations.

Modified: trunk/Source/WebCore/xml/parser/XMLDocumentParser.h (121563 => 121564)


--- trunk/Source/WebCore/xml/parser/XMLDocumentParser.h	2012-06-29 15:23:40 UTC (rev 121563)
+++ trunk/Source/WebCore/xml/parser/XMLDocumentParser.h	2012-06-29 15:27:45 UTC (rev 121564)
@@ -94,7 +94,7 @@
 
         static bool parseDocumentFragment(const String&, DocumentFragment*, Element* parent = 0, FragmentScriptingPermission = AllowScriptingContent);
 
-        // FIXME: This function used to be used by WML. Can we remove it?
+        // Used by the XMLHttpRequest to check if the responseXML was well formed.
         virtual bool wellFormed() const { return !m_sawError; }
 
         TextPosition textPosition() const;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to