Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(104)

Unified Diff: third_party/WebKit/LayoutTests/webshare/share-success.html

Issue 2327713002: Revert of Use the URL passed to navigator.share(). (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/webshare/share-success.html
diff --git a/third_party/WebKit/LayoutTests/webshare/share-success.html b/third_party/WebKit/LayoutTests/webshare/share-success.html
index d5d943639351992562dc70a085d9093d5d689189..79000b50d59d01e1b93824c30c280c8dd931d28c 100644
--- a/third_party/WebKit/LayoutTests/webshare/share-success.html
+++ b/third_party/WebKit/LayoutTests/webshare/share-success.html
@@ -5,50 +5,15 @@
<script src="resources/mock-share-service.js"></script>
<script>
-function getAbsoluteUrl(url) {
- return new URL(url, location).toString();
-}
-
share_test((t, mock) => {
- let url = 'https://www.example.com/some/path?some_query#some_fragment';
- mock.pushShareResult('the title', 'the message', getAbsoluteUrl(url), null);
+ mock.pushShareResult('the title', 'the message', null);
return callWithKeyDown(() => navigator.share(
- {title: 'the title', text: 'the message', url: url}));
+ {title: 'the title', text: 'the message', url: 'the URL'}));
}, 'successful share');
share_test((t, mock) => {
- mock.pushShareResult('', '', '', null);
+ mock.pushShareResult('', '', null);
return callWithKeyDown(() => navigator.share({}));
}, 'successful share with empty ShareData');
-share_test((t, mock) => {
- let url = '//www.example.com/some/path?some_query#some_fragment';
- mock.pushShareResult('', '', getAbsoluteUrl(url), null);
- return callWithKeyDown(() => navigator.share({url: url}));
-}, 'successful share with URL without a scheme');
-
-share_test((t, mock) => {
- let url = '/some/path?some_query#some_fragment';
- mock.pushShareResult('', '', getAbsoluteUrl(url), null);
- return callWithKeyDown(() => navigator.share({url: url}));
-}, 'successful share with a path-only URL');
-
-share_test((t, mock) => {
- let url = 'foo';
- mock.pushShareResult('', '', getAbsoluteUrl(url), null);
- return callWithKeyDown(() => navigator.share({url: url}));
-}, 'successful share with a relative URL');
-
-share_test((t, mock) => {
- let url = '';
- mock.pushShareResult('', '', getAbsoluteUrl(url), null);
- return callWithKeyDown(() => navigator.share({url: url}));
-}, 'successful share with an empty URL');
-
-share_test((t, mock) => {
- let url = 'data:foo';
- mock.pushShareResult('', '', getAbsoluteUrl(url), null);
- return callWithKeyDown(() => navigator.share({url: url}));
-}, 'successful share with a data URL');
-
</script>
« no previous file with comments | « third_party/WebKit/LayoutTests/webshare/share-error.html ('k') | third_party/WebKit/LayoutTests/webshare/share-types.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698