 Chromium Code Reviews
 Chromium Code Reviews Issue 340713003:
  Ignore visibility when checking whether Position is editable or not at DeletetSelectionCommand  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master
    
  
    Issue 340713003:
  Ignore visibility when checking whether Position is editable or not at DeletetSelectionCommand  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/editing/execCommand/delete-hidden-crash.html | 
| diff --git a/LayoutTests/editing/execCommand/delete-hidden-crash.html b/LayoutTests/editing/execCommand/delete-hidden-crash.html | 
| new file mode 100644 | 
| index 0000000000000000000000000000000000000000..0932feb7316658995c87f774ef1dade5d4627c58 | 
| --- /dev/null | 
| +++ b/LayoutTests/editing/execCommand/delete-hidden-crash.html | 
| @@ -0,0 +1,40 @@ | 
| +<!DOCTYPE html> | 
| +<html> | 
| +<head> | 
| +<script> | 
| +if (window.testRunner) | 
| + testRunner.dumpAsText(); | 
| + | 
| +onload = function(){ | 
| 
Yuta Kitamura
2014/06/18 08:08:11
Space required before the brace.
 
yosin_UTC9
2014/06/19 07:36:56
Done.
 | 
| + document.execCommand('SelectAll') | 
| 
Yuta Kitamura
2014/06/18 08:08:11
Bad indentation.
 
yosin_UTC9
2014/06/19 07:36:57
Done.
 | 
| + span = document.getElementsByTagName('span')[0]; | 
| + span.contentEditable = 'true'; | 
| + span.textContent = 'bar'; | 
| + document.execCommand('InsertText', false, '1'); | 
| + document.body.textContent = 'PASS if Blink doesn\'t crash.'; | 
| +}; | 
| +</script> | 
| +<style> | 
| +* { visibility:visible; } | 
| +.inline { display:inline; } | 
| +*:only-child { visibility:hidden; } | 
| +</style> | 
| +</head> | 
| +<body> | 
| +<table> | 
| + <tbody> | 
| 
Yuta Kitamura
2014/06/18 08:08:11
Indentation is not necessary here.
If you need th
 
yosin_UTC9
2014/06/19 07:36:56
Done.
 | 
| + <tr> | 
| + <td> | 
| + <span> | 
| + <table></table><span></span> | 
| + </span> | 
| + </td> | 
| + </tr> | 
| + <tr> | 
| + <td> | 
| + <td class="inline"> | 
| + foo | 
| + </tr> | 
| + </tbody> | 
| 
Yuta Kitamura
2014/06/18 08:08:11
This test case is very hard to understand...
It w
 
yosin_UTC9
2014/06/19 07:36:56
Done.
 | 
| +</body> | 
| 
Yuta Kitamura
2014/06/18 08:08:11
No close tag of <table>?
 
yosin_UTC9
2014/06/19 07:36:56
Done.
 | 
| +</html> |