Add support for URL.searchParams getter.

third_party/WebKit/Source/core/dom/DOMURL.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 12 matching lines @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/dom/DOMURL.h"
 
 #include "bindings/core/v8/ExceptionMessages.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/dom/ExecutionContext.h"
+#include "core/dom/URLSearchParams.h"
 #include "core/fetch/MemoryCache.h"
 #include "core/fileapi/Blob.h"
 #include "core/html/PublicURLManager.h"
 #include "platform/blob/BlobURL.h"
 #include "platform/weborigin/SecurityOrigin.h"
 
 namespace blink {
 
 DOMURL::DOMURL(const String& url, const KURL& base, ExceptionState& exceptionState)
 {
-    if (!base.isValid())
+    if (!base.isValid()) {
         exceptionState.throwTypeError("Invalid base URL");
+        return;
+    }

[Mike West, 2016/04/05 12:43:51]

Whoops! Nice catch. Is there a test for this?

[sof, 2016/04/05 15:23:18]

The final test in url-constructor.html

 
     m_url = KURL(base, url);
     if (!m_url.isValid())
         exceptionState.throwTypeError("Invalid URL");
 }
 
+DEFINE_TRACE(DOMURL)
+{
+    visitor->trace(m_searchParams);
+}
+
 void DOMURL::setInput(const String& value)
 {
     KURL url(blankURL(), value);
     if (url.isValid()) {
         m_url = url;
         m_input = String();
     } else {
         m_url = KURL();
         m_input = value;
     }
+    update();
+}
+
+void DOMURL::setSearch(const String& value)
+{
+    DOMURLUtils::setSearch(value);
+    if (!value.isEmpty() && value[0] == '?')

[Mike West, 2016/04/05 12:43:51]

Nit: It might make sense to move this check into `updateSearchParams`.

[sof, 2016/04/05 15:23:18]

That won't quite work if you set url.href with something like
"http://example.org/??a=b"; it then chopping off the leading "?".

Added a test in url-searchparams.html to verify this.

+        updateSearchParams(value.substring(1));
+    else
+        updateSearchParams(value);
 }
 
 String DOMURL::createObjectURL(ExecutionContext* executionContext, Blob* blob, ExceptionState& exceptionState)
 {
     ASSERT(blob);
     if (!executionContext)
         return String();
     if (blob->hasBeenClosed()) {
         exceptionState.throwDOMException(InvalidStateError, String(blob->isFile() ? "File" : "Blob") + " has been closed.");
         return String();
@@ skipping 23 matching lines @@
 }
 
 void DOMURL::revokeObjectUUID(ExecutionContext* executionContext, const String& uuid)
 {
     if (!executionContext)
         return;
 
     executionContext->publicURLManager().revoke(uuid);
 }
 
+URLSearchParams* DOMURL::searchParams()
+{
+    if (!m_searchParams)
+        m_searchParams = URLSearchParams::create(url().query(), this);
+
+    return m_searchParams;
+}
+
+void DOMURL::update()
+{
+    updateSearchParams(url().query());
+}
+
+void DOMURL::updateSearchParams(const String& queryString)
+{
+    if (!m_searchParams)
+        return;
+
+    ASSERT(m_searchParams->urlObject() == this);
+    m_searchParams->setInput(queryString);
+}
+
 } // namespace blink