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

Unified Diff: LayoutTests/http/tests/loading/sync-xhr-priority-expected.txt

Issue 585873002: Show a warning when using sync xhr. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Updated patch with reset -expected.txt files. Created 6 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: LayoutTests/http/tests/loading/sync-xhr-priority-expected.txt
diff --git a/LayoutTests/http/tests/loading/sync-xhr-priority-expected.txt b/LayoutTests/http/tests/loading/sync-xhr-priority-expected.txt
index 6cf0ac550addc9ed0905a975c7fd6ac28d7ca470..621f014607235d9a41647d8db43a7d15b5db9b1f 100644
--- a/LayoutTests/http/tests/loading/sync-xhr-priority-expected.txt
+++ b/LayoutTests/http/tests/loading/sync-xhr-priority-expected.txt
@@ -1,6 +1,7 @@
main frame - didStartProvisionalLoadForFrame
main frame - didCommitLoadForFrame
http://127.0.0.1:8000/js-test-resources/js-test.js has priority Medium
+CONSOLE WARNING: Avoid synchronous requests as it hampers user experience due to unresponsiveness of the web page. Refer https://xhr.spec.whatwg.org/#sync-warning.
http://127.0.0.1:8000/resources/test.mp4 has priority VeryHigh
main frame - didFinishDocumentLoadForFrame
main frame - didHandleOnloadEventsForFrame

Powered by Google App Engine
This is Rietveld 408576698