<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Expand URL class query parameter functions"
   href="https://bugs.webkit.org/show_bug.cgi?id=228122#c20">Comment # 20</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Expand URL class query parameter functions"
   href="https://bugs.webkit.org/show_bug.cgi?id=228122">bug 228122</a>
              from <span class="vcard"><a class="email" href="mailto:risul_islam@apple.com" title="Risul Islam <risul_islam@apple.com>"> <span class="fn">Risul Islam</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=434669&action=diff" name="attach_434669" title="Patch">attachment 434669</a> <a href="attachment.cgi?id=434669&action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=434669&action=review">https://bugs.webkit.org/attachment.cgi?id=434669&action=review</a>

<span class="quote">>>>> Source/WTF/ChangeLog:8
>>>> +        Added more functionalities for parsing URL query parameters.
>>> 
>>> I don’t fully understand why adding more functions that are not yet used is important for WebKIt, but I trust there is some reason.
>> 
>> Thank you Darin for your valuable comments and insights. We are trying to come up with a feature that restricts tracking parameters from the URL while loading or sharing by copying. Thats why we are adding the functionalities in this patch.

> I'd say "Added parsing of URL query strings."</span >

Great. On it.

<span class="quote">>> Tools/TestWebKitAPI/Tests/WTF/URL.cpp:522
>> +    EXPECT_EQ(url9.string(), url12.string());

> You need to have a section of negative tests, for instance named URLMalformedQueryStrings. There your test things like:
> · <a href="http://www.webkit.org/">http://www.webkit.org/</a>??
> · <a href="http://www.webkit.org/?/?test=test">http://www.webkit.org/?/?test=test</a>
> · <a href="http://www.webkit.org/?=test">http://www.webkit.org/?=test</a>
> · <a href="http://www.webkit.org/">http://www.webkit.org/</a>?==
> · <a href="http://www.webkit.org/">http://www.webkit.org/</a>?=?
> · <a href="http://www.webkit.org/">http://www.webkit.org/</a>=?
> · <a href="http://www.webkit.org">http://www.webkit.org</a>??
> · <a href="http://www.webkit.org?/?test=test">http://www.webkit.org?/?test=test</a>
> · <a href="http://www.webkit.org?=test">http://www.webkit.org?=test</a>
> · <a href="http://www.webkit.org">http://www.webkit.org</a>?==
> · <a href="http://www.webkit.org">http://www.webkit.org</a>?=?
> · <a href="http://www.webkit.org">http://www.webkit.org</a>=?</span >

We assumed that the first '?' is the start of the query string. Then the query parameters are separated by '&' and finally parameter key and value is separated by '=' . Since there is no standard we got this assumption from
 <a href="https://www.freeformatter.com/url-parser-query-string-splitter.html">https://www.freeformatter.com/url-parser-query-string-splitter.html</a> 

There are some test cases added for the above kinds of URLs. Do we want more tests?</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>