Clean up URL.h
authorachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 Nov 2017 21:38:11 +0000 (21:38 +0000)
committerachristensen@apple.com <achristensen@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 Nov 2017 21:38:11 +0000 (21:38 +0000)
https://bugs.webkit.org/show_bug.cgi?id=179710

Reviewed by Brady Eidson.

No change in behavior.

* Modules/fetch/FetchRequest.cpp:
(WebCore::FetchRequest::urlString const):
* Modules/fetch/FetchResponse.cpp:
(WebCore::FetchResponse::url const):
* platform/URL.cpp:
(WebCore::URL::serialize const): Deleted.
* platform/URL.h:
* platform/mac/URLMac.mm:
(WebCore::URL::createCFURL const):

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

Source/WebCore/ChangeLog
Source/WebCore/Modules/fetch/FetchRequest.cpp
Source/WebCore/Modules/fetch/FetchResponse.cpp
Source/WebCore/platform/URL.cpp
Source/WebCore/platform/URL.h
Source/WebCore/platform/mac/URLMac.mm

index 4442bbe..c3c775d 100644 (file)
@@ -1,5 +1,24 @@
 2017-11-17  Alex Christensen  <achristensen@webkit.org>
 
+        Clean up URL.h
+        https://bugs.webkit.org/show_bug.cgi?id=179710
+
+        Reviewed by Brady Eidson.
+
+        No change in behavior.
+
+        * Modules/fetch/FetchRequest.cpp:
+        (WebCore::FetchRequest::urlString const):
+        * Modules/fetch/FetchResponse.cpp:
+        (WebCore::FetchResponse::url const):
+        * platform/URL.cpp:
+        (WebCore::URL::serialize const): Deleted.
+        * platform/URL.h:
+        * platform/mac/URLMac.mm:
+        (WebCore::URL::createCFURL const):
+
+2017-11-17  Alex Christensen  <achristensen@webkit.org>
+
         Use RunLoop and Mode from NetworkingContext if they are given
         https://bugs.webkit.org/show_bug.cgi?id=179800
         <rdar://problem/35519421>
index 0e8742a..f2b2b29 100644 (file)
@@ -263,7 +263,7 @@ String FetchRequest::referrer() const
 const String& FetchRequest::urlString() const
 {
     if (m_requestURL.isNull())
-        m_requestURL = m_request.url().serialize();
+        m_requestURL = m_request.url();
     return m_requestURL;
 }
 
index 2af8732..b8023aa 100644 (file)
@@ -186,8 +186,11 @@ void FetchResponse::fetch(ScriptExecutionContext& context, FetchRequest& request
 
 const String& FetchResponse::url() const
 {
-    if (m_responseURL.isNull())
-        m_responseURL = m_response.url().serialize(true);
+    if (m_responseURL.isNull()) {
+        URL url = m_response.url();
+        url.removeFragmentIdentifier();
+        m_responseURL = url.string();
+    }
     return m_responseURL;
 }
 
index b915e3f..2ed44b0 100644 (file)
@@ -686,13 +686,6 @@ String decodeURLEscapeSequences(const String& string, const TextEncoding& encodi
     return decodeEscapeSequences<URLEscapeSequence>(string, encoding);
 }
 
-String URL::serialize(bool omitFragment) const
-{
-    if (omitFragment)
-        return m_string.left(m_queryEnd);
-    return m_string;
-}
-
 #if PLATFORM(IOS)
 
 static bool shouldCanonicalizeScheme = true;
index 7766629..ddd97e9 100644 (file)
@@ -207,8 +207,6 @@ public:
     template <class Decoder> static bool decode(Decoder&, URL&);
     template <class Decoder> static std::optional<URL> decode(Decoder&);
 
-    String serialize(bool omitFragment = false) const;
-
 private:
     friend class URLParser;
     WEBCORE_EXPORT void invalidate();
@@ -216,12 +214,6 @@ private:
     void init(const URL&, const String&, const TextEncoding&);
     void copyToBuffer(Vector<char, 512>& buffer) const;
 
-    // Parses the given URL. The originalString parameter allows for an
-    // optimization: When the source is the same as the fixed-up string,
-    // it will use the passed-in string instead of allocating a new one.
-    void parse(const String&);
-    void parse(const char* url, const String* originalString = 0);
-
     bool hasPath() const;
 
     String m_string;
@@ -464,10 +456,10 @@ WTF::TextStream& operator<<(WTF::TextStream&, const URL&);
 
 namespace WTF {
 
-    // URLHash is the default hash for String
-    template<typename T> struct DefaultHash;
-    template<> struct DefaultHash<WebCore::URL> {
-        typedef WebCore::URLHash Hash;
-    };
+// URLHash is the default hash for String
+template<typename T> struct DefaultHash;
+template<> struct DefaultHash<WebCore::URL> {
+    typedef WebCore::URLHash Hash;
+};
 
 } // namespace WTF
index c908c63..7299cb0 100644 (file)
@@ -57,7 +57,7 @@ URL::operator NSURL *() const
 RetainPtr<CFURLRef> URL::createCFURL() const
 {
     if (isNull())
-        return 0;
+        return nullptr;
 
     if (isEmpty()) {
         // We use the toll-free bridge between NSURL and CFURL to create a CFURLRef supporting both empty and null values.
@@ -74,6 +74,4 @@ RetainPtr<CFURLRef> URL::createCFURL() const
     return createCFURLFromBuffer(buffer.data(), buffer.size());
 }
 
-
-
 }