Change notIfViewed to scroll:"always"/"if-needed"

Author: zcorpan

cssom-view-1/Overview.bs

@@ -1078,14 +1078,15 @@ Note: This {{DOMRect}} object is not <a spec=html>live</a>.
 <h2 id=extension-to-the-element-interface>Extensions to the {{Element}} Interface</h2>
 
 <pre class=idl>
+enum ScrollIntoViewMode { "always", "if-needed" };
 enum ScrollLogicalPosition { "start", "center", "end", "nearest" };
 dictionary ScrollIntoViewOptions : ScrollOptions {
-  boolean notIfViewed = false;
+  ScrollIntoViewMode scroll = "always";
   ScrollLogicalPosition block = "start";
   ScrollLogicalPosition inline = "nearest";
 };
 dictionary FocusScrollOptions : ScrollOptions {
-  boolean notIfViewed = true;
+  ScrollIntoViewMode scroll = "if-needed";
   ScrollLogicalPosition block;
   ScrollLogicalPosition inline;
 };
@@ -1158,16 +1159,16 @@ The <dfn method for=Element caniuse=scrollintoview>scrollIntoView(<var>arg</var>
 1. Let <var>behavior</var> be "<code>auto</code>".
 1. Let <var>block</var> be "<code>start</code>".
 1. Let <var>inline</var> be "<code>nearest</code>".
-1. Let <var>notIfViewed</var> be false.
+1. Let <var>scrollMode</var> be "<code>always</code>".
 1. If <var>arg</var> is a {{ScrollIntoViewOptions}} dictionary, then:
     1. Set <var>behavior</var> to the {{ScrollOptions/behavior}} dictionary member of <var>options</var>.
     1. Set <var>block</var> to the {{ScrollIntoViewOptions/block}} dictionary member of <var>options</var>.
     1. Set <var>inline</var> to the {{ScrollIntoViewOptions/inline}} dictionary member of <var>options</var>.
-    1. Set <var>notIfViewed</var> to the {{ScrollIntoViewOptions/notIfViewed}} dictionary member of <var>options</var>.
+    1. Set <var>scrollMode</var> to the {{ScrollIntoViewOptions/scroll}} dictionary member of <var>options</var>.
 1. Otherwise, if <var>arg</var> is false, then set <var>block</var> to "<code>end</code>".
 1. If the element does not have any associated <a>layout box</a>, then return.
 1. <a lt='scroll an element into view'>Scroll the element into view</a>
-    with <var>behavior</var>, <var>block</var>, <var>inline</var>, and <var>notIfViewed</var>.
+    with <var>behavior</var>, <var>block</var>, <var>inline</var>, and <var>scrollMode</var>.
 1. Optionally perform some other action that brings the element to the user's attention.
 
 The <dfn method for=Element lt="scroll(options)|scroll(x, y)">scroll()</dfn> method must run these steps:
@@ -1320,7 +1321,7 @@ The <dfn attribute for=Element>clientHeight</dfn> attribute must run these steps
 
 To <dfn>scroll an element into view</dfn> <var>element</var>,
 with a scroll behavior <var>behavior</var>,
-a boolean indicating to not scroll if the element is already in view <var>notIfViewed</var>,
+a mode indicating whether to scroll if the element is already in view <var>scrollMode</var>,
 optionally a block flow direction position <var>block</var> (undefined if not given),
 and optionally an inline base direction position <var>inline</var> (undefined if not given),
 means to run these steps for each ancestor element or <a>viewport</a> that establishes
@@ -1334,7 +1335,7 @@ a <a>scrolling box</a> <var>scrolling box</var>, in order of innermost to outerm
 1. Let <var>scrolling box edge D</var> be the <a>ending edge</a> in the <a>inline base direction</a> of <var>scrolling box</var>, and let <var>element edge D</var> be <var>element bounding border box</var>'s edge on the same physical side as that of <var>scrolling box edge D</var>.
 1. Let <var>element width</var> be the distance between <var>element edge C</var> and <var>element edge D</var>.
 1. Let <var>scrolling box width</var> be the distance between <var>scrolling box edge C</var> and <var>scrolling box edge D</var>.
-1. If <var>notIfViewed</var> is true, and <var>element</var> is entirely in view already, then return.
+1. If <var>scrollMode</var> is "<code>if-needed</code>", and <var>element</var> is entirely in view already, then return.
 
         ISSUE: Define "entirely in view".