Closed Bug 1580408 Opened 6 years ago Closed 6 years ago

[wpt-sync] Sync PR 18987 - [resource-timing] Report performance entries with failing status codes

Categories

(Core :: Performance: General, task, P4)

Product:
Core
Component:
Performance: General
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla71
Tracking Flags:
Tracking Status
firefox71 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 18987 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/18987
Details from upstream follow.

Yoav Weiss <yoavweiss@chromium.org> wrote:

[resource-timing] Report performance entries with failing status codes

Currently we don't report performance entries with failing status codes.
From the spec's perspective, reporting aborts is a MAY, but failing
status code responses should not be considered aborts. [1]
Chromium is the only engine which doesn't report those entries.
This CL fixes that to report them similarly to successful status codes.

Bug: 883400, 990849
Change-Id: Ic5e99e3df77f3869aa0dd70f0141d88016fdb972

[1] https://github.com/w3c/resource-timing/issues/165#issuecomment-441413636

Reviewed-on: https://chromium-review.googlesource.com/1796544
WPT-Export-Revision: 2226ebf75abf14c91edb14149cc6d55319c9c6e9

Component: web-platform-tests → Performance
Product: Testing → Core
Ran 2 tests and 3 subtests OK : 2 PASS : 1[linux64-asan-opt,linux64-qr-debug,windows10-64-debug,windows7-32-opt], 2[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview,linux64-debug,linux64-opt,linux64-qr-opt,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows7-32-debug] FAIL : 1[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview,linux64-debug,linux64-opt,linux64-qr-opt,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows7-32-debug], 2[linux64-asan-opt,linux64-qr-debug,windows10-64-debug,windows7-32-opt] New tests that have failures or other problems: /resource-timing/status-codes-create-entry.html Make sure all status codes are reported: FAIL[linux64-asan-opt,linux64-qr-debug,windows10-64-debug,windows7-32-opt]
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/4505f6c2d204 [wpt PR 18987] - [resource-timing] Report performance entries with failing status codes, a=testonly https://hg.mozilla.org/integration/mozilla-inbound/rev/5887d105d88f [wpt PR 18987] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/4505f6c2d204
https://hg.mozilla.org/mozilla-central/rev/5887d105d88f

Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox71: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.