Chromium Code Reviews Chromium Code Reviews
Issue 1034433003:
Hittest for RootInlineBox/InlineFlowBox should take care of border-radius. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/borders/border-hittest-inlineFlowBox.html |
| diff --git a/LayoutTests/fast/borders/border-hittest-inlineFlowBox.html b/LayoutTests/fast/borders/border-hittest-inlineFlowBox.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..fba26d9608917d79b78bb6dc0e4f5f17a990701e |
| --- /dev/null |
| +++ b/LayoutTests/fast/borders/border-hittest-inlineFlowBox.html |
| @@ -0,0 +1,34 @@ |
| +<!doctype html> |
| +<html> |
| +<head> |
| +<script src="../../resources/js-test.js"></script> |
| +<style> |
| +body { margin: 2em; } |
| + |
| +#test { |
| + background-color: green; |
| + border-radius: 50px; |
| + padding: 1em; |
| +} |
| + |
| +#test:hover { |
|
davve
2015/03/24 13:26:03
You want to keep the hover rule for manual testing
Abhijeet Kandalkar Slow
2015/03/24 14:42:44
Yes, it is for manual testing. Keeping span:hover
|
| + background-color:red; |
| +} |
| +</style> |
| +</head> |
| +<body> |
| +<span id="test"> |
| +<button>dummy</button> |
| +</span> |
| +<br><br><br> |
| +<div id="console"></div> |
|
davve
2015/03/24 13:26:03
You may drop this element? Would probably require
Abhijeet Kandalkar Slow
2015/03/24 14:42:44
Same as above. For better readability during manua
|
| +</body> |
| +<script> |
| +description('If this test passes, area outside is body element.'); |
|
davve
2015/03/24 13:26:03
... area outside border radius is body element ...
Abhijeet Kandalkar Slow
2015/03/24 14:42:44
Done.
|
| +var offset = 4; |
| +var expectedTarget = document.getElementById('test'); |
| +var rect = expectedTarget.getBoundingClientRect(); |
| +var element = document.elementFromPoint(rect.right - offset, rect.bottom - offset); |
| +shouldBe('element.nodeName', '"BODY"'); |
| +</script> |
| +</html> |
