getInstalledRelatedApps: Change internal url fields to URL type.

third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp

 // Copyright 2016 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/installedapp/InstalledAppController.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/LocalFrame.h"
 #include "platform/RuntimeEnabledFeatures.h"
+#include "platform/weborigin/KURL.h"
 #include "public/platform/InterfaceProvider.h"
 #include "wtf/Functional.h"
 
 #include <utility>
 
 namespace blink {
 
 // Callbacks for the result of
 // WebRelatedAppsFetcher::getManifestRelatedApplications. Calls
 // filterByInstalledApps upon receiving the list of related applications.
@@ skipping 76 matching lines @@
 void InstalledAppController::filterByInstalledApps(
     const blink::WebVector<blink::WebRelatedApplication>& relatedApps,
     std::unique_ptr<blink::AppInstalledCallbacks> callbacks) {
   WTF::Vector<mojom::blink::RelatedApplicationPtr> mojoRelatedApps;
   for (const auto& relatedApplication : relatedApps) {
     mojom::blink::RelatedApplicationPtr convertedApplication(
         mojom::blink::RelatedApplication::New());
     DCHECK(!relatedApplication.platform.isEmpty());
     convertedApplication->platform = relatedApplication.platform;
     convertedApplication->id = relatedApplication.id;
-    convertedApplication->url = relatedApplication.url;
+    if (!relatedApplication.url.isNull())
+      convertedApplication->url = KURL(relatedApplication.url);
     mojoRelatedApps.push_back(std::move(convertedApplication));
   }
 
   if (!m_provider) {
     supplementable()->interfaceProvider()->getInterface(
         mojo::MakeRequest(&m_provider));
     // TODO(mgiuca): Set a connection error handler. This requires a refactor to
     // work like NavigatorShare.cpp (retain a persistent list of clients to
     // reject all of their promises).
     DCHECK(m_provider);
   }
 
   m_provider->FilterInstalledApps(
       std::move(mojoRelatedApps),
       convertToBaseCallback(
           WTF::bind(&InstalledAppController::OnFilterInstalledApps,
                     wrapPersistent(this), WTF::passed(std::move(callbacks)))));
 }
 
 void InstalledAppController::OnFilterInstalledApps(
     std::unique_ptr<blink::AppInstalledCallbacks> callbacks,
     WTF::Vector<mojom::blink::RelatedApplicationPtr> result) {
   std::vector<blink::WebRelatedApplication> applications;
   for (const auto& res : result) {
     blink::WebRelatedApplication app;
     app.platform = res->platform;
-    app.url = res->url;
+    if (res->url)
+      app.url = *res->url;
     app.id = res->id;
     applications.push_back(app);
   }
   callbacks->onSuccess(
       blink::WebVector<blink::WebRelatedApplication>(applications));
 }
 
 DEFINE_TRACE(InstalledAppController) {
   Supplement<LocalFrame>::trace(visitor);
   ContextLifecycleObserver::trace(visitor);
 }
 
 }  // namespace blink