Implement URLSearchParams.

Source/core/dom/DOMURLUtils.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  * Copyright (C) 2012 Motorola Mobility Inc.
  *
  * 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 13 matching lines @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/dom/DOMURLUtils.h"
 
 #include "platform/weborigin/KnownPorts.h"
 
 namespace WebCore {
 
+DOMURLUtils::~DOMURLUtils()
+{
+    // Safe, but not ideal. See setSearchParams() FIXME.
+    if (m_searchParams.get())

[Inactive, 2014/01/20 22:06:12]

FYI, you don't need this '.get()' in your conditionals (same below).

[sof, 2014/01/21 13:20:11]

Yes, thanks - tidied now.

+        m_searchParams->setURLObject(0);
+}
+
+void DOMURLUtils::update(DOMURLUtils* impl)
+{
+    KURL url = impl->url();
+    setSearchParams(impl, url.query());
+}
+
 void DOMURLUtils::setHref(DOMURLUtils* impl, const String& value)
 {
     impl->setInput(value);
 }
 
 void DOMURLUtils::setProtocol(DOMURLUtils* impl, const String& value)
 {
     KURL url = impl->url();
     if (url.isNull())
         return;
@@ skipping 71 matching lines @@
     url.setPath(value);
     impl->setURL(url);
 }
 
 void DOMURLUtils::setSearch(DOMURLUtils* impl, const String& value)
 {
     KURL url = impl->url();
     if (!url.isValid())
         return;
     url.setQuery(value);
+
+    if (!value.isEmpty() && value[0] == '?')
+        setSearchParams(impl, value.substring(1));
+    else
+        setSearchParams(impl, value);
+
     impl->setURL(url);
 }
 
 void DOMURLUtils::setHash(DOMURLUtils* impl, const String& value)
 {
     KURL url = impl->url();
     if (url.isNull())
         return;
 
     if (value[0] == '#')
         url.setFragmentIdentifier(value.substring(1));
     else
         url.setFragmentIdentifier(value);
 
     impl->setURL(url);
 }
 
+PassRefPtr<DOMURLSearchParams> DOMURLUtils::searchParams(DOMURLUtils* impl, bool& isNull)
+{
+    if (!impl->m_searchParams.get())
+        setSearchParams(impl, impl->url().query());
+    isNull = false;
+    return impl->m_searchParams.get();
+}
+
+void DOMURLUtils::setSearchParams(DOMURLUtils* impl, const String& queryString)
+{
+    if (!impl->m_searchParams.get()) {
+        impl->m_searchParams = DOMURLSearchParams::create(queryString);
+        // See FIXME below.
+        impl->m_searchParams->setURLObject(impl);
+    } else {
+        ASSERT(impl->m_searchParams->getURLObject() == impl);
+        impl->m_searchParams->setInput(queryString);
+    }
+}
+
+void DOMURLUtils::setSearchParams(DOMURLUtils* impl, PassRefPtr<DOMURLSearchParams> searchParams)
+{
+    // The spec tells us to ignore the setting if it is null..
+    if (searchParams.get()) {
+        if (searchParams->getURLObject())
+            impl->m_searchParams = DOMURLSearchParams::create(searchParams.get());
+        else
+            impl->m_searchParams = searchParams;
+
+        // FIXME: setting the URL object on the searchParams object
+        // really should create a strong reference back to the
+        // URLUtils object.
+        //
+        // This is needed when the URLUtils object is no longer
+        // otherwise referenced, but the searchParams object is. In
+        // that case, the updating of the searchParams should still
+        // have an effect on the URLUtils' underlying URL.
+        //
+        // A ref-count cycle is introduced if we do this, however.
+        //
+        // Notice that this will only happen if the searchParams
+        // object doesn't have a pre-existing 'url object.' i.e., the
+        // complexity here is due to the fact that URLSearchParams
+        // object can be separately constructed, and then assigned a
+        // URLUtils via its searchParams setter.
+        //
+        // How to appropriately handle the mutual dependency? Do not
+        // have a solution yet, so the URLUtils object simply removes
+        // itself as the 'url object' of its searchParams object when
+        // it goes away.
+        ASSERT(!impl->m_searchParams->getURLObject());
+        impl->m_searchParams->setURLObject(impl);
+    }
+}
+
 } // namespace WebCore