Web Share: Refactor Mojo interface to now return an enum error.

third_party/WebKit/public/platform/modules/webshare/webshare.mojom

Index: third_party/WebKit/public/platform/modules/webshare/webshare.mojom
diff --git a/third_party/WebKit/public/platform/modules/webshare/webshare.mojom b/third_party/WebKit/public/platform/modules/webshare/webshare.mojom
index 71512f8308e0950d11d0ddbf042b257d3d9b1e6a..ef49d1aa3b7ec1e0712116d4ac6ec07262c20474 100644
--- a/third_party/WebKit/public/platform/modules/webshare/webshare.mojom
+++ b/third_party/WebKit/public/platform/modules/webshare/webshare.mojom
@@ -7,11 +7,16 @@ module blink.mojom;
 
 import "url/mojo/url.mojom";
 
+enum ShareError {
+  // Share successful.
+  OK,
+  // Something went wrong in the implementation.
+  INTERNAL_ERROR,
+  // Share was canceled by user.
+  CANCELED,
+};
+
 interface ShareService {
-  // The web contents has requested to send a share text action. Returns null on
-  // success, or an error message on failure.
-  // TODO(mgiuca): Return an error enum, instead of a string (so the actual
-  // error messages are generated in a single place, rather than many service
-  // handlers).
-  Share(string title, string text, url.mojom.Url url) => (string? error);
+  // The web contents has requested to send a share text action.
+  Share(string title, string text, url.mojom.Url url) => (ShareError error);
 };