New issue
Advanced search Search tips

Issue 847789 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Sep 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

[Missing Test]: CORB: update the console message

Project Member Reported by viswa.karala@chromium.org, May 30 2018

Issue description

Automated tests for the below commit have been missing.Would it be possible to add test coverage to avoid regressions in future?

CL: 
----
https://chromium.googlesource.com/chromium/src.git/+/4742a48e3739bb63798d9bf2e5aafdfe3af1d044

Ref Bug: 
---------
https://bugs.chromium.org/p/chromium/issues/detail?id=829874

Thank you.
 
Cc: lukasza@chromium.org
Owner: pfeldman@chromium.org
Reassigning to pfeldman@ who introduced the console message in r526624.  I personally think that the current verification (checking response.params.blockedCrossSiteDocument boolean flag in LayoutTests/http/tests/inspector-protocol/network/block_cross_site_document_load.js) is fine and may be sufficient.  OTOH, maybe there is an easy way to test the contents of the message itself (e.g. to verify that the placeholders like %s got correctly populated).
Status: WontFix (was: Assigned)

Sign in to add a comment