Remove stale comments for blink::decodeURLEscapeSequences()

third_party/WebKit/Source/platform/weborigin/KURL.cpp

 /*
  * Copyright (C) 2004, 2007, 2008, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2012 Research In Motion Limited. All rights reserved.
  * Copyright (C) 2008, 2009, 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
@@ skipping 637 matching lines @@
     url::Replacements<char> replacements;
     replacements.SetPath(charactersOrEmpty(pathUTF8), url::Component(0, pathUTF8.length()));
     replaceComponents(replacements);
 }
 
 String decodeURLEscapeSequences(const String& string)
 {
     return decodeURLEscapeSequences(string, UTF8Encoding());
 }
 
-// In KURL.cpp's implementation, this is called by every component getter.
-// It will unescape every character, including '\0'. This is scary, and may
-// cause security holes. We never call this function for components, and
-// just return the ASCII versions instead.
-//
-// This function is also used to decode javascript: URLs and as a general
-// purpose unescaping function.
-//
-// FIXME These should be merged to the KURL.cpp implementation.
 String decodeURLEscapeSequences(const String& string, const WTF::TextEncoding& encoding)
 {
-    // FIXME We can probably use KURL.cpp's version of this function
-    // without modification. However, I'm concerned about
-    // https://bugs.webkit.org/show_bug.cgi?id=20559 so am keeping this old
-    // custom code for now. Using their version will also fix the bug that
-    // we ignore the encoding.
-    //
-    // FIXME b/1350291: This does not get called very often. We just convert
-    // first to 8-bit UTF-8, then unescape, then back to 16-bit. This kind of
-    // sucks, and we don't use the encoding properly, which will make some
-    // obscure anchor navigations fail.
     StringUTF8Adaptor stringUTF8(string);
     url::RawCanonOutputT<base::char16> unescaped;
     url::DecodeURLEscapeSequences(stringUTF8.data(), stringUTF8.length(), &unescaped);
     return StringImpl::create8BitIfPossible(reinterpret_cast<UChar*>(unescaped.data()), unescaped.length());
 }
 
 String encodeWithURLEscapeSequences(const String& notEncodedString)
 {
     CString utf8 = UTF8Encoding().encode(notEncodedString, WTF::URLEncodedEntitiesForUnencodables);
 
@@ skipping 226 matching lines @@
     m_string = AtomicString::fromUTF8(output.data(), output.length());
 }
 
 bool KURL::isSafeToSendToAnotherThread() const
 {
     return m_string.isSafeToSendToAnotherThread()
         && (!m_innerURL || m_innerURL->isSafeToSendToAnotherThread());
 }
 
 } // namespace blink