Add DevTools frontend support for port and scheme mismatch issues

Implements DevTools frontend support to show an issue for when
either a port or scheme are mismatched. This issue will only be
shown when kEnableSchemeBoundCookies or kEnablePortBoundCookies
are enabled.

Bug: 376025587
Change-Id: Iece976380330ab398a7e3d8b822a52317a90b010
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5988430
Reviewed-by: Alina Varkki <[email protected]>
Commit-Queue: Amarjot Gill <[email protected]>
Reviewed-by: Wolfgang Beyer <[email protected]>

Author: amarjotgill

config/gni/devtools_grd_files.gni

@@ -413,6 +413,8 @@ grd_files_release_sources = [
   "front_end/models/issues_manager/descriptions/cookieCrossSiteRedirectDowngrade.md",
   "front_end/models/issues_manager/descriptions/cookieExcludeBlockedWithinRelatedWebsiteSet.md",
   "front_end/models/issues_manager/descriptions/cookieExcludeDomainNonAscii.md",
+  "front_end/models/issues_manager/descriptions/cookieExcludePortMismatch.md",
+  "front_end/models/issues_manager/descriptions/cookieExcludeSchemeMismatch.md",
   "front_end/models/issues_manager/descriptions/cookieExcludeThirdPartyPhaseoutRead.md",
   "front_end/models/issues_manager/descriptions/cookieExcludeThirdPartyPhaseoutSet.md",
   "front_end/models/issues_manager/descriptions/cookieWarnDomainNonAscii.md",

front_end/models/issues_manager/BUILD.gn

@@ -172,6 +172,8 @@ devtools_issue_description_files = [
   "cookieWarnThirdPartyPhaseoutRead.md",
   "cookieExcludeThirdPartyPhaseoutSet.md",
   "cookieExcludeThirdPartyPhaseoutRead.md",
+  "cookieExcludePortMismatch.md",
+  "cookieExcludeSchemeMismatch.md",
   "cookieWarnMetadataGrantRead.md",
   "cookieWarnMetadataGrantSet.md",
   "propertyRuleIssue.md",

front_end/models/issues_manager/CookieIssue.ts

@@ -197,6 +197,14 @@ export class CookieIssue extends Issue {
         reason === Protocol.Audits.CookieWarningReason.WarnSameSiteLaxCrossDowngradeStrict) {
       return [Protocol.Audits.InspectorIssueCode.CookieIssue, 'WarnCrossDowngrade', operation, secure].join('::');
     }
+
+    if (reason === Protocol.Audits.CookieExclusionReason.ExcludePortMismatch) {
+      return [Protocol.Audits.InspectorIssueCode.CookieIssue, 'ExcludePortMismatch'].join('::');
+    }
+
+    if (reason === Protocol.Audits.CookieExclusionReason.ExcludeSchemeMismatch) {
+      return [Protocol.Audits.InspectorIssueCode.CookieIssue, 'ExcludeSchemeMismatch'].join('::');
+    }
     return [Protocol.Audits.InspectorIssueCode.CookieIssue, reason, operation].join('::');
   }
 
@@ -542,6 +550,16 @@ const cookieCrossSiteRedirectDowngrade: LazyMarkdownIssueDescription = {
   }],
 };
 
+const ExcludePortMismatch: LazyMarkdownIssueDescription = {
+  file: 'cookieExcludePortMismatch.md',
+  links: [],
+};
+
+const ExcludeSchemeMismatch: LazyMarkdownIssueDescription = {
+  file: 'cookieExcludeSchemeMismatch.md',
+  links: [],
+};
+
 // This description will be used by cookie issues that need to be added to the
 // issueManager, but aren't intended to be surfaced in the issues pane. This
 // is why they are using a placeholder description
@@ -600,4 +618,6 @@ const issueDescriptions: Map<string, LazyMarkdownIssueDescription> = new Map([
   ['CookieIssue::ExcludeThirdPartyPhaseout::ReadCookie', cookieExcludeThirdPartyPhaseoutRead],
   ['CookieIssue::ExcludeThirdPartyPhaseout::SetCookie', cookieExcludeThirdPartyPhaseoutSet],
   ['CookieIssue::CrossSiteRedirectDowngradeChangesInclusion', cookieCrossSiteRedirectDowngrade],
+  ['CookieIssue::ExcludePortMismatch', ExcludePortMismatch],
+  ['CookieIssue::ExcludeSchemeMismatch', ExcludeSchemeMismatch],
 ]);

front_end/models/issues_manager/descriptions/cookieExcludePortMismatch.md

@@ -0,0 +1,8 @@
+# Cookie was not sent due to port mismatch
+
+Cookies can only be accessed by origins that share the same port as the origin
+that initially set the cookie.
+
+If this cookie is required for the request, the cookie will need to be set by an
+origin with the same port as the request. Alternatively, you can utilize the
+Domain attribute, which allows access to a cookie with a mismatched port.

front_end/models/issues_manager/descriptions/cookieExcludeSchemeMismatch.md

@@ -0,0 +1,7 @@
+# Cookie was not sent due to scheme mismatch
+
+The cookie can only be accessed by origins that share the same scheme as
+the origin that initially set it.
+
+If this cookie is required for the request, the cookie will need to be set
+by an origin with the same scheme as the request.