New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 736172 link

Starred by 1 user
Status: Started
Owner:
█ mgiuca@chromium.org
Long OOO (go/where-is-mgiuca)
Cc:
█ thorogood@google.com
sa...@chromium.org
█ mgiuca@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug



Sign in to add a comment

navigator.share method can be enabled on Mac (with a flag) but Mojo service doesn't exist

Project Member Reported by mgiuca@chromium.org, Jun 23 2017 
Chrome Version: 61
OS: Mac

What steps will reproduce the problem?
(1) Run Chrome with --enable-experimental-web-platform-features.
(2) https://wicg.github.io/web-share/demos/share.html
(3) Click Share.

What is the expected result?
Error: Unsupported feature: navigator.share
(because Web Share is not implemented on Mac)

What happens instead?
Error sharing: SecurityError: WebShare is disabled.
(An internal error that was never intended to be seen; it means the Mojo service is not registered but navigator.share exists.)

The error message for when the Mojo service isn't registered should be made clearer. And this should be fixed by either launching the feature on Mac behind a flag, or hiding navigator.share on Mac even when the flag is flipped on.
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 30 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/f27805021c0b528d6655d71427bd707b98a0110f

commit f27805021c0b528d6655d71427bd707b98a0110f
Author: Matt Giuca <mgiuca@chromium.org>
Date: Fri Jun 30 06:46:48 2017

navigator.share: Made error message for missing Mojo interface clearer.

Also changed from SecurityError to AbortError (it's an internal error).

Bug: 736172
Change-Id: Ie59910de72986b37ee993241c4bc9d0858c8ec70
Reviewed-on: https://chromium-review.googlesource.com/544383
Reviewed-by: Sam McNally <sammc@chromium.org>
Commit-Queue: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#483645}
[modify] https://crrev.com/f27805021c0b528d6655d71427bd707b98a0110f/third_party/WebKit/Source/modules/webshare/NavigatorShare.cpp
►

Sign in to add a comment