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

Unified Diff: third_party/WebKit/LayoutTests/fast/dom/Range/range-spanning-elements-bounding-client-rect.html

Issue 2209513002: Exclude start/end container in Range::getBorderAndTextQuads (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address test failure Created 4 years, 4 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/fast/dom/Range/range-spanning-elements-bounding-client-rect.html
diff --git a/third_party/WebKit/LayoutTests/fast/dom/Range/range-spanning-elements-bounding-client-rect.html b/third_party/WebKit/LayoutTests/fast/dom/Range/range-spanning-elements-bounding-client-rect.html
new file mode 100644
index 0000000000000000000000000000000000000000..dd89f18a10d469bd548a19c846bb8da84ff14251
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/dom/Range/range-spanning-elements-bounding-client-rect.html
@@ -0,0 +1,41 @@
+<!DOCTYPE html>
+<html>
+ <head>
+ <title>Tests getBoundingClientRect for ranges spanning multiple elements</title>
+ <script src="../../../resources/testharness.js"></script>
+ <script src="../../../resources/testharnessreport.js"></script>
+ </head>
+ <style>
+ p { visibility: hidden; }
+ p > span { text-decoration: underline; }
+ </style>
+ <body>
+ <p id="test">Test that getBoundingClientRect <span>returns the correct
+ results for ranges spanning multiple</span> elements.</p>
+ <p id="reference">Test that getBoundingClientRect <span>returns</span>
+ the correct results for ranges spanning multiple elements.</p>
+ <script>
+ test(function() {
+ var element = document.getElementById('test');
+ var span = element.getElementsByTagName('span')[0];
+ var range = document.createRange();
+ range.setStart(element.firstChild, 10);
+ range.setEnd(element.firstElementChild.firstChild, 7);
+
+ element = document.getElementById('reference');
+ var reference = document.createRange();
+ reference.setStart(element.firstChild, 10);
+ reference.setEnd(element.firstElementChild.firstChild, 7);
+
+ assert_equals(
+ Math.round(range.getBoundingClientRect().width),
+ Math.round(reference.getBoundingClientRect().width),
+ 'Width of range should match reference range.');
+ assert_less_than(
+ Math.round(range.getBoundingClientRect().width),
+ Math.round(span.getBoundingClientRect().width),
+ 'Width of range should be less than inner span.');
+ }, 'Check reported width of range.');
+ </script>
+ </body>
+</html>
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/dom/Range.cpp » ('j') | third_party/WebKit/Source/core/dom/Range.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698