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

Unified Diff: third_party/WebKit/LayoutTests/fast/dynamic/checkbox-selection-crash.html

Issue 2693813002: Selection API: Some functions should throw InvalidNodeTypeError and IndexSizeError. (Closed)
Patch Set: adjust 2 more tests 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/fast/dynamic/checkbox-selection-crash.html
diff --git a/third_party/WebKit/LayoutTests/fast/dynamic/checkbox-selection-crash.html b/third_party/WebKit/LayoutTests/fast/dynamic/checkbox-selection-crash.html
index 45ae70df157ea73019a2b13b6a770bb29c74cde6..c2ee3c480d9a6373cb6bd2fad3571de5f87c7c95 100644
--- a/third_party/WebKit/LayoutTests/fast/dynamic/checkbox-selection-crash.html
+++ b/third_party/WebKit/LayoutTests/fast/dynamic/checkbox-selection-crash.html
@@ -16,7 +16,7 @@ Loading this page should not crash.
var sel = window.getSelection();
var td1 = document.getElementById('td1')
// having selection triggers Document::updateRendering() from paint()
- sel.setBaseAndExtent(td1, 0, td1, 1000);
+ sel.setBaseAndExtent(td1, 0, td1, 3);
// this causes style recalc and rendering tree tear down (from updateRendering) in middle of painting, which crashes
document.body.setAttribute('class','gone');
var cb = document.getElementById('cb')

Powered by Google App Engine
This is Rietveld 408576698