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

chrome/browser/webshare/share_service_impl.cc

 // 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 "chrome/browser/webshare/share_service_impl.h"
 
 #include <algorithm>
 #include <functional>
 #include <utility>
 
@@ skipping 96 matching lines @@
     return false;
   }
   split_template.push_back(url_template.substr(
       start_index_to_copy, url_template.size() - start_index_to_copy));
 
   *url_template_filled = JoinString(split_template);
   return true;
 }
 
 void ShareServiceImpl::ShowPickerDialog(
-    const std::vector<base::string16>& targets,
-    const base::Callback<void(SharePickerResult)>& callback) {
+    const std::vector<std::pair<base::string16, GURL>>& targets,
+    const base::Callback<void(base::Optional<std::string>)>& 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
 }
 
 void ShareServiceImpl::OpenTargetURL(const GURL& target_url) {
 // TODO(constantina): Prevent this code from being run/compiled in android.
 #if defined(OS_LINUX) || defined(OS_WIN)
   Browser* browser = BrowserList::GetInstance()->GetLastActive();
   chrome::AddTabAt(browser, target_url,
                    browser->tab_strip_model()->active_index() + 1, true);
 #endif
 }
 
 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

[Matt Giuca, 2017/02/03 01:44:45]

name and manifest URL

[constantina, 2017/02/03 04:32:39]

Done.

+  // registered targets' manifest URLs.
+  constexpr char kTargetName[] = "Web Share Target Demo App";

[Matt Giuca, 2017/02/03 01:44:45]

s/Demo/Test (no point changing this line from what we had originally, and "Test"
is the title of the page at
https://wicg.github.io/web-share-target/demos/sharetarget.html).

[constantina, 2017/02/03 04:32:39]

Done.

+  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
+  // to picker.
+  std::vector<std::pair<base::string16, GURL>> targets{make_pair(
+      base::ASCIIToUTF16(kTargetName), GURL(kManifestURL))};
 
   ShowPickerDialog(targets, base::Bind(&ShareServiceImpl::OnPickerClosed,
                                        base::Unretained(this), title, text,
                                        share_url, callback));
 }
 
 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<std::string> 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,
                            &url_template_filled)) {
     callback.Run(base::Optional<std::string>(
         "Error: unable to replace placeholders in url template"));
     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."));
     return;
   }
   OpenTargetURL(target_url);
 
   callback.Run(base::nullopt);
 }