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

Unified Diff: third_party/WebKit/LayoutTests/editing/deleting/delete-3928305-fix.html

Issue 2170753003: Convert editing/deleting/delete-3928305-fix.html to use w3c test harness (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 2016-07-22T13:13:45 Created 4 years, 5 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/platform/linux/editing/deleting/delete-3928305-fix-expected.png » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/editing/deleting/delete-3928305-fix.html
diff --git a/third_party/WebKit/LayoutTests/editing/deleting/delete-3928305-fix.html b/third_party/WebKit/LayoutTests/editing/deleting/delete-3928305-fix.html
index 2d6fdad12dcfbac269e2e8d5f398443c90c2097d..e15e00a05e1e1cbc2e5ef67fcedf4fa1508ad136 100644
--- a/third_party/WebKit/LayoutTests/editing/deleting/delete-3928305-fix.html
+++ b/third_party/WebKit/LayoutTests/editing/deleting/delete-3928305-fix.html
@@ -1,49 +1,12 @@
-<html>
-<head>
-
-<script src=../editing.js language="JavaScript" type="text/JavaScript" ></script>
-
-<style>
-.editing {
- border: 2px solid red;
- padding: 12px;
- font-size: 24px;
-}
-</style>
-
+<!doctype html>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script src="../assert_selection.js"></script>
<script>
-
-function editingTest() {
- for (i = 0; i < 4; i++)
- moveSelectionForwardByCharacterCommand();
- for (i = 0; i < 3; i++)
- extendSelectionForwardByCharacterCommand();
- deleteCommand();
-}
-
+test(() => {
+ assert_selection(
+ '<div contenteditable><div><p>foo</p>^bar|<p>baz</p></div></div>',
+ 'Delete',
+ '<div contenteditable><p>foo</p>|<br><p>baz</p></div>');
+}, 'Delete characters between blocks in block inserts BR.');
</script>
-
-<title>Editing Test</title>
-</head>
-<body contenteditable="true">
-
-Should see a blank line between "foo" and "baz"
-<div style="height: 24px"></div>
-
-<div id="root" class="editing">
-<div id="test">foo</div>bar<div>baz</div>
-
-<!--
-
-Fix for this bug:
-
-<rdar://problem/3928305> selecting an entire line and typing over causes new inserted text at top of document
-
--->
-
-<script>
-runEditingTest();
-</script>
-
-</body>
-</html>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/platform/linux/editing/deleting/delete-3928305-fix-expected.png » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698