New issue
Advanced search Search tips

Issue 720303 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 724929
issue 490015



Sign in to add a comment

Revive the removed sendBeacon redirect test

Project Member Reported by tyoshino@chromium.org, May 10 2017

Issue description

Revive the item in beacon-cross-origin-redirect-blob.html removed by https://codereview.chromium.org/2870383002 when we re-enable sendBeacon with non-CORS-safelisted Content-Type.

 
Blockedon: 490015
Project Member

Comment 2 by bugdroid1@chromium.org, May 12 2017

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

commit 234e40e94c24453189eade9c7b3f745669e9208f
Author: tyoshino <tyoshino@chromium.org>
Date: Fri May 12 10:54:49 2017

Have sendBeacon throw for Blobs with a type that is not CORS-safelisted.

Intent to Temporarily Remove thread:
https://groups.google.com/a/chromium.org/d/topic/blink-dev/dAfYF2gauw4/discussion

Pre-discussion happened at blink-api-owners-discuss@:
https://groups.google.com/a/chromium.org/d/topic/blink-api-owners-discuss/3_6C7wWviAI/discussion

BUG= 490015 ,720303
R=yhirano@chromium.org,mkwst@chromium.org

Review-Url: https://codereview.chromium.org/2870383002
Cr-Commit-Position: refs/heads/master@{#471268}

[modify] https://crrev.com/234e40e94c24453189eade9c7b3f745669e9208f/third_party/WebKit/LayoutTests/TestExpectations
[add] https://crrev.com/234e40e94c24453189eade9c7b3f745669e9208f/third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob-expected.txt
[modify] https://crrev.com/234e40e94c24453189eade9c7b3f745669e9208f/third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob.html
[delete] https://crrev.com/9416cfd9144a5503f449936f2e47e91c54d13235/third_party/WebKit/LayoutTests/virtual/stable/http/tests/sendbeacon/beacon-cross-origin-redirect-blob-expected.txt
[modify] https://crrev.com/234e40e94c24453189eade9c7b3f745669e9208f/third_party/WebKit/Source/modules/beacon/NavigatorBeacon.cpp
[modify] https://crrev.com/234e40e94c24453189eade9c7b3f745669e9208f/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.json5

Project Member

Comment 3 by bugdroid1@chromium.org, May 18 2017

Labels: merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a267516c16419debb0559659fd7e2ef065cb8c91

commit a267516c16419debb0559659fd7e2ef065cb8c91
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Thu May 18 01:32:59 2017

Have sendBeacon throw for Blobs with a type that is not CORS-safelisted.

Intent to Temporarily Remove thread:
https://groups.google.com/a/chromium.org/d/topic/blink-dev/dAfYF2gauw4/discussion

Pre-discussion happened at blink-api-owners-discuss@:
https://groups.google.com/a/chromium.org/d/topic/blink-api-owners-discuss/3_6C7wWviAI/discussion

BUG= 490015 ,720303
R=yhirano@chromium.org,mkwst@chromium.org

Review-Url: https://codereview.chromium.org/2870383002
Cr-Original-Commit-Position: refs/heads/master@{#471268}
Review-Url: https://codereview.chromium.org/2889683004 .
Cr-Commit-Position: refs/branch-heads/3071@{#614}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/a267516c16419debb0559659fd7e2ef065cb8c91/third_party/WebKit/LayoutTests/TestExpectations
[add] https://crrev.com/a267516c16419debb0559659fd7e2ef065cb8c91/third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob-expected.txt
[modify] https://crrev.com/a267516c16419debb0559659fd7e2ef065cb8c91/third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob.html
[delete] https://crrev.com/eda6f4cf4ff69bea98350b37c43609fae020175c/third_party/WebKit/LayoutTests/virtual/stable/http/tests/sendbeacon/beacon-cross-origin-redirect-blob-expected.txt
[modify] https://crrev.com/a267516c16419debb0559659fd7e2ef065cb8c91/third_party/WebKit/Source/modules/beacon/NavigatorBeacon.cpp
[modify] https://crrev.com/a267516c16419debb0559659fd7e2ef065cb8c91/third_party/WebKit/Source/platform/RuntimeEnabledFeatures.json5

Blockedon: 724929
Owner: yhirano@chromium.org

Sign in to add a comment