Skip to content

Commit

Permalink
Bug 1667023 [wpt PR 25755] - Web Share: restrict URL scheme to http a…
Browse files Browse the repository at this point in the history
…nd https, a=testonly

Automatic update from web-platform-tests
Web Share: restrict URL scheme to http and https

We now follow the recent spec change limiting the permitted scheme
for shared urls to http and https - see
w3c/web-share#173
w3c/web-share#174
w3c/web-share#177

We make an exception if the page performing the share it itself loaded
from a different scheme (e.g. file) - in that case we allow the same
scheme to be used for the shared url.

Bug: 1131755
Change-Id: I6abf0f9acd40ef79ec49379314e2ef3a81d3467e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425977
Commit-Queue: Eric Willigers <ericwilligerschromium.org>
Reviewed-by: Glen Robertson <glenrobchromium.org>
Auto-Submit: Eric Willigers <ericwilligerschromium.org>
Cr-Commit-Position: refs/heads/master{#810180}

--

wpt-commits: a28408e23e7cb1e4e8dc070445a51fc2f2d9a4e6
wpt-pr: 25755

UltraBlame original commit: 527027b783378db7e3c1fcdd908e718799400989
  • Loading branch information
marco-c committed Sep 28, 2020
1 parent ed37ecf commit 6248942
Showing 1 changed file with 51 additions and 68 deletions.
119 changes: 51 additions & 68 deletions testing/web-platform/tests/web-share/canShare.tentative.https.html
Original file line number Diff line number Diff line change
Expand Up @@ -88,14 +88,13 @@
=
>
{
assert_equals
assert_false
(
navigator
.
canShare
(
)
false
)
;
}
Expand All @@ -120,7 +119,7 @@
=
>
{
assert_equals
assert_false
(
navigator
.
Expand All @@ -129,7 +128,6 @@
{
}
)
false
)
;
}
Expand All @@ -149,15 +147,14 @@
=
>
{
assert_equals
assert_false
(
navigator
.
canShare
(
undefined
)
false
)
;
}
Expand All @@ -183,15 +180,14 @@
=
>
{
assert_equals
assert_false
(
navigator
.
canShare
(
null
)
false
)
;
}
Expand All @@ -217,7 +213,7 @@
=
>
{
assert_equals
assert_false
(
navigator
.
Expand All @@ -232,7 +228,6 @@
'
}
)
false
)
;
}
Expand Down Expand Up @@ -291,7 +286,7 @@
65536
'
;
assert_equals
assert_false
(
navigator
.
Expand All @@ -301,7 +296,6 @@
url
}
)
false
)
;
}
Expand All @@ -321,7 +315,42 @@
=
>
{
assert_equals
assert_false
(
navigator
.
canShare
(
{
url
:
'
data
:
the
url
'
}
)
)
;
}
'
canShare
with
data
URL
'
)
;
test
(
(
)
=
>
{
assert_false
(
navigator
.
Expand All @@ -333,7 +362,6 @@
undefined
}
)
false
)
;
}
Expand All @@ -358,7 +386,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand All @@ -372,7 +400,6 @@
'
}
)
true
)
;
}
Expand All @@ -390,7 +417,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand All @@ -404,7 +431,6 @@
'
}
)
true
)
;
}
Expand All @@ -422,7 +448,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand Down Expand Up @@ -452,7 +478,6 @@
'
}
)
true
)
;
}
Expand All @@ -470,7 +495,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand All @@ -482,7 +507,6 @@
null
}
)
true
)
;
}
Expand All @@ -501,7 +525,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand All @@ -513,7 +537,6 @@
123
}
)
true
)
;
}
Expand All @@ -531,7 +554,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand Down Expand Up @@ -561,7 +584,6 @@
}
}
)
true
)
;
}
Expand All @@ -579,7 +601,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand Down Expand Up @@ -616,7 +638,6 @@
'
}
)
true
)
;
}
Expand All @@ -635,43 +656,7 @@
=
>
{
assert_equals
(
navigator
.
canShare
(
{
url
:
'
data
:
the
url
'
}
)
true
)
;
}
'
canShare
with
data
URL
'
)
;
test
(
(
)
=
>
{
assert_equals
assert_true
(
navigator
.
Expand All @@ -684,7 +669,6 @@
'
}
)
true
)
;
}
Expand All @@ -703,7 +687,7 @@
=
>
{
assert_equals
assert_true
(
navigator
.
Expand Down Expand Up @@ -731,7 +715,6 @@
'
}
)
true
)
;
}
Expand Down

0 comments on commit 6248942

Please sign in to comment.