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

Side by Side Diff: LayoutTests/fast/borders/border-hittest-inlineFlowBox.html

Issue 1034433003: Hittest for RootInlineBox/InlineFlowBox should take care of border-radius. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | LayoutTests/fast/borders/border-hittest-inlineFlowBox-expected.txt » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <!doctype html>
2 <html>
3 <head>
4 <script src="../../resources/js-test.js"></script>
5 <style>
6 body { margin: 2em; }
leviw_travelin_and_unemployed 2015/03/27 17:42:26 Why?
Abhijeet Kandalkar Slow 2015/03/30 05:42:20 Was added for manual test. Removed it.
7
8 span {
9 background-color: lightgray;
10 border-radius: 2em;
11 padding: 1em;
12 line-height: 4em;
13 }
14
15 span:hover {
leviw_travelin_and_unemployed 2015/03/27 17:42:26 Are these hover styles neessary?
Abhijeet Kandalkar Slow 2015/03/30 05:42:20 Was added for manual test. Removed it.
16 background-color: red;
17 }
18
19 label {
20 padding: 2em;
21 background-color: lightgreen;
22 border-radius: 2em;
23 }
24
25 label:hover {
26 background-color: green;
27 }
28
29 </style>
30 </head>
31 <body>
32 <span id="A">
33 Two lines with <br/>a hard line break.
34 </span>
35 <br/><br/><br/>
36 <span id="B">B<label href="#" id="C">C</label></span>
37 <br/><br /><br />
38 <span id="D" style="padding: 2em;">D<label id="E" href="#" style="padding: 1em;" >E</label></span>
39 <br><br><br>
leviw_travelin_and_unemployed 2015/03/27 17:42:26 Are all these BRs necessary? Can you take them out
Abhijeet Kandalkar Slow 2015/03/30 05:42:20 Unnecessary <br/> are removed.
40 <div id="console"></div>
41 </body>
42 <script>
43 description('If this test passes, area outside border radius is body element.');
44
45 var rects = document.getElementById('A').getClientRects();
46 var elementInTopLeftCorner = document.elementFromPoint(rects[0].left, rects[0].t op);
47 shouldBeEqualToString('elementInTopLeftCorner.nodeName', 'BODY');
48 elementInTopLeftCorner = document.elementFromPoint(rects[1].left, rects[1].top);
49 shouldBeEqualToString('elementInTopLeftCorner.nodeName', 'SPAN');
50
51 var rect = document.getElementById('B').getBoundingClientRect();
52 elementInTopLeftCorner = document.elementFromPoint(rect.left, rect.top);
53 shouldBeEqualToString('elementInTopLeftCorner.nodeName', 'BODY');
54
55 rect = document.getElementById('C').getBoundingClientRect();
56 elementInTopLeftCorner = document.elementFromPoint(rect.left, rect.top);
57 shouldBeEqualToString('elementInTopLeftCorner.nodeName', 'BODY');
58
59 rect = document.getElementById('D').getBoundingClientRect();
60 elementInTopLeftCorner = document.elementFromPoint(rect.left, rect.top);
61 shouldBeEqualToString('elementInTopLeftCorner.nodeName', 'BODY');
62
63 rect = document.getElementById('E').getBoundingClientRect();
64 elementInTopLeftCorner = document.elementFromPoint(rect.left, rect.top);
65 shouldBeEqualToString('elementInTopLeftCorner.nodeName', 'SPAN');
66 </script>
67 </html>
OLDNEW
« no previous file with comments | « no previous file | LayoutTests/fast/borders/border-hittest-inlineFlowBox-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698