webshare: Small refactor of WebShare (OnceCallback, const&, and alias)

chrome/browser/webshare/share_service_impl.h

Index: chrome/browser/webshare/share_service_impl.h
diff --git a/chrome/browser/webshare/share_service_impl.h b/chrome/browser/webshare/share_service_impl.h
index 7fb1609314efcded6284bc0b1d9b5da524fb58b2..98bae06efe04ebcda41b0a5a0bd531749d5b673e 100644
--- a/chrome/browser/webshare/share_service_impl.h
+++ b/chrome/browser/webshare/share_service_impl.h
@@ -57,7 +57,7 @@ class ShareServiceImpl : public blink::mojom::ShareService {
   // target, or is null if the user cancelled the share. Virtual for testing.
   virtual void ShowPickerDialog(
       const std::vector<std::pair<base::string16, GURL>>& targets,
-      const base::Callback<void(base::Optional<std::string>)>& callback);
+      const chrome::WebShareTargetPickerCallback& callback);
 
   // Opens a new tab and navigates to |target_url|.
   // Virtual for testing purposes.
@@ -88,7 +88,7 @@ class ShareServiceImpl : public blink::mojom::ShareService {
                       const std::string& text,
                       const GURL& share_url,
                       const ShareCallback& callback,
-                      base::Optional<std::string> result);
+                      const base::Optional<std::string>& result);
 
   base::WeakPtrFactory<ShareServiceImpl> weak_factory_;