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

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

 // 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.
 
 [JavaPackage="org.chromium.webshare.mojom"]
 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,
+  // The URL template was invalid (e.g., mismatching braces).
+  // TODO(mgiuca): This error should not be possible at share time, because
+  // targets with invalid templates should not be chooseable.
+  // https://crbug.com/694380
+  INVALID_TEMPLATE,

[Sam McNally, 2017/02/21 23:02:40]

With the impending removal of this error, can we skip straight to not having it
and use INTERNAL_ERROR instead?

[Matt Giuca, 2017/02/23 04:18:37]

Done.

+};
+
 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);
 };