Editorial: move COOP access report endpoint checks

"Queue violation reports for accesses" started with steps that used
coop, coopValue, and disposition, none of which it defines. Move the
null reporting endpoint check into the "access to" report algorithms,
as the "access from" ones already have it, and remove the rest.

Also remove a stray comma nearby.

This is a follow-up to aa347f3.

Author: annevk

source

@@ -94974,7 +94974,7 @@ interface <dfn interface data-x="dom-Origin-interface">Origin</dfn> {
     <var>isInitialAboutBlank</var>, <var>responseOrigin</var>,
     <var>currentCOOPEnforcementResult</var>'s <span data-x="coop-enforcement-origin">origin</span>,
     <var>responseCOOP</var>, and <var>currentCOOPEnforcementResult</var>'s <span
-    data-x="coop-enforcement-coop">opener policy</span>, is true:</p>
+    data-x="coop-enforcement-coop">opener policy</span> is true:</p>
 
     <ol>
      <li><p>Set <var>newCOOPEnforcementResult</var>'s <span data-x="coop-enforcement-bcg-switch-report-only">would
@@ -95410,16 +95410,6 @@ interface <dfn interface data-x="dom-Origin-interface">Origin</dfn> {
   object</span> <var>environment</var>:</p>
 
   <ol>
-   <li><p>If <var>coop</var>'s <span data-x="coop-struct-report-endpoint">reporting endpoint</span>
-   is null, return.</p></li>
-
-   <li><p>Let <var>coopValue</var> be <var>coop</var>'s <span
-   data-x="coop-struct-value">value</span>.</p></li>
-
-   <li><p>If <var>disposition</var> is "<code data-x="">reporting</code>", then set
-   <var>coopValue</var> to <var>coop</var>'s <span
-   data-x="coop-struct-report-only-value">report-only value</span>.</p></li>
-
    <li>
     <p>If <var>accessorAccessedRelationship</var> is <span
     data-x="accessor-accessed-opener">accessor is opener</span>:</p>
@@ -95483,6 +95473,9 @@ interface <dfn interface data-x="dom-Origin-interface">Origin</dfn> {
   <var>referrer</var>, and an <span>environment settings object</span> <var>environment</var>:</p>
 
   <ol>
+   <li><p>If <var>coop</var>'s <span data-x="coop-struct-report-endpoint">reporting endpoint</span>
+   is null, then return.</p></li>
+
    <li><p>Let <var>sourceFile</var>, <var>lineNumber</var>, and <var>columnNumber</var> be the
    relevant script URL and problematic position which triggered this report.</p></li>
 
@@ -95562,6 +95555,9 @@ interface <dfn interface data-x="dom-Origin-interface">Origin</dfn> {
   <var>environment</var>:</p>
 
   <ol>
+   <li><p>If <var>coop</var>'s <span data-x="coop-struct-report-endpoint">reporting endpoint</span>
+   is null, then return.</p></li>
+
    <li><p>Let <var>sourceFile</var>, <var>lineNumber</var>, and <var>columnNumber</var> be the
    relevant script URL and problematic position which triggered this report.</p></li>
 
@@ -95637,6 +95633,9 @@ interface <dfn interface data-x="dom-Origin-interface">Origin</dfn> {
   <var>environment</var>:</p>
 
   <ol>
+   <li><p>If <var>coop</var>'s <span data-x="coop-struct-report-endpoint">reporting endpoint</span>
+   is null, then return.</p></li>
+
    <li><p>Let <var>sourceFile</var>, <var>lineNumber</var>, and <var>columnNumber</var> be the
    relevant script URL and problematic position which triggered this report.</p></li>