[PresentationAPI] Use KURL and WebURL in place of string types.

third_party/WebKit/Source/modules/presentation/PresentationConnection.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "modules/presentation/PresentationConnection.h"
 
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "core/dom/DOMArrayBuffer.h"
 #include "core/dom/DOMArrayBufferView.h"
 #include "core/dom/Document.h"
@@ skipping 139 matching lines @@
     DEFINE_INLINE_TRACE()
     {
         visitor->trace(m_PresentationConnection);
     }
 
 private:
     Member<PresentationConnection> m_PresentationConnection;
     std::unique_ptr<FileReaderLoader> m_loader;
 };
 
-PresentationConnection::PresentationConnection(LocalFrame* frame, const String& id, const String& url)
+PresentationConnection::PresentationConnection(LocalFrame* frame, const String& id, const KURL& url)
     : DOMWindowProperty(frame)
     , m_id(id)
     , m_url(url)
     , m_state(WebPresentationConnectionState::Connected)
     , m_binaryType(BinaryTypeBlob)
 {
 }
 
 PresentationConnection::~PresentationConnection()
 {
@@ skipping 228 matching lines @@
         return;
     WebPresentationClient* client = presentationClient(getExecutionContext());
     if (client)
         client->terminateSession(m_url, m_id);
 
     tearDown();
 }
 
 bool PresentationConnection::matches(WebPresentationConnectionClient* client) const
 {
-    return client && m_url == static_cast<String>(client->getUrl()) && m_id == static_cast<String>(client->getId());
+    return client && m_url == KURL(client->getUrl()) && m_id == static_cast<String>(client->getId());

[dcheng, 2016/09/14 20:07:27]

Can we tag WebPresentationConnectionClient::getUrl with a TODO to use WebURL as
well?

 }
 
 void PresentationConnection::didChangeState(WebPresentationConnectionState state)
 {
     if (m_state == state)
         return;
 
     m_state = state;
     switch (m_state) {
     case WebPresentationConnectionState::Connected:
@@ skipping 46 matching lines @@
 {
     // Cancel current Blob loading if any.
     if (m_blobLoader) {
         m_blobLoader->cancel();
         m_blobLoader.clear();
     }
     m_messages.clear();
 }
 
 } // namespace blink