Use the URL passed to navigator.share().

third_party/WebKit/Source/modules/webshare/NavigatorShare.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/webshare/NavigatorShare.h"
 
 #include "core/dom/DOMException.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/frame/LocalFrame.h"
@@ skipping 86 matching lines @@
         DCHECK(frame);
         frame->interfaceProvider()->getInterface(mojo::GetProxy(&m_service));
         DCHECK(m_service);
     }
 
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ShareClientImpl* client = new ShareClientImpl(this, resolver);
     m_clients.add(client);
     ScriptPromise promise = resolver->promise();
 
-    // TODO(sammc): Use shareData.url().
-    m_service->Share(shareData.hasTitle() ? shareData.title() : emptyString(), shareData.hasText() ? shareData.text() : emptyString(), convertToBaseCallback(WTF::bind(&ShareClientImpl::callback, wrapPersistent(client))));
+    m_service->Share(
+        shareData.hasTitle() ? shareData.title() : emptyString(),
+        shareData.hasText() ? shareData.text() : emptyString(),
+        shareData.hasURL() ? KURL(scriptState->getExecutionContext()->url(), shareData.url()) : KURL(),

[dcheng, 2016/09/08 03:17:37]

Should this be using doc->completeURL(shareData.url())?

[Sam McNally, 2016/09/08 03:32:05]

Done.

+        convertToBaseCallback(WTF::bind(&ShareClientImpl::callback, wrapPersistent(client))));
 
     return promise;
 }
 
 ScriptPromise NavigatorShare::share(ScriptState* scriptState, Navigator& navigator, const ShareData& shareData)
 {
     return from(navigator).share(scriptState, shareData);
 }
 
 } // namespace blink