Remove some pointless instructions
authorAryeh Gregor <AryehGregor+gitcommit@gmail.com>
Mon, 30 May 2011 12:01:09 -0600
changeset 192 b23b2abea8cb
parent 191 924539fdd89d
child 193 fdd3bfd4b2fc
Remove some pointless instructions
editcommands.html
implementation.js
source.html
--- a/editcommands.html	Sun May 29 16:11:32 2011 -0600
+++ b/editcommands.html	Mon May 30 12:01:09 2011 -0600
@@ -35,7 +35,7 @@
 <body class=draft>
 <div class=head id=head>
 <h1>HTML Editing Commands</h1>
-<h2 class="no-num no-toc" id=work-in-progress-&mdash;-last-update-29-may-2011>Work in Progress &mdash; Last Update 29 May 2011</h2>
+<h2 class="no-num no-toc" id=work-in-progress-&mdash;-last-update-30-may-2011>Work in Progress &mdash; Last Update 30 May 2011</h2>
 <dl>
  <dt>Editor
  <dd>Aryeh Gregor &lt;[email protected]&gt;
@@ -4197,15 +4197,7 @@
   <li>Set <var title="">range</var>'s <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#concept-range-start title=concept-range-start>start</a> and <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#concept-range-end title=concept-range-end>end</a> to
   (<var title="">node</var>, <var title="">offset</var>).
 
-  <li>If <var title="">node</var> has an <code class=external data-anolis-spec=domcore><a href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#element>Element</a></code> <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#concept-tree-child title=concept-tree-child>child</a> of <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#concept-indexof title=concept-indexof>index</a>
-  <var title="">offset</var>, let <var title="">container</var> equal that <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#concept-tree-child title=concept-tree-child>child</a>.
-
-  <p class=XXX>This doesn't seem to make sense.  Investigate.
-
-  <li>Otherwise, if <var title="">node</var> has an <code class=external data-anolis-spec=domcore><a href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#element>Element</a></code> <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#concept-tree-child title=concept-tree-child>child</a> of <a class=external data-anolis-spec=domrange href=http://html5.org/specs/dom-range.html#concept-indexof title=concept-indexof>index</a>
-  <var title="">offset</var> minus one, let <var title="">container</var> equal that <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#concept-tree-child title=concept-tree-child>child</a>.
-
-  <li>Otherwise, let <var title="">container</var> equal <var title="">node</var>.
+  <li>Let <var title="">container</var> equal <var title="">node</var>.
 
   <li>While <var title="">container</var> is not a <a href=#single-line-container>single-line container</a>,
   and <var title="">container</var>'s <a class=external data-anolis-spec=domcore href=http://dvcs.w3.org/hg/domcore/raw-file/tip/Overview.html#concept-tree-parent title=concept-tree-parent>parent</a> is <a href=#editable>editable</a> and <a href=#in-the-same-editing-host>in
--- a/implementation.js	Sun May 29 16:11:32 2011 -0600
+++ b/implementation.js	Mon May 30 12:01:09 2011 -0600
@@ -3186,24 +3186,8 @@
 		range.setStart(node, offset);
 		range.setEnd(node, offset);
 
-		// "If node has an Element child of index offset, let container equal
-		// that child."
-		var container;
-		if (offset < node.childNodes.length
-		&& node.childNodes[offset].nodeType == Node.ELEMENT_NODE) {
-			container = node.childNodes[offset];
-
-		// "Otherwise, if node has an Element child of index offset minus one,
-		// let container equal that child."
-		} else if (offset > 0
-		&& offset - 1 < node.childNodes.length
-		&& node.childNodes[offset - 1].nodeType == Node.ELEMENT_NODE) {
-			container = node.childNodes[offset - 1];
-
-		// "Otherwise, let container equal node."
-		} else {
-			container = node;
-		}
+		// "Let container equal node."
+		var container = node;
 
 		// "While container is not a single-line container, and container's
 		// parent is editable and in the same editing host as node, set
--- a/source.html	Sun May 29 16:11:32 2011 -0600
+++ b/source.html	Mon May 30 12:01:09 2011 -0600
@@ -4232,15 +4232,7 @@
   <li>Set <var>range</var>'s [[rangestart]] and [[rangeend]] to
   (<var>node</var>, <var>offset</var>).
 
-  <li>If <var>node</var> has an [[element]] [[child]] of [[index]]
-  <var>offset</var>, let <var>container</var> equal that [[child]].
-
-  <p class=XXX>This doesn't seem to make sense.  Investigate.
-
-  <li>Otherwise, if <var>node</var> has an [[element]] [[child]] of [[index]]
-  <var>offset</var> minus one, let <var>container</var> equal that [[child]].
-
-  <li>Otherwise, let <var>container</var> equal <var>node</var>.
+  <li>Let <var>container</var> equal <var>node</var>.
 
   <li>While <var>container</var> is not a <span>single-line container</span>,
   and <var>container</var>'s [[parent]] is <span>editable</span> and <span>in