Picker takes a vector of pairs, and passes back the user chosen target.

chrome/browser/webshare/share_service_impl.cc

Index: chrome/browser/webshare/share_service_impl.cc
diff --git a/chrome/browser/webshare/share_service_impl.cc b/chrome/browser/webshare/share_service_impl.cc
index 051bb65c364dad989fdb1038e1723e73991d76d0..bf9f4ebd3570916da535c721dda6a28ba148f98b 100644
--- a/chrome/browser/webshare/share_service_impl.cc
+++ b/chrome/browser/webshare/share_service_impl.cc
@@ -115,13 +115,13 @@ bool ShareServiceImpl::ReplacePlaceholders(base::StringPiece url_template,
 
 void ShareServiceImpl::ShowPickerDialog(
     const std::vector<base::string16>& targets,
-    const base::Callback<void(SharePickerResult)>& callback) {
+    const base::Callback<void(base::Optional<base::string16>)>& callback) {
   // TODO(mgiuca): Get the browser window as |parent_window|.
 #if defined(OS_LINUX) || defined(OS_WIN)
   chrome::ShowWebShareTargetPickerDialog(nullptr /* parent_window */, targets,
                                          callback);
 #else
-  callback.Run(SharePickerResult::CANCEL);
+  callback.Run(base::nullopt);
 #endif
 }
 
@@ -138,9 +138,13 @@ void ShareServiceImpl::Share(const std::string& title,
                              const std::string& text,
                              const GURL& share_url,
                              const ShareCallback& callback) {
-  // TODO(constantina): Replace hard-coded name with the registered target list.
-  constexpr char kTargetName[] = "Web Share Target Test App";
-  std::vector<base::string16> targets{base::ASCIIToUTF16(kTargetName)};
+  // TODO(constantina): Replace hard-coded manifest URL with the list of
+  // registered targets' manifest URLs.
+  constexpr char kManifestURL[] =
+      "https://wicg.github.io/web-share-target/demos/manifest.json";
+  // TODO(constantina): Pass vector of pairs of target names and manifest URls

[Matt Giuca, 2017/02/02 00:07:40]

nit: "URL" not "URl".

[constantina, 2017/02/02 00:54:17]

Done.

+  // to picker.
+  std::vector<base::string16> targets{base::ASCIIToUTF16(kManifestURL)};
 
   ShowPickerDialog(targets, base::Bind(&ShareServiceImpl::OnPickerClosed,
                                        base::Unretained(this), title, text,
@@ -151,16 +155,17 @@ void ShareServiceImpl::OnPickerClosed(const std::string& title,
                                       const std::string& text,
                                       const GURL& share_url,
                                       const ShareCallback& callback,
-                                      SharePickerResult result) {
-  if (result == SharePickerResult::CANCEL) {
+                                      base::Optional<base::string16> result) {
+  if (!result.has_value()) {
     callback.Run(base::Optional<std::string>("Share was cancelled"));
     return;
   }
 
-  // TODO(constantina): replace hard-coded URL with one from user-chosen site.
-  constexpr char kUrlBase[] = "https://wicg.github.io/web-share-target/";
+  // TODO(constantina): use manifest URL in result to look up corresponding URL
+  // template.
   constexpr char kUrlTemplate[] =
-      "demos/sharetarget.html?title={title}&text={text}&url={url}";
+      "https://wicg.github.io/web-share-target/demos/"
+      "sharetarget.html?title={title}&text={text}&url={url}";
 
   std::string url_template_filled;
   if (!ReplacePlaceholders(kUrlTemplate, title, text, share_url,
@@ -170,7 +175,7 @@ void ShareServiceImpl::OnPickerClosed(const std::string& title,
     return;
   }
 
-  GURL target_url(kUrlBase + url_template_filled);
+  GURL target_url(url_template_filled);
   if (!target_url.is_valid()) {
     callback.Run(base::Optional<std::string>(
         "Error: url of share target is not a valid url."));