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

Unified Diff: third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation

Issue 2720763002: PlzNavigate: preserve SourceLocation when navigating (Closed)
Patch Set: Addressed nate's comments Created 3 years, 10 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/FlagExpectations/enable-browser-side-navigation
diff --git a/third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation b/third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation
index 46366bc5627bf1f9976b0768bade3bdb7ce13823..7b38778053c538c5fb31f0ede3b2cdc978ebe755 100644
--- a/third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation
+++ b/third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation
@@ -4,9 +4,6 @@
# https://crbug.com/551000: PlzNavigate: DevTools support
crbug.com/551000 http/tests/inspector/network/x-frame-options-deny.html [ Failure ]
crbug.com/551000 virtual/mojo-loading/http/tests/inspector/network/x-frame-options-deny.html [ Failure ]
-# Console error messages are wrongly ordered.
-crbug.com/551000 http/tests/inspector/console-resource-errors.html [ Failure ]
-crbug.com/551000 virtual/mojo-loading/http/tests/inspector/console-resource-errors.html [ Failure ]
# Flipped order between diStartProvisionalLoad & willSendRequest
crbug.com/647698 http/tests/loading/307-after-303-after-post.html [ Failure ]
@@ -17,10 +14,8 @@ crbug.com/625765 virtual/mojo-loading/http/tests/loading/redirect-methods.html [
# https://crbug.com/555418: Move `X-Frame-Options` and CSP's `frame-ancestor`
# checks up out of the renderer.
crbug.com/555418 http/tests/security/contentSecurityPolicy/1.1/form-action-src-redirect-blocked.html [ Failure ]
-crbug.com/555418 http/tests/security/contentSecurityPolicy/redirect-does-not-match-paths.html [ Timeout ]
crbug.com/555418 http/tests/security/isolatedWorld/bypass-main-world-csp-iframes.html [ Failure ]
crbug.com/555418 virtual/mojo-loading/http/tests/security/contentSecurityPolicy/1.1/form-action-src-redirect-blocked.html [ Failure ]
-crbug.com/555418 virtual/mojo-loading/http/tests/security/contentSecurityPolicy/redirect-does-not-match-paths.html [ Timeout ]
nasko 2017/03/02 23:56:05 Are these really passing with just this fix? If ye
clamy 2017/03/06 13:27:53 Yes they were incorrectly classified.
crbug.com/555418 virtual/mojo-loading/http/tests/security/isolatedWorld/bypass-main-world-csp-iframes.html [ Failure ]
# Issue with ServiceWorker timing.
@@ -32,10 +27,8 @@ Bug(none) external/wpt/html/browsers/history/the-location-interface/location-pro
# Missing error messages.
Bug(none) external/wpt/html/browsers/history/the-location-interface/location-protocol-setter-non-broken.html [ Failure ]
-# https://crbug.com/695072: Preserve SourceLocation information.
-# This results in a missing line number in console error messages.
-Bug(695072) http/tests/security/location-href-clears-username-password.html [ Failure ]
-Bug(695072) virtual/mojo-loading/http/tests/security/location-href-clears-username-password.html [ Failure ]
+# Timeout
+Bug(none) fast/loader/crash-replacing-location-before-load.html [ Timeout ]
# These tests are flaky.
Bug(none) external/wpt/service-workers/service-worker/fetch-event-redirect.https.html [ Crash ]

Powered by Google App Engine
This is Rietveld 408576698