Title: [120936] trunk/Source/WebCore
Revision
120936
Author
commit-qu...@webkit.org
Date
2012-06-21 10:21:44 -0700 (Thu, 21 Jun 2012)

Log Message

Remove unused static function skipComment.
https://bugs.webkit.org/show_bug.cgi?id=89641

Patch by Kwang Yul Seo <sk...@company100.net> on 2012-06-21
Reviewed by Alexey Proskuryakov.

No behavioral changes.

* loader/TextResourceDecoder.cpp:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (120935 => 120936)


--- trunk/Source/WebCore/ChangeLog	2012-06-21 17:19:15 UTC (rev 120935)
+++ trunk/Source/WebCore/ChangeLog	2012-06-21 17:21:44 UTC (rev 120936)
@@ -1,3 +1,14 @@
+2012-06-21  Kwang Yul Seo  <sk...@company100.net>
+
+        Remove unused static function skipComment.
+        https://bugs.webkit.org/show_bug.cgi?id=89641
+
+        Reviewed by Alexey Proskuryakov.
+
+        No behavioral changes.
+
+        * loader/TextResourceDecoder.cpp:
+
 2012-06-21  Nico Weber  <tha...@chromium.org>
 
         Remove two more member variables found by clang's Wunused-private-field

Modified: trunk/Source/WebCore/loader/TextResourceDecoder.cpp (120935 => 120936)


--- trunk/Source/WebCore/loader/TextResourceDecoder.cpp	2012-06-21 17:19:15 UTC (rev 120935)
+++ trunk/Source/WebCore/loader/TextResourceDecoder.cpp	2012-06-21 17:21:44 UTC (rev 120936)
@@ -502,38 +502,6 @@
     return true;
 }
 
-// Other browsers allow comments in the head section, so we need to also.
-// It's important not to look for tags inside the comments.
-static inline void skipComment(const char*& ptr, const char* pEnd)
-{
-    const char* p = ptr;
-    if (p == pEnd)
-        return;
-
-    // Allow <!-->; other browsers do.
-    if (*p == '>') {
-        ptr = p + 1;
-        return;
-    }
-
-    while (p + 2 < pEnd) {
-        if (*p == '-') {
-            // This is the real end of comment, "-->".
-            if (bytesEqual(p + 1, '-', '>')) {
-                p += 3;
-                break;
-            }
-            // This is the incorrect end of comment that other browsers allow, "--!>".
-            if (p + 3 < pEnd && bytesEqual(p + 1, '-', '!', '>')) {
-                p += 4;
-                break;
-            }
-        }
-        p++;
-    }
-    ptr = p;
-}
-
 bool TextResourceDecoder::checkForHeadCharset(const char* data, size_t len, bool& movedDataToBuffer)
 {
     if (m_source != DefaultEncoding && m_source != EncodingFromParentFrame) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to