Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(922)

Unified Diff: third_party/WebKit/LayoutTests/editing/selection/legal-positions.html

Issue 2695193002: Selection API: setBaseAndExtent() and selectAllChildren() should create new Range based on the spec… (Closed)
Patch Set: rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/editing/selection/legal-positions.html
diff --git a/third_party/WebKit/LayoutTests/editing/selection/legal-positions.html b/third_party/WebKit/LayoutTests/editing/selection/legal-positions.html
index 679f6ec57dbb548ee4727cf9810060bf095a713f..f87f80c4da3982787d610db856e7cc3492475876 100644
--- a/third_party/WebKit/LayoutTests/editing/selection/legal-positions.html
+++ b/third_party/WebKit/LayoutTests/editing/selection/legal-positions.html
@@ -33,10 +33,10 @@ if (window.testRunner)
// The textnode
s.setBaseAndExtent(e, 0, e, 1);
r = s.getRangeAt(0);
-assert(r.startContainer == e.childNodes[0]);
+assert(r.startContainer == e);
assert(r.startOffset == 0);
-assert(r.endContainer == e.childNodes[0]);
-assert(r.endOffset == r.endContainer.length);
+assert(r.endContainer == e);
+assert(r.endOffset == 1);
// The image
s.setBaseAndExtent(e, 1, e, 2);

Powered by Google App Engine
This is Rietveld 408576698